From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] arch/mips: Set BR2_GCC_TARGET_ARCH for MIPS
Date: Thu, 31 Oct 2013 15:55:21 +0100 [thread overview]
Message-ID: <20131031155521.7c17a702@skate> (raw)
In-Reply-To: <CAAXf6LVY7YE0ZmtT7EFouvnvRpja4sqdWV+ceYU_e9-iHVQyqw@mail.gmail.com>
Dear Thomas De Schampheleire,
On Thu, 31 Oct 2013 13:45:43 +0100, Thomas De Schampheleire wrote:
> I think we need a single strategy to be used by all maintainer-proxies.
> The problem with not resending the patches is that the
> maintainer-proxy (or whatever you want to call it) needs to have a git
> tree accessible by Peter. Although not impossible, I'm not really fond
> of it.
>
> Also, consider this scenario: a developer submits a patch on day 1, a
> maintainer-proxy adds the patch to his branch on day 2. However,
> another contributer gives some remarks on the patch and expects it to
> be reworked. The maintainer-proxy could forget about this when v2 of
> that patch is sent. In this case, v2 is pending in patchwork, and v1
> is on a branch for which a pull request is sent. The actual contents
> of the branch is not visible on the list. This opens a window for
> errors. By sending the patches to the list, there is a bit more
> visibility. The downside is that the list gets additional mails.
>
> An additional question is: is there a policy on who takes which
> patches, if certain actions need to be taken on these patches, etc.?
> At this moment, I still don't really see the difference between
> providing an Acked-by: and adding a patch to a for-peter branch,
> except for the fact that incoming patches are prearranged in batches
> for Peter.
What I merely intended to do with the for-peter-2013.11 is to act
more-or-less as an interim maintainer during the short period Peter is
not available. I've done this before (i.e not having commit access to
the official repository, but gathering relevant patches in a branch,
until Peter comes back). The idea here is that there would be only one
such 'interim maintainer' at a given point in time.
And yes, the only difference between the for-peter-2013.11 branch and
Acked-by is simply that the batch of patches is already prepared for
Peter to review and pull, he doesn't have to go in the list and/or
patchwork and filter out which patches he should have a look right now,
and which patches are anyway delayed to 2014.02.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2013-10-31 14:55 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-28 14:21 [Buildroot] [PATCH] arch/mips: Set BR2_GCC_TARGET_ARCH for MIPS Markos Chandras
2013-10-30 17:45 ` Thomas Petazzoni
2013-10-31 9:19 ` Thomas De Schampheleire
2013-10-31 9:22 ` Thomas Petazzoni
2013-10-31 10:13 ` Thomas De Schampheleire
2013-10-31 10:17 ` Arnout Vandecappelle
2013-10-31 10:47 ` Thomas Petazzoni
2013-10-31 12:45 ` Thomas De Schampheleire
2013-10-31 14:55 ` Thomas Petazzoni [this message]
2013-10-31 17:10 ` Thomas De Schampheleire
2013-10-31 21:54 ` Thomas Petazzoni
2013-10-31 17:57 ` Arnout Vandecappelle
2013-10-31 10:30 ` Thomas Petazzoni
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=20131031155521.7c17a702@skate \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/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