[Buildroot] [PATCH] toolchain-external: add fortran support option for custom external toolchains

[Romain Naour]

Hi Samuel,

Le 26/07/2018 à 20:46, Samuel Martin a écrit :
> Hi Romain,
> 
> Sorry for the late response. :s

Ctrl-c Ctrl-v :s

> 
> On Sat, Jul 14, 2018 at 7:15 PM, Romain Naour <romain.naour at gmail.com> wrote:
>> If a custom external toolchain is used, we can't enable the fortran
>> support. Add a new option for that.
> 
> Indeed, it is a long-time limitation of using buildroot toolchain as
> external ones, all options are not necessarily available in the
> external toolchain backend. :-/
> 
>>
>> Signed-off-by: Romain Naour <romain.naour at gmail.com>
>> Cc: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
>> Cc: Samuel Martin <s.martin49 at gmail.com>
>> ---
>>  .../toolchain-external-custom/Config.in.options                   | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>>
>> diff --git a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options
>> index 6a9921c149..5ec3f961a1 100644
>> --- a/toolchain/toolchain-external/toolchain-external-custom/Config.in.options
>> +++ b/toolchain/toolchain-external/toolchain-external-custom/Config.in.options
>> @@ -401,6 +401,14 @@ config BR2_TOOLCHAIN_EXTERNAL_CXX
>>           support. If you don't know, leave the default value,
>>           Buildroot will tell you if it's correct or not.
>>
>> +config BR2_TOOLCHAIN_EXTERNAL_FORTRAN
>> +       bool "Toolchain has Fortran support?"
>> +       select BR2_TOOLCHAIN_HAS_FORTRAN
>> +       help
>> +         Select this option if your external toolchain has Fortran
>> +         support. If you don't know, leave the default value,
>> +         Buildroot will tell you if it's correct or not.
>> +
> 
> In the gcc package, there is some extra checks for fortran with
> libquadmath when available (which depends on the target architecture).
> So, I wonder whether it should be sensitive to libquadmath (so wchar)
> on the relevant architectures.

Indeed, I added:

config BR2_TOOLCHAIN_EXTERNAL_FORTRAN
	bool "Toolchain has Fortran support?"
	# on architecture building libquadmath, wchar is required
	depends on !BR2_TOOLCHAIN_HAS_LIBQUADMATH || \
		(BR2_TOOLCHAIN_HAS_LIBQUADMATH && BR2_USE_WCHAR)
	select BR2_TOOLCHAIN_HAS_FORTRAN
	help
	  Select this option if your external toolchain has Fortran
	  support. If you don't know, leave the default value,
	  Buildroot will tell you if it's correct or not.

comment "Fortran support needs a toolchain w/ wchar"
	depends on BR2_TOOLCHAIN_HAS_LIBQUADMATH
	depends on !BR2_USE_WCHAR # libquadmath

I'll send a v2.

Best regards,
Romain


> 
>>  config BR2_TOOLCHAIN_EXTRA_EXTERNAL_LIBS
>>         string "Extra toolchain libraries to be copied to target"
>>         help
>> --
>> 2.14.4
>>
> 
> Regards,
>