[U-Boot] Patman / buildman pull request

Hi Tom,
I have brought in the patman changes for this release and also some changes that I consider to be bug fixes. For now I have left out these patches:
http://patchwork.ozlabs.org/patch/231570/ http://patchwork.ozlabs.org/patch/231572/ http://patchwork.ozlabs.org/patch/231575/
however they do improve the operation of patman, so if you are OK with it I would quiet like to bring in the last two at least.
As I mentioned earlier buildman is still not perfect - I am hoping that others will help fix the known deficiencies and find others, as happened with patman.
The following changes since commit c8142633e169665b246352918df5b76fd243bb71:
Prepare v2013.04-rc2 (2013-04-03 15:02:40 -0400)
are available in the git repository at:
git://git.denx.de/u-boot-x86.git patman
for you to fetch changes up to fc3fe1c287fc5ee4c528b4059405571fcd2d55bd:
buildman - U-Boot multi-threaded builder and summary tool (2013-04-04 14:04:35 -0700)
---------------------------------------------------------------- Doug Anderson (2): patman: Make "Reviewed-by" an important tag patman: Allow specifying the message ID your series is in reply to
Simon Glass (14): patman: Use ANSI colours only when outputting to a terminal patman: Use bright ANSI colours by default patman: Add cros_subprocess library to manage subprocesses patman: Make command methods return a CommandResult patman: Allow commands to raise on error, or not patman: Allow reading metadata from a list of commits patman: Add additional git utilties patman: Add Cover-letter-cc tag to Cc cover letter to people patman: Fix up checkpatch parsing to deal with 'CHECK' lines patman: Don't allow spaces in tags patman: Fix the comment in CheckTags to mention multiple tags patman: Minor help message/README fixes patman: Ignore all Gerrit Commit-* tags buildman - U-Boot multi-threaded builder and summary tool
tools/buildman/.gitignore | 1 + tools/buildman/README | 679 ++++++++++++++++++++++++++++ tools/buildman/board.py | 167 +++++++ tools/buildman/bsettings.py | 60 +++ tools/buildman/builder.py | 1445 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ tools/buildman/buildman | 1 + tools/buildman/buildman.py | 126 ++++++ tools/buildman/control.py | 181 ++++++++ tools/buildman/test.py | 185 ++++++++ tools/buildman/toolchain.py | 185 ++++++++ tools/patman/README | 18 +- tools/patman/checkpatch.py | 110 +++-- tools/patman/command.py | 89 +++- tools/patman/commit.py | 7 +- tools/patman/cros_subprocess.py | 397 ++++++++++++++++ tools/patman/gitutil.py | 136 +++++- tools/patman/patchstream.py | 50 ++- tools/patman/patman.py | 8 +- tools/patman/series.py | 11 +- tools/patman/terminal.py | 30 +- tools/patman/test.py | 64 ++- 21 files changed, 3829 insertions(+), 121 deletions(-) create mode 100644 tools/buildman/.gitignore create mode 100644 tools/buildman/README create mode 100644 tools/buildman/board.py create mode 100644 tools/buildman/bsettings.py create mode 100644 tools/buildman/builder.py create mode 120000 tools/buildman/buildman create mode 100755 tools/buildman/buildman.py create mode 100644 tools/buildman/control.py create mode 100644 tools/buildman/test.py create mode 100644 tools/buildman/toolchain.py create mode 100644 tools/patman/cros_subprocess.py
Regards, Simon

On Thu, Apr 04, 2013 at 02:28:26PM -0700, Simon Glass wrote:
Hi Tom,
I have brought in the patman changes for this release and also some changes that I consider to be bug fixes. For now I have left out these patches:
http://patchwork.ozlabs.org/patch/231570/ http://patchwork.ozlabs.org/patch/231572/ http://patchwork.ozlabs.org/patch/231575/
however they do improve the operation of patman, so if you are OK with it I would quiet like to bring in the last two at least.
Go ahead and bring in all 3. I keep trying to convince my more-kernel-than-u-boot coworkers to give patman a spin in kernel-land since they're 2 managing 2-3 N part series by hand.
As I mentioned earlier buildman is still not perfect - I am hoping that others will help fix the known deficiencies and find others, as happened with patman.
The following changes since commit c8142633e169665b246352918df5b76fd243bb71:
Prepare v2013.04-rc2 (2013-04-03 15:02:40 -0400)
are available in the git repository at:
git://git.denx.de/u-boot-x86.git patman
for you to fetch changes up to fc3fe1c287fc5ee4c528b4059405571fcd2d55bd:
buildman - U-Boot multi-threaded builder and summary tool (2013-04-04 14:04:35 -0700)
Doug Anderson (2): patman: Make "Reviewed-by" an important tag patman: Allow specifying the message ID your series is in reply to
Simon Glass (14): patman: Use ANSI colours only when outputting to a terminal patman: Use bright ANSI colours by default patman: Add cros_subprocess library to manage subprocesses patman: Make command methods return a CommandResult patman: Allow commands to raise on error, or not patman: Allow reading metadata from a list of commits patman: Add additional git utilties patman: Add Cover-letter-cc tag to Cc cover letter to people patman: Fix up checkpatch parsing to deal with 'CHECK' lines patman: Don't allow spaces in tags patman: Fix the comment in CheckTags to mention multiple tags patman: Minor help message/README fixes patman: Ignore all Gerrit Commit-* tags buildman - U-Boot multi-threaded builder and summary tool
Applied to u-boot/master, thanks!

