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

cpu: Move ldscripts into separate directory #3120

Merged
merged 2 commits into from
May 30, 2015

Conversation

jnohlgard
Copy link
Member

Less clutter inside the cpu/* directories by placing linker scripts under cpu/*/ldscripts/CPUNAME.ld

Less clutter inside the cpu/* directories by placing linker scripts
under cpu/*/ldscripts/CPUNAME.ld
@jnohlgard jnohlgard added Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: AVR Platform: This PR/issue effects AVR-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels May 30, 2015
@jnohlgard jnohlgard added this to the Release 2015.06 milestone May 30, 2015
@jnohlgard jnohlgard added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label May 30, 2015
Linker warnings usually means something does not get placed where it is
supposed to and the most likely result is a broken binary.
Linker warnings should be treated as serious problems with the program
and should be solved before the program is tested on actual hardware.
@haukepetersen
Copy link
Contributor

I like! ACK when Travis is happy.

@haukepetersen
Copy link
Contributor

all green -> go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Platform: AVR Platform: This PR/issue effects AVR-based platforms Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants