Re: [PATCH 00/12] tools: move more libraries into tools/libs

[Jürgen Groß <jgross@xxxxxxxx>]

On 15.07.20 15:01, Jan Beulich wrote:
> On 15.07.2020 14:51, Juergen Gross wrote:
> > Move some more libraries under tools/libs, including libxenctrl. This
> > is resulting in a lot of cleanup work regarding building libs and
> > restructuring of the tools directory.
> >
> > I have (for now) left out some more libraries like libxenguest and
> > libxl, but I can have a try moving those, too, if wanted.
> >
> > Please note that patch 8 ("tools: move libxenctrl below tools/libs")
> > needs the related mini-os and qemu-trad patches applied in order not
> > to break the build:
> >
> > https://lists.xen.org/archives/html/xen-devel/2020-07/msg00548.html
> > https://lists.xen.org/archives/html/xen-devel/2020-07/msg00617.html
> >
> > As discussed at the Xen developers summit this series has been
> > selected to act as a test case for sending pull requests via gitlab.
> > This is the reason the patches are _not_ sent individually to
> > xen-devel, but only the cover letter.
> I don't think I've seen any summary of that discussion, and hence I
> also can't judge or know whether this is meant just for huge and
> presumably very mechanical series, or as a general form of patch
> submission. The immediate point that strikes me is - how would I
> comment on such series without having to go to gitlab? (There are
> likely other issues I'd want to see addressed before this becomes
> a common process.)
Those are basically the reasons we decided to have a try.

There is certainly a how-to needed for a submitter and some scripting
to at least ping the maintainers. I have volunteered to send a series
via gitlab as I had already this one in the works and the main
maintainers of the modified files agreed to try the gitlab workflow.

We need to learn whether this is something to consider or not, and for
this purpose we need a practical example.

TBH I had some reservations as I like to be able to review patches while
being offline, so I want to make sure this is still possible somehow.
And being part of the experiment ensures I'm not missing anything. :-)


Juergen