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

dist: compile_test.py: build everything on changes in pkg/ #5149

Merged
merged 1 commit into from
Mar 23, 2016

Conversation

kaspar030
Copy link
Contributor

Currently, changes in pkg/ don't trigger rebuilding of affected applications or tests.
This PR rebuilds everything for changes in pkg/, analog to dist/.

Not the most efficient option, but simple and better than false CI positives.

@kaspar030 kaspar030 added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: build system Area: Build system labels Mar 23, 2016
@kaspar030 kaspar030 added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Mar 23, 2016
@miri64
Copy link
Member

miri64 commented Mar 23, 2016

Alright. ACK and go, when Murdock is happy.

@kaspar030 kaspar030 force-pushed the rebuild_all_if_pkg_changed branch from 7838fde to e308ad4 Compare March 23, 2016 13:29
@kaspar030
Copy link
Contributor Author

(force-pushed a typo fix)

@kaspar030
Copy link
Contributor Author

&go.

@kaspar030 kaspar030 merged commit 64cef15 into RIOT-OS:master Mar 23, 2016
@kaspar030 kaspar030 deleted the rebuild_all_if_pkg_changed branch March 23, 2016 13:37
@OlegHahm OlegHahm added this to the Release 2016.04 milestone Mar 23, 2016
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 CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants