qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH 12/12] configure: bump min required CLang to 7.0.0 / XCode 10


From: Daniel P . Berrangé
Subject: Re: [PATCH 12/12] configure: bump min required CLang to 7.0.0 / XCode 10.2
Date: Wed, 12 May 2021 15:43:15 +0100
User-agent: Mutt/2.0.6 (2021-03-06)

On Wed, May 12, 2021 at 04:10:03PM +0200, Philippe Mathieu-Daudé wrote:
> On 5/12/21 2:25 PM, Daniel P. Berrangé wrote:
> > On Wed, May 12, 2021 at 01:44:51PM +0200, Philippe Mathieu-Daudé wrote:
> >> On 5/11/21 3:26 PM, Daniel P. Berrangé wrote:
> >>> Several distros have been dropped since the last time we bumped the
> >>> minimum required CLang version.
> >>>
> >>> Per repology, currently shipping versions are:
> >>>
> >>>              RHEL-8: 10.0.1
> >>>      Debian Stretch: 7.0.1
> >>>       Debian Buster: 7.0.1
> >>>  openSUSE Leap 15.2: 9.0.1
> >>>    Ubuntu LTS 18.04: 10.0.0
> >>>    Ubuntu LTS 20.04: 11.0.0
> >>>          FreeBSD 12: 8.0.1
> >>>           Fedora 33: 11.0.0
> >>>           Fedora 34: 11.1.0
> >>>
> >>> With this list Debian Stretch is the constraint at 7.0.1
> >>>
> >>> An LLVM version of 7.0.1 corresponds to macOS XCode version of 10.2
> >>> which dates from March 2019.
> >>
> >> But we still rely on Travis-CI (Ubuntu Bionic 18.04 LTS)
> >> for non-x86 targets until we have figured out who is willing
> >> to share/maintain such non-x86 native runners on Gitlab.
> >>
> >> There:
> >>
> >> $ clang --version
> >> clang version 6.0.0-1ubuntu2 (tags/RELEASE_600/final)
> >>
> >> If we don't use Travis-CI, then your patch is fine, but we
> >> need a previous patch removing .travis.yml.
> > 
> > Bioic has clang 10 available so not sure why you're seeing
> > version 6 there
> > 
> >   https://packages.ubuntu.com/bionic/clang-10
> 
> But only i386/amd64 are listed there.
> 
> However s390x is indeed in the bionic-updates stream:
> 
> https://packages.ubuntu.com/bionic-updates/clang-10

Never mind, lets just stick as 6.0.0 for min clang. There's no reason
why we /must/ pick 7.0.0 - it is just what i thought was the min across
our distros. I didn't realize ubuntu had multiple streams for clang.


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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]