[Mesa-dev] [PATCH 1/2] configure.ac: require pthread-stubs only where available

[Jeremy Sequoia]

Reviewed-by: Jeremy Huddleston Sequoia <jeremyhu at apple.com>

Seems like the right change to me.  I was always a bit perplexed as to why that was there in the first place, so I'm glad to see it get the boot where not needed.

Sent from my iPhone...

> On Mar 2, 2017, at 11:02, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> 
> From: Emil Velikov <emil.velikov at collabora.com>
> 
> The project is a thing only for BSD platforms. Or in other words - for
> any other platforms building/installing pthread-stubs results only in a
> pthread-stub.pc file.
> 
> And even where it provides a DSO, there's a fundamental design issue
> with it - see the pthread-stubs mailing list for the specifics.
> 
> Cc: Jeremy Huddleston Sequoia <jeremyhu at apple.com>
> CC: Gary Wong <gtw at gnu.org>
> Cc: Randy Fishel <randy.fishel at oracle.com>
> Cc: Niveditha Rau <niveditha.rau at oracle.com>
> Signed-off-by: Emil Velikov <emil.velikov at collabora.com>
> ---
> Jeremy, others,
> 
> Afaict pthread-stubs expands to a simple .pc on your platforms, but a
> confirmation will be greatly appreciated.
> ---
> configure.ac | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure.ac b/configure.ac
> index a3d1a00bdd..e94e46a0b8 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -799,7 +799,7 @@ fi
> 
> dnl pthread-stubs is mandatory on targets where it exists
> case "$host_os" in
> -cygwin* )
> +linux* | cygwin* | darwin* | solaris* | gnu*)
>     pthread_stubs_possible="no"
>     ;;
> * )
> -- 
> 2.11.1
>