[PATCH] build: Add PROGRAM_PREFIX option

[Chris Johns]

On 2/8/2023 6:56 pm, Sebastian Huber wrote:
> On 02.08.23 10:50, Chris Johns wrote:
>> On 2/8/2023 6:39 pm, Sebastian Huber wrote:
>>> On 02.08.23 10:33, Chris Johns wrote:
>>>>> diff --git a/spec/build/bsps/makeinc.yml b/spec/build/bsps/makeinc.yml
>>>>> index ac395f2f02..08fc75a8b9 100644
>>>>> --- a/spec/build/bsps/makeinc.yml
>>>>> +++ b/spec/build/bsps/makeinc.yml
>>>>> @@ -16,14 +16,14 @@ content: |
>>>>>      prefix = ${PREFIX}
>>>>>      exec_prefix = $${prefix}/${ARCH}-rtems${__RTEMS_MAJOR__}
>>>>>    -  CC_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-gcc
>>>>> -  CXX_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-g++
>>>>> -  AS_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-as
>>>>> -  AR_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-ar
>>>>> -  NM_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-nm
>>>>> -  LD_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-ld
>>>>> -  SIZE_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-size
>>>>> -  OBJCOPY_FOR_TARGET = ${ARCH}-rtems${__RTEMS_MAJOR__}-objcopy
>>>>> +  CC_FOR_TARGET = ${PROGRAM_PREFIX}gcc
>>>>> +  CXX_FOR_TARGET = ${PROGRAM_PREFIX}g++
>>>> Is it worth doing the same to gcc and g++ as well so these can be replaced as
>>>> well?
>>> Yes, but this would be another patch and it is a bit more work since you have to
>>> test the clang support.
>>>
>>>>> +  AS_FOR_TARGET = ${PROGRAM_PREFIX}as
>>>>> +  AR_FOR_TARGET = ${PROGRAM_PREFIX}ar
>>>>> +  NM_FOR_TARGET = ${PROGRAM_PREFIX}nm
>>>>> +  LD_FOR_TARGET = ${PROGRAM_PREFIX}ld
>>>>> +  SIZE_FOR_TARGET = ${PROGRAM_PREFIX}size
>>>>> +  OBJCOPY_FOR_TARGET = ${PROGRAM_PREFIX}objcopy
>>>> Where is PROGRAM_PFREFIX set?
>>> It is a new configuration option:
>>>
>>> [sparc/gr740]
>>> PROGRAM_PREFIX = ${ARCH}-rtems7-
>>>
>> Yes, but if these files are installed does it need to be in defined in those
>> files?
> 
> The option substitution takes place before the files are installed. For example:

Ah OK. I did not pick that up in the patch.

Chris