Re: [edk2-devel] [PATCH 0/2] BaseTools: Switch ARM/AARCH64 CI gcc from Linaro to Arm

Re: [edk2-devel] [PATCH 0/2] BaseTools: Switch ARM/AARCH64 CI gcc from Linaro to Arm

[Rebecca Cran]

I've created a ticket in Bugzilla: 
https://bugzilla.tianocore.org/show_bug.cgi?id=3594.


-- 
Rebecca Cran


On 8/22/21 11:35 PM, Rebecca Cran wrote:
> Linaro no longer do gcc releases - Arm creates them now.
>
> Update the gcc_[arm,aarch64]_linux_ext_dep.yaml files in BaseTools/Bin to
> switch from Linaro's old release to the latest gcc 10.3-2021.07 release
> from Arm.
>
> The private PR https://github.com/tianocore/edk2/pull/1909 failed due to
> a problem with GCC5_[ARM,AARCH64]_PREFIX. I don't know if there are more
> changes I need to add, or if it's a problem with the CI system.
>
> Rebecca Cran (2):
>    BaseTools: Switch to downloading the ARM compiler from Arm's site
>    BaseTools: Switch to downloading the AARCH64 compiler from Arm's site
>
>   BaseTools/Bin/gcc_aarch64_linux_ext_dep.yaml | 10 +++++-----
>   BaseTools/Bin/gcc_arm_linux_ext_dep.yaml     | 10 +++++-----
>   2 files changed, 10 insertions(+), 10 deletions(-)
>

Re: [edk2-devel] [PATCH 0/2] BaseTools: Switch ARM/AARCH64 CI gcc from Linaro to Arm

[Ard Biesheuvel]

On Thu, 26 Aug 2021 at 23:03, Rebecca Cran <rebecca@bsdio.com> wrote:
>
> I've created a ticket in Bugzilla:
> https://bugzilla.tianocore.org/show_bug.cgi?id=3594.
>
>
> --
> Rebecca Cran
>
>
> On 8/22/21 11:35 PM, Rebecca Cran wrote:
> > Linaro no longer do gcc releases - Arm creates them now.
> >
> > Update the gcc_[arm,aarch64]_linux_ext_dep.yaml files in BaseTools/Bin to
> > switch from Linaro's old release to the latest gcc 10.3-2021.07 release
> > from Arm.
> >
> > The private PR https://github.com/tianocore/edk2/pull/1909 failed due to
> > a problem with GCC5_[ARM,AARCH64]_PREFIX. I don't know if there are more
> > changes I need to add, or if it's a problem with the CI system.
> >


You are changing the compiler from

  arm-linux-gnueabihf-gcc

to

  arm-none-linux-gnueabihf-gcc

(and likewise for aarch64) so I am not surprised you are hitting problems.


> > Rebecca Cran (2):
> >    BaseTools: Switch to downloading the ARM compiler from Arm's site
> >    BaseTools: Switch to downloading the AARCH64 compiler from Arm's site
> >
> >   BaseTools/Bin/gcc_aarch64_linux_ext_dep.yaml | 10 +++++-----
> >   BaseTools/Bin/gcc_arm_linux_ext_dep.yaml     | 10 +++++-----
> >   2 files changed, 10 insertions(+), 10 deletions(-)
> >
>

Re: [edk2-devel] [PATCH 0/2] BaseTools: Switch ARM/AARCH64 CI gcc from Linaro to Arm

[Rebecca Cran]

On 8/27/2021 1:31 PM, Ard Biesheuvel wrote:
> On Thu, 26 Aug 2021 at 23:03, Rebecca Cran <rebecca@bsdio.com> wrote:
>> I've created a ticket in Bugzilla:
>> https://bugzilla.tianocore.org/show_bug.cgi?id=3594.
>>
>>
>> --
>> Rebecca Cran
>>
>>
>> On 8/22/21 11:35 PM, Rebecca Cran wrote:
>>> Linaro no longer do gcc releases - Arm creates them now.
>>>
>>> Update the gcc_[arm,aarch64]_linux_ext_dep.yaml files in BaseTools/Bin to
>>> switch from Linaro's old release to the latest gcc 10.3-2021.07 release
>>> from Arm.
>>>
>>> The private PR https://github.com/tianocore/edk2/pull/1909 failed due to
>>> a problem with GCC5_[ARM,AARCH64]_PREFIX. I don't know if there are more
>>> changes I need to add, or if it's a problem with the CI system.
>>>
>
> You are changing the compiler from
>
>    arm-linux-gnueabihf-gcc
>
> to
>
>    arm-none-linux-gnueabihf-gcc
>
> (and likewise for aarch64) so I am not surprised you are hitting problems.

Ah, I see why that's a problem now. The path is constructed in 
BaseTools/Plugin/LinuxGcc5ToolChain/LinuxGcc5ToolChain.py:

             # make GCC5_ARM_PREFIX to align with tools_def.txt
             prefix = os.path.join(install_path, "bin", 
"arm-linux-gnueabihf-")
shell_environment.GetEnvironment().set_shell_var("GCC5_ARM_PREFIX", prefix)


That explains why it worked on my machine where GCC5_ARM_PREFIX and 
GCC5_AARCH64_PREFIX are already set.


-- 
Rebecca Cran