Re: [jenkins-ci PATCH 8/8] guests: Move native packages out of MinGW packages

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

 



On Wed, May 16, 2018 at 11:20:26AM +0200, Andrea Bolognani wrote:
> On Wed, 2018-05-16 at 10:53 +0200, Pavel Hrdina wrote:
> > On Tue, May 15, 2018 at 04:40:09PM +0200, Andrea Bolognani wrote:
> > > Even though the packages are technically only needed when
> > > performing a MinGW build, it doesn't hurt to have them
> > > installed when performing native build, and keeping the
> > > native packages all in one place is cleaner.
> > > 
> > > Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> > > ---
> > >  guests/vars/projects/libosinfo+mingw.yml   | 1 -
> > >  guests/vars/projects/libosinfo.yml         | 1 +
> > >  guests/vars/projects/virt-viewer+mingw.yml | 1 -
> > >  guests/vars/projects/virt-viewer.yml       | 1 +
> > >  4 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > And I'm not sure about this one as well.  Yes, it might make things
> > cleaner that the mingw dependencies are only mingw-* packages but it's
> > not correct.  They are required only by the mingw build and not by the
> > regular build so it will pollute systems where we don't run mingw
> > builds.
> 
> I agree with your argument, to the point where I had basically
> already acknowledged it in the commit message :)
> 
> Given how tiny the packages involved are, though, I think the
> trade-off is worth it, especially in light of the stuff I
> mentioned in the comments to the previous patch.

For me this is different issue than the previous patch.  This actually
moves the dependency into project that doesn't require it out of project
that requires it.  It's not like both will use it so it's safe and good
enough to have it in the parent project.

The only benefit that I can see is to have only mingw-* packages as
dependency of mingw builds, which might look nice and neat in the
configuration file but IMHO it's wrong.  If we move them, it would be
nice to mention, that these packages are required by mingw build only
and that would feel weird, like why they are not only in the mingw
projects?

I would rather have it technically correct than having the configuration
files look nice :).

Pavel

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