Re: [libvirt-jenkins-ci PATCH 1/7] lcitool: Improve ccache symlinks creation

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

 



On Fri, Mar 27, 2020 at 08:34:53PM +0100, Andrea Bolognani wrote:
> Instead of having the same code snippet twice, once for Linux and
> once for FreeBSD, the only actual difference being the name of the
> compiler, include the compiler's path in the inventory among other
> paths and then figure out the correct name for the symlink based
> on that information.
> 
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  guests/host_vars/libvirt-centos-7/main.yml    |  1 +
>  guests/host_vars/libvirt-centos-8/main.yml    |  1 +
>  guests/host_vars/libvirt-debian-10/main.yml   |  1 +
>  guests/host_vars/libvirt-debian-9/main.yml    |  1 +
>  guests/host_vars/libvirt-debian-sid/main.yml  |  1 +
>  guests/host_vars/libvirt-fedora-30/main.yml   |  1 +
>  guests/host_vars/libvirt-fedora-31/main.yml   |  1 +
>  .../host_vars/libvirt-fedora-rawhide/main.yml |  1 +
>  guests/host_vars/libvirt-freebsd-11/main.yml  |  1 +
>  guests/host_vars/libvirt-freebsd-12/main.yml  |  1 +
>  .../libvirt-freebsd-current/main.yml          |  1 +
>  .../host_vars/libvirt-opensuse-151/main.yml   |  1 +
>  guests/host_vars/libvirt-ubuntu-1604/main.yml |  1 +
>  guests/host_vars/libvirt-ubuntu-1804/main.yml |  1 +
>  guests/playbooks/update/tasks/users.yml       | 19 +------------------
>  15 files changed, 15 insertions(+), 18 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>


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 :|





[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