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

login
register
mail settings
Submitter Darren Hart
Date May 16, 2011, 10:25 p.m.
Message ID <cover.1305584418.git.dvhart@linux.intel.com>
Download mbox
Permalink /patch/4181/
State New, archived
Headers show

Pull-request

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

Comments

Darren Hart - May 16, 2011, 10:25 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.

V2: do REMOTE_REPO regex per remote
    fix various regex issues
    fix wget ssl issues
    add github and oe (untested) REMOTE support
    simplify git-send-email usage logic, avoid duplicating git confirmation
    add -c, auto-cc per patch option

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 c12126ebf07604f998cb35546aa51d806d97af22:

  create-pull-request: switch URL from git.pokylinux.org/poky-contrib to git.openembedded.org/openembedded-core-contrib (2011-05-13 20:38:20 -0700)

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

Darren Hart (13):
  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
  create-pull-request: add untested oe repository support
  send-pull-request: streamline git-send-email usage
  *pull-request: add copyright, license, and descriptions

Otavio Salvador (2):
  create-pull-request: do not check certificate
  create-pull-request: add GitHub remote support

 scripts/create-pull-request |  222 ++++++++++++++++++++++---------
 scripts/send-pull-request   |  307 ++++++++++++++++++++-----------------------
 2 files changed, 302 insertions(+), 227 deletions(-)
Otavio Salvador - May 17, 2011, 2:02 p.m.
On Mon, May 16, 2011 at 19:25, Darren Hart <dvhart@linux.intel.com> wrote:
...
> Darren Hart (13):
>  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
>  create-pull-request: add untested oe repository support
>  send-pull-request: streamline git-send-email usage
>  *pull-request: add copyright, license, and descriptions
>
> Otavio Salvador (2):
>  create-pull-request: do not check certificate
>  create-pull-request: add GitHub remote support

Acked-by: Otavio Salvador <otavio@ossystems.com.br>
Tom Rini - May 17, 2011, 7:02 p.m.
On 05/16/2011 03:25 PM, Darren Hart wrote:

...
> Darren Hart (13):
>   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
>   create-pull-request: add untested oe repository support
>   send-pull-request: streamline git-send-email usage
>   *pull-request: add copyright, license, and descriptions
> 
> Otavio Salvador (2):
>   create-pull-request: do not check certificate
>   create-pull-request: add GitHub remote support

For the bits I'm cc'd on:
Acked-by: Tom Rini <tom_rini@mentor.com>

and thanks again for picking this up and running with it!
Saul Wold - May 20, 2011, 12:46 a.m.
On 05/16/2011 03:25 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.
>
> V2: do REMOTE_REPO regex per remote
>      fix various regex issues
>      fix wget ssl issues
>      add github and oe (untested) REMOTE support
>      simplify git-send-email usage logic, avoid duplicating git confirmation
>      add -c, auto-cc per patch option
>
> 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 c12126ebf07604f998cb35546aa51d806d97af22:
>
>    create-pull-request: switch URL from git.pokylinux.org/poky-contrib to git.openembedded.org/openembedded-core-contrib (2011-05-13 20:38:20 -0700)
>
> are available in the git repository at:
>    git://git.pokylinux.org/poky-contrib dvhart/git-pull-v2-oe
>    http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=dvhart/git-pull-v2-oe
>
> Darren Hart (13):
>    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
>    create-pull-request: add untested oe repository support
>    send-pull-request: streamline git-send-email usage
>    *pull-request: add copyright, license, and descriptions
>
> Otavio Salvador (2):
>    create-pull-request: do not check certificate
>    create-pull-request: add GitHub remote support
>
>   scripts/create-pull-request |  222 ++++++++++++++++++++++---------
>   scripts/send-pull-request   |  307 ++++++++++++++++++++-----------------------
>   2 files changed, 302 insertions(+), 227 deletions(-)
>

Pulled into oe-core

Thanks

	Sau!