Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Darren Hart <dvhart@linux.intel.com>
To: Darren Hart <dvhart@linux.intel.com>
Cc: Martin Jansa <martin.jansa@gmail.com>,
	Koen Kooi <koen@dominion.thruhere.net>,
	openembedded-core@lists.openembedded.org,
	Paul Eggleton <paul.eggleton@intel.com>
Subject: Re: [RFC PATCH 00/10] *-pull-request: cleanup and overhaul
Date: Fri, 13 May 2011 20:47:00 -0700	[thread overview]
Message-ID: <4DCDFB34.3080306@linux.intel.com> (raw)
In-Reply-To: <cover.1305329659.git.dvhart@linux.intel.com>

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(-)
> 

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel



      parent reply	other threads:[~2011-05-14  3:49 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-13 23:35 [RFC PATCH 00/10] *-pull-request: cleanup and overhaul Darren Hart
2011-05-13 23:36 ` [RFC PATCH 01/10] create-pull-request: alphabetize arguments Darren Hart
2011-05-13 23:36 ` [RFC PATCH 02/10] create-pull-request: whitespace cleanup Darren Hart
2011-05-13 23:37 ` [RFC PATCH 03/10] create-pull-request: use git request-pull and arbitrary remotes Darren Hart
2011-05-14  0:14   ` Joshua Lock
2011-05-14  2:04     ` Darren Hart
2011-05-14  3:19     ` Darren Hart
2011-05-16 16:31       ` Joshua Lock
2011-05-13 23:37 ` [RFC PATCH 04/10] create-pull-request: rewrite known private URLs to public URLs Darren Hart
2011-05-13 23:37 ` [RFC PATCH 05/10] create-pull-request: provide an RFC mode via -c argument Darren Hart
2011-05-13 23:37 ` [RFC PATCH 06/10] send-pull-request: whitespace cleanup Darren Hart
2011-05-13 23:38 ` [RFC PATCH 07/10] send-pull-request: remove local mta support Darren Hart
2011-05-13 23:38 ` [RFC PATCH 08/10] send-pull-request: fix greedy auto-cc regex Darren Hart
2011-05-14  5:41   ` Bruce Ashfield
2011-05-13 23:38 ` [RFC PATCH 09/10] send-pull-request: don't send all patches to everyone even with -a Darren Hart
2011-05-13 23:44   ` Khem Raj
2011-05-14  0:14     ` Joshua Lock
2011-05-14  0:31     ` Darren Hart
2011-05-13 23:38 ` [RFC PATCH 10/10] send-pull-request: verify git sendemail config Darren Hart
2011-05-14  5:43   ` Bruce Ashfield
2011-05-14 18:45     ` Darren Hart
2011-05-15  2:48       ` Bruce Ashfield
2011-05-16 15:13         ` Darren Hart
2011-05-16 15:16           ` Bruce Ashfield
2011-05-14  3:47 ` Darren Hart [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4DCDFB34.3080306@linux.intel.com \
    --to=dvhart@linux.intel.com \
    --cc=koen@dominion.thruhere.net \
    --cc=martin.jansa@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=paul.eggleton@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox