Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Offer xPack-provided toolchain and OpenOCD packages for mac-arm64 #73

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

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

markmentovai
Copy link
Contributor

@markmentovai markmentovai commented Jan 21, 2025

This should improve the development experience on mac-arm64 systems (Apple Silicon, M1 and later CPUs) by using toolchain and OpenOCD builds for that architecture, rather than relying on binary translation of the mac-x86_64 tools.

mac-arm64 tools are only added to the current versions of these packages in this index: xpack-arm-none-eabi-gcc 13.2.1-1.1, and xpack-openocd 0.12.0-4, both as used in the current STM32 platform version, 2.9.0. Future versions should continue to offer mac-arm64 builds in this package index.

Fixes #72

Cc @fpistm

This should improve the development experience on mac-arm64 systems
(Apple Silicon, M1 and later CPUs) by using toolchain and OpenOCD builds
for that architecture, rather than relying on binary translation of the
mac-x86_64 tools.

mac-arm64 tools are only added to the current versions of these packages
in this index: xpack-arm-none-eabi-gcc 13.2.1-1.1, and xpack-openocd
0.12.0-4, both as used in the current STM32 platform version, 2.9.0.
Future versions should continue to offer mac-arm64 builds in this
package index.

Link: stm32duino#72
Signed-off-by: Mark Mentovai <[email protected]>
@fpistm fpistm marked this pull request as draft January 22, 2025 08:05
@fpistm
Copy link
Member

fpistm commented Jan 22, 2025

Hi @markmentovai
First lookin at the Arduino cli specification and does not found this target:
https://arduino.github.io/arduino-cli/1.1/
But looking at GitHub Arudino and found it:
https://github.com/arduino/arduino-cli/blob/e9092ccad473288cc1f51c7e5f23ce1a54844761/docs/package_index_json-specification.md?plain=1#L184

Don't know why it is not in present in the online documentation...

Anyway, That would be fine to add but I will not merge it until next core release as it not enough to provide the support.
User will get an error to install it as other packages are not generated/referenced for this target as stated by you in the issue.

@markmentovai
Copy link
Contributor Author

#73 (comment):

Hi @markmentovai First lookin at the Arduino cli specification and does not found this target: https://arduino.github.io/arduino-cli/1.1/ But looking at GitHub Arudino and found it: https://github.com/arduino/arduino-cli/blob/e9092ccad473288cc1f51c7e5f23ce1a54844761/docs/package_index_json-specification.md?plain=1#L184

Don't know why it is not in present in the online documentation...

Starting at https://arduino.github.io/arduino-cli/1.1/, go to Package index specification, and you should find it there. Look for “MacOSX Arm64”.

Anyway, That would be fine to add but I will not merge it until next core release as it not enough to provide the support. User will get an error to install it as other packages are not generated/referenced for this target as stated by you in the issue.

I tested this and don’t think it’s true. If you wanted to, you could add a mac-arm64 version of the xPack tools to the package index now, and the only thing that would happen is that new mac-arm64 users would get those tools. (Existing mac-arm64 users that already have the mac-x86_64 tools would not be upgraded automatically.) It’s true that the other tools from https://github.com/stm32duino/Arduino_Tools wouldn’t change, but that wouldn’t be a problem. It’s fine for the mac-arm64 xPack tools and the other mac-x86_64 tools to coexist. So it’s up to you, but this should be fine to take now even without waiting for a release.

I also prepared stm32duino/Arduino_Tools#104 to provide mac-arm64 tools, although in that case, because the “universal” binaries replace the older mac-x86_64-only ones in the same package and at the same paths, it would absolutely be prudent to wait for a new release and new package build before making that change.

@fpistm fpistm marked this pull request as ready for review February 5, 2025 10:14
@fpistm
Copy link
Member

fpistm commented Feb 5, 2025

Let's go.

@fpistm fpistm merged commit 5d9a37c into stm32duino:main Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support mac-arm64 properly
2 participants