public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Marek Vasut <marek.vasut@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] RFC - PatchTrack Specification (revised)
Date: Sun, 29 Jul 2012 05:15:08 +0200	[thread overview]
Message-ID: <201207290515.08289.marek.vasut@gmail.com> (raw)
In-Reply-To: <CALButCJXJE+mo_W5=kYT8btO4e36-i5ajmcGrE-cG0fyP1RgFw@mail.gmail.com>

Dear Graeme Russ,

> A revised version of the spec (sorry, I would have used reply-to but
> something went amiss with gmail and I've lost the original)

How does it surprise me ... superawesome google imap just crashed on me like a 
week ago, I finished syncing my emails today ...

[...]

> Operation Sequence - New single stand-alone patch:
>  - Determine which git repository the patch is intended to be applied to
>  - Add the patch to the top of the 'patch stack' of the target repository
>    NOTE: Patches are placed in the patch stack in the order they are
>    received
>  - Run the configurable set of sanity checks on the raw patch. A typical
>    example is the checkpatch.pl script which checks the patch for
>    correct style
>  - Perform a dummy git-apply of the patch onto the HEAD of the target
>    repository with all patches already on the repositories 'patch stack'
>    applied
>  - Record the results of the sanity checks and git-apply against the patch

You might want to actually create an mbox of all these stacked patches so people 
can download and apply them and rebase their patches on top of them.

[...]

> Operation Sequence - Revised version of existing patch
>  - Remove existing patch from patch stack
>  - Insert new patch into patch stack at same location as the removed patch
>  - Run the configurable set of sanity checks on the raw patch. A typical
>    example is the checkpatch.pl script which checks the patch for
>    correct style
>  - Perform a dummy git-apply of the patch onto the HEAD of the target
>    repository with all patches already on the repositories 'patch stack'
>    applied

Yes? Maybe this should be applied at the place where the old patch was, rather 
than on top.

[...]

> Web Interface
> In addition to processing inbound emails, PatchTrack includes web interface
> which allows a user to:
>  - Visualise the state of the patch stack for a given repository
>     * Passed sanity checks
>     * Passed git-apply
>     * Acked
>     * Tested
>     * Committed
>    NOTE: The patch stack is displayed as a series of rows in a table - One
>    row per patch. The colour of the row can be configured for each patch
>    state. For example, 'Acked' might be blue and 'Committed' white

Why not make committed like ... lavender or something </jab> ? :)

[...]
The rest sounds quite cool.

Best regards,
Marek Vasut

  parent reply	other threads:[~2012-07-29  3:15 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-24 13:48 [U-Boot] RFC - PatchTrack Specification (revised) Graeme Russ
2012-07-24 18:41 ` Wolfgang Denk
2012-07-24 22:33   ` Graeme Russ
2012-07-25 10:36   ` Andy Pont
2012-07-25 11:00     ` Wolfgang Denk
2012-07-25 12:50       ` Graeme Russ
2012-07-29  3:15 ` Marek Vasut [this message]
2012-07-29  3:49   ` Graeme Russ
2012-09-22 14:44     ` Marek Vasut

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=201207290515.08289.marek.vasut@gmail.com \
    --to=marek.vasut@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox