Skip to content

newlib: link without -lnosys#8795

Merged
kaspar030 merged 2 commits intoRIOT-OS:masterfrom
jnohlgard:pr/newlib-rm-lnosys
Apr 4, 2018
Merged

newlib: link without -lnosys#8795
kaspar030 merged 2 commits intoRIOT-OS:masterfrom
jnohlgard:pr/newlib-rm-lnosys

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

@jnohlgard jnohlgard commented Mar 18, 2018

Contribution description

lnosys and nosys.specs is mainly intended to provide empty stubs for syscalls, which we do not want since we implement the syscalls ourselves.

Issues/PRs references

none

@jnohlgard jnohlgard added Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: MIPS Platform: This PR/issue effects MIPS-based platforms labels Mar 18, 2018
@jnohlgard jnohlgard added this to the Release 2018.04 milestone Mar 18, 2018
@jnohlgard jnohlgard requested a review from kaspar030 March 18, 2018 09:07
@kaspar030 kaspar030 added CI: run tests If set, CI server will run tests on hardware for the labeled PR CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR labels Mar 29, 2018
@kaspar030
Copy link
Copy Markdown
Contributor

kaspar030 commented Apr 4, 2018

Alright, basic samr21-xpro tests still work. ACK.

@kaspar030 kaspar030 merged commit 16ff8a5 into RIOT-OS:master Apr 4, 2018
@jnohlgard jnohlgard deleted the pr/newlib-rm-lnosys branch April 5, 2018 10:19
@danpetry danpetry mentioned this pull request May 22, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kaspar030 kaspar030 kaspar030 approved these changes

Assignees

No one assigned

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: run tests If set, CI server will run tests on hardware for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: MIPS Platform: This PR/issue effects MIPS-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Milestone

Release 2018.04

Development

Successfully merging this pull request may close these issues.

2 participants

@jnohlgard @kaspar030