Patchwork [RFC,00/10] *-pull-request: cleanup and overhaul

login
register
mail settings
Submitter Darren Hart
Date May 13, 2011, 11:35 p.m.
Message ID <cover.1305329659.git.dvhart@linux.intel.com>
Download mbox
Permalink /patch/3997/
State New, archived
Headers show

Pull-request

git://git.pokylinux.org/poky-contrib.git dvhart/git-pull

Comments

Darren Hart - May 13, 2011, 11:35 p.m.
After some discussion on the oe-core list regarding the *-pull-request scripts
I provided a proposal, received some feedback, and these patches are the result
of that discussion. The Cc list varies from patch to patch a bit more wildly
than is typical as a final sanity test, although it does reflect who commented
and expressed interest in which parts of the overhaul.

Please review the following changes for suitability for inclusion. If you have
any objections or suggestions for improvement, please respond to the patches. If
you agree with the changes, please provide your Acked-by.

The following changes since commit 65e5ebc1d27b0a6cb2e33596d6d32c536b857f0d:

  tcmode-default: fix a typo (2011-05-13 11:02:20 +0100)

are available in the git repository at:
  git://git.pokylinux.org/poky-contrib.git dvhart/git-pull
  http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dvhart/git-pull

Darren Hart (10):
  create-pull-request: alphabetize arguments
  create-pull-request: whitespace cleanup
  create-pull-request: use git request-pull and arbitrary remotes
  create-pull-request: rewrite known private URLs to public URLs
  create-pull-request: provide an RFC mode via -c argument
  send-pull-request: whitespace cleanup
  send-pull-request: remove local mta support
  send-pull-request: fix greedy auto-cc regex
  send-pull-request: don't send all patches to everyone even with -a
  send-pull-request: verify git sendemail config

 scripts/create-pull-request |  176 +++++++++++++++++++----------
 scripts/send-pull-request   |  265 ++++++++++++++++++-------------------------
 2 files changed, 228 insertions(+), 213 deletions(-)
Darren Hart - May 14, 2011, 3:47 a.m.
Note that oe-core has one additional commit to create-pull-request which
I didn't have in poky. I guess the scripts dirs aren't kept in sync the
same way meta is.

If you want to apply these as they are and not wait for v2, you just
need to revert the following commit from oe-core:

commit 84e07ef76bf4f484a2d5e0180632e7a4981a1af7
Author: Martin Jansa <Martin.Jansa@gmail.com>
Date:   Tue Mar 15 11:07:22 2011 +0100

    create-pull-request: switch URL from git.pokylinux.org/poky-contrib
to git.o

    Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>

There is nothing wrong with the patch, I just didn't have it and this
series obsoletes it, so reverting it was easier than dealing with the
merge conflicts ;-)

--
Darren

On 05/13/2011 04:35 PM, Darren Hart wrote:
> After some discussion on the oe-core list regarding the *-pull-request scripts
> I provided a proposal, received some feedback, and these patches are the result
> of that discussion. The Cc list varies from patch to patch a bit more wildly
> than is typical as a final sanity test, although it does reflect who commented
> and expressed interest in which parts of the overhaul.
> 
> Please review the following changes for suitability for inclusion. If you have
> any objections or suggestions for improvement, please respond to the patches. If
> you agree with the changes, please provide your Acked-by.
> 
> The following changes since commit 65e5ebc1d27b0a6cb2e33596d6d32c536b857f0d:
> 
>   tcmode-default: fix a typo (2011-05-13 11:02:20 +0100)
> 
> are available in the git repository at:
>   git://git.pokylinux.org/poky-contrib.git dvhart/git-pull
>   http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dvhart/git-pull
> 
> Darren Hart (10):
>   create-pull-request: alphabetize arguments
>   create-pull-request: whitespace cleanup
>   create-pull-request: use git request-pull and arbitrary remotes
>   create-pull-request: rewrite known private URLs to public URLs
>   create-pull-request: provide an RFC mode via -c argument
>   send-pull-request: whitespace cleanup
>   send-pull-request: remove local mta support
>   send-pull-request: fix greedy auto-cc regex
>   send-pull-request: don't send all patches to everyone even with -a
>   send-pull-request: verify git sendemail config
> 
>  scripts/create-pull-request |  176 +++++++++++++++++++----------
>  scripts/send-pull-request   |  265 ++++++++++++++++++-------------------------
>  2 files changed, 228 insertions(+), 213 deletions(-)
>