[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tests: vm: auto_install OpenBSD
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH] tests: vm: auto_install OpenBSD |
Date: |
Fri, 24 Aug 2018 15:43:41 +0100 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Fri, Aug 24, 2018 at 10:36:30AM -0400, Brad Smith wrote:
> I very much appreciate the effort to bump up to 6.3 as I was going
> to suggest doing that at some point. But bumping up to 6.3 at the
> moment will fail with the configure script. We've switched from GCC 4.2
> to Clang. The TLS check will fail with Clang's emulated TLS. We've
> had a local patch for awhile to fix the test but I don't think it is
> appropriate to upstream as is.
>
> Index: configure
> --- configure.orig
> +++ configure
> @@ -1876,7 +1876,7 @@ static __thread int tls_var;
> int main(void) { return tls_var; }
> EOF
> -if ! compile_prog "-Werror" "" ; then
> +if ! compile_prog "-Werror" "-pthread" ; then
> error_exit "Your compiler does not support the __thread specifier for " \
> "Thread-Local Storage (TLS). Please upgrade to a version that does."
> fi
Later on in the configure script there's a check for pthreads that
sets $PTHREAD_LIB to the desired arg.
Best is probably to move that check higher up, and then use $PTHREAD_LIB
as the argument to the compile_prog call you show.
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 :|
Re: [Qemu-devel] [PATCH] tests: vm: auto_install OpenBSD, Gerd Hoffmann, 2018/08/27
Re: [Qemu-devel] [PATCH] tests: vm: auto_install OpenBSD, Fam Zheng, 2018/08/27