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

[toolchains] Refactor sys libs #3600

Merged
merged 2 commits into from
Jan 26, 2017
Merged

Conversation

theotherjimmy
Copy link
Contributor

@theotherjimmy theotherjimmy commented Jan 17, 2017

Refactor toolchains to use a common sys_libs variable. This allows the exporters to rely on that information.

Status

READY

##Todos

  • Tests - /morph export-build

@theotherjimmy
Copy link
Contributor Author

/morph export-build

@ilg-ul
Copy link
Contributor

ilg-ul commented Jan 17, 2017

please be aware that the sys_libs must also be used to generate the start/end group

-Wl,--start-group -lstdc++ -lsupc++ -lm -lc -lgcc -lnosys -Wl,--end-group

@theotherjimmy
Copy link
Contributor Author

Oh, thanks!

@mbed-bot
Copy link

Result: ABORTED

Your command has finished executing! Here's what you wrote!

/morph export-build

@theotherjimmy
Copy link
Contributor Author

Thanks @bridadan, I'm about to add another change.

@theotherjimmy
Copy link
Contributor Author

@ilg-ul The new implementation should generate the -Wl,--start-group ... -Wl,--end-group. Thanks for pointing it out!

@bridadan
Copy link
Contributor

/morph export-build

@bridadan
Copy link
Contributor

/morph test

@ilg-ul
Copy link
Contributor

ilg-ul commented Jan 17, 2017

@theotherjimmy, you're welcome!

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph export-build

Output

mbed Build Number: 89

All exports and builds passed!

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1421

All builds and test 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.

6 participants