From: Lars Kurth <lars.kurth@citrix.com>
To: Ian Jackson <Ian.Jackson@citrix.com>
Cc: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
"committers@xenproject.org" <committers@xenproject.org>,
George Dunlap <George.Dunlap@citrix.com>
Subject: Re: [PATCH v2 3/3] Significant changes to decision making; some new roles and minor changes
Date: Mon, 19 Sep 2016 14:46:55 +0000 [thread overview]
Message-ID: <D405B5D8.2DFAE%lars.kurth@citrix.com> (raw)
In-Reply-To: <22495.61359.153317.720966@mariner.uk.xensource.com>
On 19/09/2016 15:01, "Ian Jackson" <ian.jackson@eu.citrix.com> wrote:
>Lars Kurth writes ("Re: [PATCH v2 3/3] Significant changes to decision
>making; some new roles and minor changes"):
>> I could, but it isn't really that long. And it makes life hard for me,
>>as
>> there is only one file. Thus breaking it into bits and maintaining the
>> changes is hard in git.
>
>Are you aware of `git add -i' ?
No: will look at it next time.
>If you find it too difficult I would be happy to try to rework your
>final patch into a series of smaller changes.
I talked to George and he is happy to review this as-is.
The only sensible way to break it into smaller bits is by chapter: there
are very few dependencies across chapters (with the exception of
cross-references such as anchors and links).
Almost everyone else has reviewed this series already: Jan, Stefano, Wei,
Tim and I think Konrad.
That leaves you and George amongst the committers: so I am not sure
whether there is much point in doing so. But I am not against it.
Lars
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-09-19 14:47 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-12 16:20 [PATCH v2 0/3] Significant changes to Xen Project Governance (governance.html) Lars Kurth
2016-09-12 16:20 ` [PATCH v2 1/3] Code motion changes to make real patches easier to read Lars Kurth
2016-09-12 16:20 ` [PATCH v2 2/3] Added comment sections to highight problem areas Lars Kurth
2016-09-12 16:20 ` [PATCH v2 3/3] Significant changes to decision making; some new roles and minor changes Lars Kurth
2016-09-13 16:46 ` George Dunlap
2016-09-13 16:57 ` Lars Kurth
2016-09-19 14:01 ` Ian Jackson
2016-09-19 14:46 ` Lars Kurth [this message]
2016-09-19 15:36 ` Ian Jackson
2016-09-20 17:01 ` Ian Jackson
2016-09-20 18:53 ` Lars Kurth
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=D405B5D8.2DFAE%lars.kurth@citrix.com \
--to=lars.kurth@citrix.com \
--cc=George.Dunlap@citrix.com \
--cc=Ian.Jackson@citrix.com \
--cc=committers@xenproject.org \
--cc=xen-devel@lists.xenproject.org \
/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;
as well as URLs for NNTP newsgroup(s).