[Buildroot] [PATCH v2 1/1] perl: Add option to enable threads

Arnout Vandecappelle arnout at mind.be
Tue Jan 21 22:23:36 UTC 2020



On 21/01/2020 20:29, Clayton Shotwell wrote:
> From: Hector Kesari <hector.kesari at rockwellcollins.com>
> 
> Add config option for Perl to enable threads usage.
> 
> Signed-off-by: Hector Kesari <hector.kesari at rockwellcollins.com>
> Signed-off-by: Clayton Shotwell <clayton.shotwell at rockwellcollins.com>
> 
> ---
> Changes v1 -> v2:
>   - Changed references of PERL to Perl
>   - Added a warning about using Perl threads
> ---
>  package/perl/Config.in | 13 +++++++++++++
>  package/perl/perl.mk   |  4 ++++
>  2 files changed, 17 insertions(+)
> 
> diff --git a/package/perl/Config.in b/package/perl/Config.in
> index 32a3fd2ee9..34b87ee575 100644
> --- a/package/perl/Config.in
> +++ b/package/perl/Config.in
> @@ -24,4 +24,17 @@ config BR2_PACKAGE_PERL_MODULES
>  	  Leave empty for all modules (as far as the external libraries
>  	  are available).
>  
> +config BR2_PACKAGE_PERL_ENABLE_THREADS
> +	bool "Support for Threads in Perl"
> +	depends on BR2_TOOLCHAIN_HAS_THREADS
> +	help
> +	  Enable use of threads in Perl scripts
> +
> +	  WARNING: The use of interpreter-based threads in Perl
> +	  is officially discouraged.
> +	  See https://perldoc.perl.org/threads.html#WARNING

 So why do you want it then?

 Regards,
 Arnout

> +
> +comment "Perl threads need toolchain w/ threads"
> +	depends on !BR2_TOOLCHAIN_HAS_THREADS
> +
>  endif
> diff --git a/package/perl/perl.mk b/package/perl/perl.mk
> index 0ab81ddb34..e0bbf9cfd4 100644
> --- a/package/perl/perl.mk
> +++ b/package/perl/perl.mk
> @@ -62,6 +62,10 @@ ifeq ($(shell expr $(PERL_VERSION_MAJOR) % 2), 1)
>  PERL_CONF_OPTS += -Dusedevel
>  endif
>  
> +ifeq ($(BR2_PACKAGE_PERL_ENABLE_THREADS),y)
> +PERL_CONF_OPTS += -Dusethreads
> +endif
> +
>  ifeq ($(BR2_STATIC_LIBS),y)
>  PERL_CONF_OPTS += --all-static --no-dynaloader
>  endif
> 


More information about the buildroot mailing list