All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Andreas Bießmann" <andreas.devel@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [STATUS] "Quality" of patches / testing.
Date: Tue, 18 Oct 2011 15:36:21 +0200	[thread overview]
Message-ID: <4E9D80D5.7090304@gmail.com> (raw)
In-Reply-To: <20111018130507.GR7102@titan.lakedaemon.net>

Dear Jason,

Am 18.10.2011 15:05, schrieb Jason:
> Wolfgang,
> 
> On Tue, Oct 18, 2011 at 11:44:37AM +0200, Wolfgang Denk wrote:
>> Dear Andreas,
>>
>> In message <4E9D4552.5040506@gmail.com> you wrote:
>>> I guess it will be doable to have some scripts/prepare-patch which runs
>>>  a) git format-patch
>>>  b) checkpatch on the patch
>>>  c) (configurable subset of) MAKEALL on some clean tree with that patch
>>>     applied
>>>  d) append the results to the patch
>>>
>>> This tool would implement some kind of CI but utilize the computing
>>> power of submitter.
>>
>> Sounds good.  Any takers?
> 
> I've been mulling this over, and here's my approach:
> 
> 1.) modify git to add a hook in format-patch, output is dumped after the
> 	 '---' and before the diff.  We use this the run checkpatch.pl with our
> 	 config.  Our script includes a version tag of checkpatch.pl?

I think of some script that utilizes git rather than changing git.

> 2.) Add a '--versioning' option to format-patch which will scan the
> 	 output directory for previous versions of the patch.

nice, but ...

> 	a.) Use the Message-Id of the first version as an In-Reply-To
> 	b.) Migrate patch changelog over from previous version, append '***
> 		 ADD CHANGELOG ENTRY HERE ***'
> 	c.) enforce [PATCH 1/7 V#] Subject line format.

I never hold old patches long, they are on the list, patchwork or in my
tree. But feel free to discuss this on git ML. And I doubt this will
help us in near future (remember the debian users ;)

Best regards

Andreas Bie?mann

  parent reply	other threads:[~2011-10-18 13:36 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-18  6:23 [U-Boot] [STATUS] "Quality" of patches / testing Wolfgang Denk
2011-10-18  6:51 ` Simon Schwarz
2011-10-18  9:22   ` Andreas Bießmann
2011-10-18  9:44     ` Wolfgang Denk
2011-10-18 13:05       ` Jason
2011-10-18 13:10         ` Jason
2011-10-18 13:13         ` Simon Schwarz
2011-10-18 13:49           ` Jason
2011-10-18 15:37             ` Jason
2011-10-18 16:12           ` Mike Frysinger
2011-10-18 13:36         ` Andreas Bießmann [this message]
2011-10-18 15:55           ` Jason
2011-10-18 14:05       ` Simon Glass
2011-10-18 16:59         ` Anton Staaf
2011-10-18 20:23         ` Wolfgang Denk
2011-10-20  0:39           ` Simon Glass
2011-10-20 15:32             ` Wolfgang Denk
2011-10-18 10:24     ` Lukasz Majewski
2011-10-18 11:02       ` Wolfgang Denk
2011-10-18  9:34   ` Wolfgang Denk
2011-10-18 13:05     ` Simon Schwarz
2011-10-18  8:49 ` Lukasz Majewski
2011-10-18 17:01 ` [U-Boot] MAKEALL Mike Frysinger
2011-10-18 17:39   ` Simon Glass
2011-10-18 17:58   ` Tom Rini
2011-10-18 18:11     ` Mike Frysinger
2011-10-18 18:31     ` Mike Frysinger
2011-10-18 18:54       ` Tom Rini
2011-10-18 19:49         ` Mike Frysinger
2011-10-18 20:07   ` Wolfgang Denk
2011-10-18 20:14     ` Mike Frysinger
2011-10-18 20:47       ` Wolfgang Denk
2011-10-18 20:55         ` Mike Frysinger
2011-10-18 21:30           ` Simon Glass
2011-10-18 22:21             ` Mike Frysinger
2011-10-19 11:36               ` Albert ARIBAUD
2011-10-19 14:25                 ` Mike Frysinger
2011-10-19 19:57                   ` Wolfgang Denk
2011-10-18 21:50           ` Wolfgang Denk
2011-10-18 22:18             ` Mike Frysinger
2011-10-18 22:33     ` Graeme Russ
2011-10-19  7:12       ` Andreas Bießmann
2011-10-19  8:57       ` Wolfgang Denk
2011-10-18 17:16 ` [U-Boot] [STATUS] "Quality" of patches / testing Anton Staaf
2011-10-18 17:44   ` Albert ARIBAUD
2011-10-18 18:07     ` Anton Staaf
2011-10-20  9:25 ` Detlev Zundel

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=4E9D80D5.7090304@gmail.com \
    --to=andreas.devel@googlemail.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.