diff mbox series

[FFmpeg-devel] configure: warn when threading is disabled

Message ID 20231003133720.8458-1-anton@khirnov.net
State New
Headers show
Series [FFmpeg-devel] configure: warn when threading is disabled | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished

Commit Message

Anton Khirnov Oct. 3, 2023, 1:37 p.m. UTC
Explicitly state what the implications of this are.
---
 configure | 6 ++++++
 1 file changed, 6 insertions(+)

Comments

James Almer Oct. 3, 2023, 1:40 p.m. UTC | #1
On 10/3/2023 10:37 AM, Anton Khirnov wrote:
> Explicitly state what the implications of this are.
> ---
>   configure | 6 ++++++
>   1 file changed, 6 insertions(+)
> 
> diff --git a/configure b/configure
> index 8a1a1b8584..5ba2698798 100755
> --- a/configure
> +++ b/configure
> @@ -7493,6 +7493,12 @@ check_deps $CONFIG_LIST       \
>   
>   enabled threads && ! enabled pthreads && ! enabled atomics_native && die "non pthread threading without atomics not supported, try adding --enable-pthreads or --cpu=i486 or higher if you are on x86"
>   
> +enabled threads || warn \
> +    "Threading support was disabled or is not available on this platform."   \

nit: The target platform

> +    "This means that not only is this build not multi-threaded, but also"    \
> +    "that the libraries from this build MUST NOT be used in a multi-threaded"\
> +    "environment."
> +
>   case $target_os in
>   haiku)
>       disable memalign
Anton Khirnov Oct. 3, 2023, 2:04 p.m. UTC | #2
Quoting James Almer (2023-10-03 15:40:12)
> On 10/3/2023 10:37 AM, Anton Khirnov wrote:
> > Explicitly state what the implications of this are.
> > ---
> >   configure | 6 ++++++
> >   1 file changed, 6 insertions(+)
> > 
> > diff --git a/configure b/configure
> > index 8a1a1b8584..5ba2698798 100755
> > --- a/configure
> > +++ b/configure
> > @@ -7493,6 +7493,12 @@ check_deps $CONFIG_LIST       \
> >   
> >   enabled threads && ! enabled pthreads && ! enabled atomics_native && die "non pthread threading without atomics not supported, try adding --enable-pthreads or --cpu=i486 or higher if you are on x86"
> >   
> > +enabled threads || warn \
> > +    "Threading support was disabled or is not available on this platform."   \
> 
> nit: The target platform

very right, fixed locally
diff mbox series

Patch

diff --git a/configure b/configure
index 8a1a1b8584..5ba2698798 100755
--- a/configure
+++ b/configure
@@ -7493,6 +7493,12 @@  check_deps $CONFIG_LIST       \
 
 enabled threads && ! enabled pthreads && ! enabled atomics_native && die "non pthread threading without atomics not supported, try adding --enable-pthreads or --cpu=i486 or higher if you are on x86"
 
+enabled threads || warn \
+    "Threading support was disabled or is not available on this platform."   \
+    "This means that not only is this build not multi-threaded, but also"    \
+    "that the libraries from this build MUST NOT be used in a multi-threaded"\
+    "environment."
+
 case $target_os in
 haiku)
     disable memalign