Skip to content

Make: buildtest: match exact board name#6483

Merged
PeterKietzmann merged 1 commit intoRIOT-OS:masterfrom
cgundogan:pr/buildtest_match_exact
Jan 26, 2017
Merged

Make: buildtest: match exact board name#6483
PeterKietzmann merged 1 commit intoRIOT-OS:masterfrom
cgundogan:pr/buildtest_match_exact

Conversation

@cgundogan
Copy link
Copy Markdown
Member

@cgundogan cgundogan commented Jan 26, 2017

Currently, the make buildtest matches msb-430 when determining whether an application needs to be linked or not, although only msb-430h is blacklisted. With this patch the board names are matched exactly, not as substrings.

See #6482

@cgundogan cgundogan added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: build system Area: Build system labels Jan 26, 2017
@cgundogan cgundogan added this to the Release 2017.01 milestone Jan 26, 2017
@cgundogan cgundogan added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Jan 26, 2017
Copy link
Copy Markdown
Contributor

@kaspar030 kaspar030 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK. Thanks for teaching me grep's <word> syntax! ;)

@cgundogan cgundogan force-pushed the pr/buildtest_match_exact branch from cee5fb1 to 70ceb9a Compare January 26, 2017 13:32
@cgundogan
Copy link
Copy Markdown
Member Author

rebased to include #6482 (master)

@PeterKietzmann PeterKietzmann merged commit 9534b84 into RIOT-OS:master Jan 26, 2017
@cgundogan cgundogan deleted the pr/buildtest_match_exact branch January 26, 2017 14:03
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