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

Jon Turney jon.turney at dronecode.org.uk
Fri Mar 10 20:03:37 UTC 2017


On 02/03/2017 19:02, Emil Velikov 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

This comment could probably do with revising, since that test has almost 
the opposite meaning now, perhaps something like "don't bother linking 
with pthread-stubs when it's a no-op"

>  case "$host_os" in
> -cygwin* )
> +linux* | cygwin* | darwin* | solaris* | gnu*)
>      pthread_stubs_possible="no"
>      ;;
>  * )
>



More information about the mesa-dev mailing list