Skip to content

make: buildtests, info-buildsizes improvements#3358

Merged
jnohlgard merged 5 commits intoRIOT-OS:masterfrom
jnohlgard:pr/make-info-buildsizes-fixes
Jul 13, 2015
Merged

make: buildtests, info-buildsizes improvements#3358
jnohlgard merged 5 commits intoRIOT-OS:masterfrom
jnohlgard:pr/make-info-buildsizes-fixes

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

The make info-buildsizes target has been somewhat less useful since clean was added as a default target when doing buildtests.
This PR introduces a clean-intermediates target which will delete all archives and object files from the build directory but keep the elf and hex files.
make buildtests now calls clean-intermediates instead of clean after each platform run.
This PR also fixes a typo with the color codes in the terminal output of info-buildsizes-diff.

@jnohlgard jnohlgard added Area: tests Area: tests and testing framework Area: build system Area: Build system labels Jul 10, 2015
@jnohlgard jnohlgard mentioned this pull request Jul 10, 2015
4 tasks
@phiros phiros added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jul 11, 2015
@phiros
Copy link
Copy Markdown
Member

phiros commented Jul 11, 2015

ACK when Travis is happy.
BTW: I should have used a make distclean in compile_test.py instead of executing
clean in buildtest.

jnohlgard pushed a commit that referenced this pull request Jul 13, 2015
make: buildtests, info-buildsizes improvements
@jnohlgard jnohlgard merged commit 8724e6c into RIOT-OS:master Jul 13, 2015
@jnohlgard jnohlgard deleted the pr/make-info-buildsizes-fixes branch July 13, 2015 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: build system Area: Build system Area: tests Area: tests and testing framework CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants