[PATCH libdrm] configure.ac: pthread-stubs not present on OpenBSD
Eric Engestrom
eric.engestrom at imgtec.com
Tue Feb 20 10:21:38 UTC 2018
On Tuesday, 2018-02-20 17:09:14 +1100, Jonathan Gray wrote:
> pthread-stubs is no longer required on OpenBSD and has been removed.
> libpthread parts involved moved to libc.
Great news!
Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
Note that meson needs the same change:
----8<----
diff --git a/meson.build b/meson.build
index d600a906948c0e680885..bd00cdc2cae9f0749180 100644
--- a/meson.build
+++ b/meson.build
@@ -35,8 +35,7 @@ with_install_tests = get_option('install-test-programs')
config = configuration_data()
-# TODO: openbsd is guess, the others are correct
-if ['freebsd', 'dragonfly', 'netbsd', 'openbsd'].contains(host_machine.system())
+if ['freebsd', 'dragonfly', 'netbsd'].contains(host_machine.system())
dep_pthread_stubs = dependency('pthread-stubs', version : '>= 0.4')
else
dep_pthread_stubs = []
---->8----
Do you have commit access, or do you want me to push this for you?
No need to send a v2, I'll add the meson bit when I push it, or you can
add it if you push it yourself.
>
> Signed-off-by: Jonathan Gray <jsg at jsg.id.au>
> ---
> configure.ac | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 35378b33..d024fcd8 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -65,10 +65,10 @@ AC_CHECK_HEADERS([sys/sysctl.h sys/select.h])
> LT_PREREQ([2.2])
> LT_INIT([disable-static])
>
> -dnl pthread-stubs is mandatory on BSD platforms, due to the nature of the
> +dnl pthread-stubs is mandatory on some BSD platforms, due to the nature of the
> dnl project. Even then there's a notable issue as described in the project README
> case "$host_os" in
> -linux* | cygwin* | darwin* | solaris* | *-gnu* | gnu*)
> +linux* | cygwin* | darwin* | solaris* | *-gnu* | gnu* | openbsd*)
> pthread_stubs_possible="no"
> ;;
> * )
> --
> 2.16.0
>
More information about the dri-devel
mailing list