Re: [Xen-devel] [PATCH lp-metadata 2/3] livepatch: Handle arbitrary size names with the list operation

[Julien Grall <julien.grall@xxxxxxx>]

Hi George,

On 15/08/2019 16:48, George Dunlap wrote:
> On 8/15/19 4:36 PM, Julien Grall wrote:
> > Hi George,
> >
> > On 15/08/2019 16:32, George Dunlap wrote:
> > > On 8/15/19 4:29 PM, Julien Grall wrote:
> > > > On 15/08/2019 16:19, Wieczorkiewicz, Pawel wrote:
> > > > > Hi Lars, Julien,
> > > > Hi,
> > > >
> > > > > Thanks for the pointers, I will read them up and follow the
> > > > > recommendations with my future contributions.
> > > > > Sorry for the mess…
> > > > >
> > > > > But, let me ask first before reading the wikis, how do you prefer
> > > > > submitting series that contain patches belonging to 2 distinct repos
> > > > > (e.g. xen and livepatch-build-tools)?
> > > > I can see two ways:
> > > >
> > > >     1) One series per project and mention in the cover letter that
> > > > modifications are required in another project (with link/title).
> > > >     2) Combine all the patches in one series and tag them differently.
> > > > I.e
> > > > [XEN] [LIVEPATCH].
> > > >
> > > > 1) is preferable if you have a lot of patches in each repo. 2) can be
> > > > handy if you have only a couple of patches for one repo.
> > > 1 is also easier for automated tools (like patchew) to deal with.
> > Out of interest, in general developer will tend to cross-post those
> > patches. So in what way this would make it easier?
> If you have two separate series, then patchew will be able to handle one
> and not handle the other.  If they're mixed in a single series, patchew
> won't be able to handle it at all.  At the moment patchew doesn't do
> anything but give you a nice mbox / git branch to pull; but eventually
> the idea is that it will do some level of testing and give feedback
> (patch does/n't apply, patch does/n't build, patch does/n't pass smoke
> tests / &c).
Oh, so patchew will try to apply the series. If it does not fully apply, then it 
means the series does not target Xen, right?
I haven't used much patchew so far, but it looks like I should give a try when 
committing/reviewing series :).
Thank you for the explanation!

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel