Patchwork [U-Boot] Patman / buildman pull request

login
register
mail settings
Submitter Simon Glass
Date April 4, 2013, 9:28 p.m.
Message ID <CAPnjgZ1eNONGtUWbVA6bewRv8GNCsoSaPdF=Wfta85B_bx2kkw@mail.gmail.com>
Download mbox
Permalink /patch/233950/
State Accepted
Delegated to: Tom Rini
Headers show

Pull-request

git://git.denx.de/u-boot-x86.git patman

Comments

Simon Glass - April 4, 2013, 9:28 p.m.
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
Tom Rini - April 8, 2013, 4:54 p.m.
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!