Re: [jenkins-ci PATCH 7/8] guests: Remove duplicate native packages

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, May 16, 2018 at 11:14:26AM +0200, Andrea Bolognani wrote:
> On Wed, 2018-05-16 at 10:49 +0200, Pavel Hrdina wrote:
> > On Tue, May 15, 2018 at 04:40:08PM +0200, Andrea Bolognani wrote:
> > > The MinGW variant of a build can assume native dependencies
> > > are installed, so no need to spell them out again.
> > > 
> > > Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> > > ---
> > >  guests/vars/projects/libvirt+mingw.yml      | 2 --
> > >  guests/vars/projects/libvirt-glib+mingw.yml | 4 ----
> > >  guests/vars/projects/virt-viewer+mingw.yml  | 2 --
> > >  3 files changed, 8 deletions(-)
> > 
> > I'm not sure about this one, yes our setup runs mingw builds only on
> > systems where we run the native builds but we don't have it configured
> > as strict dependency.
> 
> There was a discussion about a very similar issue a while ago,
> sparked by
> 
>   https://www.redhat.com/archives/libvir-list/2018-April/msg01016.html
> 
> The bottom line for me is that non-native builds are something
> you might decide to perform in addition to native builds (see
> that + sign there? ;), so it's perfectly reasonable not to list
> native dependencies for the MinGW variant.
> 
> It's true that we don't have a mechanism to ensure this
> relationship is enforced, but as long as we don't start getting
> a lot of external contributions I think reviews are a good
> enough bar to clear.
> 
> I also plan to actually document this formally, along with the
> other kind of relationship we need to concern ourselves with:
> that between eg. libvirt-glib and libvirt, where the former can
> only be built after building the latter, and as such doesn't
> need to explicitly depend on any package the latter already
> depends on. There's also some more cleaning up to do there, I
> just haven't gotten around to it yet :)

Good enough for me :)

Reviewed-by: Pavel Hrdina <phrdina@xxxxxxxxxx>

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux