Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

platform: moved from arm-none-eabi-gcc 10.3.1-2.3 to 12.2.1-1.2 #1944

Merged
merged 6 commits into from Feb 10, 2023

Conversation

fpistm
Copy link
Member

@fpistm fpistm commented Feb 7, 2023

dev branch of stm32duino/BoardManagerFiles updated to references the new toolchain used by CI:
https://raw.githubusercontent.com/stm32duino/BoardManagerFiles/dev/package_stmicroelectronics_index.json

Deprecation notices

32-bit support

Support for 32-bit Intel Linux and Intel Windows was dropped in 2022. Support for 32-bit Arm Linux (armv7l) will be preserved for a while, due to the large user base of 32-bit Raspberry Pi systems.

due to newlib version 4.2.0 provided within
arm-none-eabi-gcc-12.2.1

See:
https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git

Title: In <stdio.h> provide only necessary types
Commit: 357d7fcc6ad1b0660e2b3131d09eb22e2559e46d

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
due to binutils version 2.39 provided within
arm-none-eabi-gcc-12.2.1
Enabled by default since this version.

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm added this to In progress in STM32 core based on ST HAL via automation Feb 7, 2023
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
@fpistm fpistm added this to the 2.5.0 milestone Feb 8, 2023
@ilg-ul
Copy link

ilg-ul commented Feb 8, 2023

If I understand your index.json file correctly, you provide support for multiple toolchain releases, including the latest 12.x.

Any reasons for not also including the 11.x? At least the latest v11.3.1-1.1.2?

@fpistm
Copy link
Member Author

fpistm commented Feb 8, 2023

Toolchain is linked to a release so next one will be with v12.

@ilg-ul
Copy link

ilg-ul commented Feb 8, 2023

Your index.json lists the following:

  • "9.3.1-1.3"
  • "10.2.1-1.1"
  • "10.3.1-2.3"
  • "12.2.1-1.2"

I confess that I don't know how this file is used, but if users can select any of the above releases, I think they should also be allowed to chose "11.3.1-1.1", even if the latest one is "12.2.1-1.2".

@fpistm
Copy link
Member Author

fpistm commented Feb 8, 2023

No user could not select the toolchain. The release give their tools dependencies.

@ilg-ul
Copy link

ilg-ul commented Feb 8, 2023

No user could not select the toolchain.

I see. But then why are those old definitions still present in the JSON file?

@fpistm
Copy link
Member Author

fpistm commented Feb 8, 2023

No user could not select the toolchain.

I see. But then why are those old definitions still present in the JSON file?

Because all release are available. User select the core version and then tools dependencies are selected.

@fpistm
Copy link
Member Author

fpistm commented Feb 9, 2023

To go further, the .json provides all information to install a specifc core version.
Ex here the 2.0.0:
https://github.com/stm32duino/BoardManagerFiles/blob/b815bc0f5ae37ec571dd9f7829d060dbebdb0382/package_stmicroelectronics_index.json#L15
which provides the tools dependencies:
https://github.com/stm32duino/BoardManagerFiles/blob/b815bc0f5ae37ec571dd9f7829d060dbebdb0382/package_stmicroelectronics_index.json#L49-L65

So 2.0.0 request to have gcc version 9.3.1-1.3

For the next 2.5.0, the 12.2.1-1.2 will be required.

@ilg-ul
Copy link

ilg-ul commented Feb 9, 2023

I'm not familiar with the Duino versioning scheme, and I don't know how much freedom you have with your releases, but, from the toolchain point of view, it is useful to have the choice to build a project with a specific version. Can you go back in time and insert an intermediate release, like 2.4.x, configured with GCC 11.3? Or make 2.5.0 with 11.3, and 2.5.1 with 12.2?

@fpistm
Copy link
Member Author

fpistm commented Feb 9, 2023

Honestly, I see no reason for doing this nor the interest.

@fpistm fpistm merged commit 2e03c80 into stm32duino:main Feb 10, 2023
STM32 core based on ST HAL automation moved this from In progress to Done Feb 10, 2023
@fpistm fpistm deleted the xpack_gcc_12.2.1-1.2 branch February 10, 2023 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants