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

doc/releases: Add picolibc migration notes #62261

Merged

Conversation

keith-packard
Copy link
Collaborator

There aren't any 'required' or 'recommended' changes when switching from the minimal C library to picolibc, but there are a bunch of things which application developers might need to be aware of. Add a separate section to list those.

fabiobaltieri
fabiobaltieri previously approved these changes Sep 4, 2023
@fabiobaltieri
Copy link
Member

@kartben ping

Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only two cosmetic/styling issues.
The text is great, and I like the idea of having a dedicated section for it, but we'll have to see how the overall structure of the migration guide might need to evolve as it becomes more crowded closer to the release date. // @carlescufi

from the toolchain as the standard C++ library depends upon the C library
ABI.

* Picolibc removes the `-ffreestanding` compiler option. This allows
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Picolibc removes the `-ffreestanding` compiler option. This allows
* Picolibc removes the ``-ffreestanding`` compiler option. This allows

doc/releases/migration-guide-3.5.rst Outdated Show resolved Hide resolved
There aren't any 'required' or 'recommended' changes when switching from
the minimal C library to picolibc, but there are a bunch of things which
application developers might need to be aware of. Add a separate section to
list those.

Signed-off-by: Keith Packard <keithp@keithp.com>
@fabiobaltieri fabiobaltieri merged commit 28e6839 into zephyrproject-rtos:main Sep 8, 2023
14 checks passed
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.

4 participants