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

Enhance 3rd party libc support #8922

Closed
nashif opened this issue Jul 13, 2018 · 7 comments
Closed

Enhance 3rd party libc support #8922

nashif opened this issue Jul 13, 2018 · 7 comments
Assignees
Labels
area: C Library C Standard Library area: Out Of Tree Port Enhancement Changes/Updates/Additions to existing features

Comments

@nashif
Copy link
Member

nashif commented Jul 13, 2018

Using newlib by default across the project is currently not possible due to type conflicts. Zephyr should build with the newlib that comes with the Zephyr SDK and any newlib that comes pre-integrated with 3rd party toolchains.

Beside newlib, we need to support any 3rd libc and not limit the support to newlib and the minimal libc in the zephyr tree. A few options were discussed and being explored:

  • make newlib zephyr aware
  • Use PRI macro for 32bit integer types
  • Make types in Zephyr compatible with newlib
  • etc.

A few bugs are open that are related to this issue:

@nashif nashif added the Enhancement Changes/Updates/Additions to existing features label Jul 13, 2018
@nashif
Copy link
Member Author

nashif commented Jul 16, 2018

@galak @carlescufi @dleach02 FYI

@nashif
Copy link
Member Author

nashif commented Jul 16, 2018

I have just found what appears like a bug in our Zephyr SDK configuration. The toolchain does not expose any headers like unistd.h or sys/types.h, unlike any other toolchain we use. for example when you build with xtools or arm embedded toolchain, all the headers are available to zephyr, not with the Zephyr SDK forcing us to add all of those headers into the minimal libc in a non-compatible way.

This PR #8944 add the missing include

@nashif
Copy link
Member Author

nashif commented Jul 19, 2018

more issues #7441

@cfriedt
Copy link
Member

cfriedt commented Mar 19, 2022

@nashif - This can probably be closed now, no?

@stephanosio
Copy link
Member

This can probably be closed now, no?

@cfriedt technically, this issue is not resolved.

make newlib zephyr aware

zephyrproject-rtos/sdk-ng#350

Use PRI macro for 32bit integer types
Make types in Zephyr compatible with newlib

#46032

@zephyrbot
Copy link
Collaborator

Hi @stephanosio,

This issue, marked as an Enhancement, was opened a while ago and did not get any traction. It was just assigned to you based on the labels. If you don't consider yourself the right person to address this issue, please re-assing it to the right person.

Please take a moment to review if the issue is still relevant to the project. If it is, please provide feedback and direction on how to move forward. If it is not, has already been addressed, is a duplicate, or is no longer relevant, please close it with a short comment explaining the reason.

@nashif you are also encouraged to help moving this issue forward by providing additional information and confirming this request/issue is still relevant to you.

Thanks!

@stephanosio
Copy link
Member

Closing since Zephyr is currently compatible with various third-party C libraries without any major issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: C Library C Standard Library area: Out Of Tree Port Enhancement Changes/Updates/Additions to existing features
Projects
None yet
Development

No branches or pull requests

5 participants