Re: [libvirt] [PATCH 7/8] travis: Install more build dependencies
On Thu, Sep 14, 2017 at 04:09:37PM +0200, Andrea Bolognani wrote:
On Thu, 2017-09-14 at 14:50 +0100, Daniel P. Berrange wrote:
> > @@ -5,37 +5,60 @@ addons:
> > apt:
> > # Please keep this list sorted alphabetically
> > packages:
[...]
> > + - zfs-fuse
>
> Did you check if travis actually lets you install all these ? They have
> to whitelist each individual package that's desired. So if some of these
> aren't in the whitelist, adding them here will have no effect IIUC.
That's kind of insane.
They don't fully trust the security of the container env in which they
populate the packages I guess.
Anyway, it looks like all the packages in the list are allowed,
so whatever :)
Ok,
Reviewed-by: Daniel P. Berrange <berrange(a)redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|