Hi Tom,
On Mon, Apr 8, 2013 at 9:54 AM, Tom Rini trini@ti.com wrote:
On Thu, Apr 04, 2013 at 02:28:26PM -0700, Simon Glass wrote:
Hi Tom,
I have brought in the patman changes for this release and also some changes that I consider to be bug fixes. For now I have left out these patches:
http://patchwork.ozlabs.org/patch/231570/ http://patchwork.ozlabs.org/patch/231572/ http://patchwork.ozlabs.org/patch/231575/
however they do improve the operation of patman, so if you are OK with it I would quiet like to bring in the last two at least.
Go ahead and bring in all 3. I keep trying to convince my more-kernel-than-u-boot coworkers to give patman a spin in kernel-land since they're 2 managing 2-3 N part series by hand.
OK, here is a pull request for the next 3, plus one bug fix that I missed last time. This is http://patchwork.ozlabs.org/patch/224458/, which for some reason doesn't show up in my TODO.
The following changes since commit f140b5863b258120f5412ea86733f70c87837ee9:
Merge branch 'patman' of git://git.denx.de/u-boot-x86 (2013-04-08 12:03:22 -0400)
are available in the git repository at:
git://git.denx.de/u-boot-x86.git patman
for you to fetch changes up to 645b271a6039e79b368f027a5624dc0820441733:
patman: Add Series-process-log tag to sort/uniq change logs (2013-04-08 15:21:22 -0700)
---------------------------------------------------------------- Doug Anderson (1): patman: Don't barf if the word 'commit' starts a line
Simon Glass (3): patman: Provide option to ignore bad aliases patman: Add -a option to refrain from test-applying the patches patman: Add Series-process-log tag to sort/uniq change logs
tools/patman/README | 9 ++++++++- tools/patman/gitutil.py | 65 +++++++++++++++++++++++++++++++++++++++++++++++------------------ tools/patman/patchstream.py | 4 ++-- tools/patman/patman.py | 20 ++++++++++++++------ tools/patman/series.py | 19 ++++++++++++++----- 5 files changed, 85 insertions(+), 32 deletions(-)
Regards, Simon

On Mon, Apr 08, 2013 at 03:26:43PM -0700, Simon Glass wrote:
Hi Tom,
On Mon, Apr 8, 2013 at 9:54 AM, Tom Rini trini@ti.com wrote:
On Thu, Apr 04, 2013 at 02:28:26PM -0700, Simon Glass wrote:
Hi Tom,
I have brought in the patman changes for this release and also some changes that I consider to be bug fixes. For now I have left out these patches:
http://patchwork.ozlabs.org/patch/231570/ http://patchwork.ozlabs.org/patch/231572/ http://patchwork.ozlabs.org/patch/231575/
however they do improve the operation of patman, so if you are OK with it I would quiet like to bring in the last two at least.
Go ahead and bring in all 3. I keep trying to convince my more-kernel-than-u-boot coworkers to give patman a spin in kernel-land since they're 2 managing 2-3 N part series by hand.
OK, here is a pull request for the next 3, plus one bug fix that I missed last time. This is http://patchwork.ozlabs.org/patch/224458/, which for some reason doesn't show up in my TODO.
The following changes since commit f140b5863b258120f5412ea86733f70c87837ee9:
Merge branch 'patman' of git://git.denx.de/u-boot-x86 (2013-04-08 12:03:22 -0400)
are available in the git repository at:
git://git.denx.de/u-boot-x86.git patman
for you to fetch changes up to 645b271a6039e79b368f027a5624dc0820441733:
patman: Add Series-process-log tag to sort/uniq change logs (2013-04-08 15:21:22 -0700)
Doug Anderson (1): patman: Don't barf if the word 'commit' starts a line
Simon Glass (3): patman: Provide option to ignore bad aliases patman: Add -a option to refrain from test-applying the patches patman: Add Series-process-log tag to sort/uniq change logs
tools/patman/README | 9 ++++++++- tools/patman/gitutil.py | 65 +++++++++++++++++++++++++++++++++++++++++++++++------------------ tools/patman/patchstream.py | 4 ++-- tools/patman/patman.py | 20 ++++++++++++++------ tools/patman/series.py | 19 ++++++++++++++----- 5 files changed, 85 insertions(+), 32 deletions(-)
Also applied to u-boot/master, thanks!
participants (2)
-
Simon Glass
-
Tom Rini