On Tue, Feb 20, 2018 at 10:21:38AM +0000, Eric Engestrom wrote:
> 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@xxxxxxxxxx>
>
> 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.
An almost identical patch was sent to the Mesa list. I don't have
commit access to either repository so if you could push it that would be
great.
We can't take a build dependency on python for the xserver and related
parts in OpenBSD so I have no interest in meson and very much hope
autoconf remains otherwise it is back to writing custom makefiles for
everything. The meson patch at least reads fine but I have not tried it.
>
> >
> > Signed-off-by: Jonathan Gray <jsg@xxxxxxxxx>
> > ---
> > 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
> >
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
