public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Albert ARIBAUD <albert.u.boot@aribaud.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4] Add 'patman' patch generation, checking and submission script
Date: Fri, 03 Feb 2012 21:00:35 +0100	[thread overview]
Message-ID: <4F2C3CE3.8060705@aribaud.net> (raw)
In-Reply-To: <CAPnjgZ2SCgFe38qBnKVSran0Rd0cY3fdC+rK0JdWNf+AHS3=wA@mail.gmail.com>

Hi Simon,

Le 03/02/2012 20:30, Simon Glass a ?crit :
> Hi Albert,
>
> On Fri, Feb 3, 2012 at 11:00 AM, Albert ARIBAUD
> <albert.u.boot@aribaud.net>  wrote:
>> Hi Simon,
>>
>> Le 15/01/2012 02:20, Simon Glass a ?crit :
>>
>>> Hi,
>>>
>>> Hmmm patman found a tag in this commit and tried to send it to Fred
>>> Bloggs. I have added the line below - sorry for the confusion.
>>>
>>> On Sat, Jan 14, 2012 at 5:12 PM, Simon Glass<sjg@chromium.org>    wrote:
>>>>
>>>> What is this?
>>>> =============
>>>>
>>>> This tool is a Python script which:
>>>> - Creates patch directly from your branch
>>>> - Cleans them up by removing unwanted tags
>>>> - Inserts a cover letter with change lists
>>>> - Runs the patches through checkpatch.pl and its own checks
>>>> - Optionally emails them out to selected people
>>
>> It seems to me that this is not specific to u-boot, and as such, has no
>> reason to be managed along with U-Boot.
>>
>> Why should it not simply have its own project and resources?
>
> It was created in response to Wolfgang's comments that the sometimes
> low quality of patches on the U-Boot ML chews up a lot of time.
>
> It is not technically specific to U-Boot (since it could also be used
> with the Linux kernel), but that was my intent when creating it. If
> this goes in then I plan to look at how to automate build testing also
> so we can more easily see that patches are MAKEALL-clean.
>
> The discussion at the time was here:
>
> http://lists.denx.de/pipermail/u-boot/2011-October/105862.html
>
> and the many messages which followed. My patch was in response to this.

This aptly explains why the tool was created and what it intends to 
attain, and that its use in U-Boot was discussed -- things that I did 
not question in the first place, and still do not.

What I question is the relevance of tying together the development, 
source code management, release management etc of U-Boot and patman. Is 
there any reason that a release of U-Boot should also be a release of 
patman, or the other way around? Is there any reason even to have 
similar development models for both projects?

As an example, the creation of git was strongly motivated by, and tied 
to, source code management requirements of Linux, but git is not 
integrated in the Linux source tree and its development is fully 
independent from that of Linux.

My point is that patman, not being tied to a given project and being 
certainly just as useful to may other git-based projects, it should have 
its own project, be able to evolve at its own pace, etc.

> Regards,
> Simon

Amicalement,
-- 
Albert.

  reply	other threads:[~2012-02-03 20:00 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-15  1:12 [U-Boot] [PATCH v4] Add 'patman' patch generation, checking and submission script Simon Glass
2012-01-15  1:20 ` Simon Glass
2012-02-03 19:00   ` Albert ARIBAUD
2012-02-03 19:30     ` Simon Glass
2012-02-03 20:00       ` Albert ARIBAUD [this message]
2012-02-05  5:46         ` Simon Glass
2012-02-12 13:55           ` Albert ARIBAUD
2012-04-04 10:35 ` Gerlando Falauto
2012-04-04 16:16   ` Simon Glass
2012-04-04 16:38     ` Gerlando Falauto
2012-04-04 16:43       ` Simon Glass
2012-04-21 15:26 ` Wolfgang Denk

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=4F2C3CE3.8060705@aribaud.net \
    --to=albert.u.boot@aribaud.net \
    --cc=u-boot@lists.denx.de \
    /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