[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gitlab-ci: Remove the second superfluous macos task
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH] gitlab-ci: Remove the second superfluous macos task |
Date: |
Mon, 19 Jul 2021 10:04:34 +0100 |
User-agent: |
Mutt/2.0.7 (2021-05-04) |
On Mon, Jul 19, 2021 at 09:30:51AM +0200, Thomas Huth wrote:
> While there might have been bigger differnces between the -base and
> the -xcode images in the beginning, they almost vanished in the
> current builds, e.g. when comparing the output of the "configure"
> step after cleaning up the differences due to temporary path names,
> I only get:
>
> $ diff -u /tmp/base.txt /tmp/xcode.txt
> --- /tmp/base.txt 2021-07-16 09:16:24.211427940 +0200
> +++ /tmp/xcode.txt 2021-07-16 09:16:43.029684274 +0200
> @@ -19,14 +19,14 @@
> Build type: native build
> Project name: qemu
> Project version: 6.0.50
> -C compiler for the host machine: cc (clang 12.0.0 "Apple clang version
> 12.0.0 (clang-1200.0.32.29)")
> +C compiler for the host machine: cc (clang 12.0.0 "Apple clang version
> 12.0.0 (clang-1200.0.32.28)")
> C linker for the host machine: cc ld64 609.8
> Host machine cpu family: x86_64
> Host machine cpu: x86_64
> Program sh found: YES (/bin/sh)
> Program python3 found: YES (/usr/local/opt/python@3.9/bin/python3.9)
> Program bzip2 found: YES (/usr/bin/bzip2)
> -C++ compiler for the host machine: c++ (clang 12.0.0 "Apple clang version
> 12.0.0 (clang-1200.0.32.29)")
> +C++ compiler for the host machine: c++ (clang 12.0.0 "Apple clang version
> 12.0.0 (clang-1200.0.32.28)")
> C++ linker for the host machine: c++ ld64 609.8
> Objective-C compiler for the host machine: clang (clang 12.0.0)
> Objective-C linker for the host machine: clang ld64 609.8
>
> Since we're not using Xcode itself at all, it seems like it does not
> make much sense anymore to waste compute cycles with two images here.
> Thus let's delete the -xcode job now.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .gitlab-ci.d/cirrus.yml | 15 ---------------
> 1 file changed, 15 deletions(-)
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
We can always add it back easily later if it becomes relevant again in
a future release/update.
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 :|