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.

Sign up for GitHub

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

Jump to bottom

[smoke test] Remove msp430 from blacklists #13393

Closed
wants to merge 2 commits into from
Closed

[smoke test] Remove msp430 from blacklists #13393

wants to merge 2 commits into from

Conversation

benpicco
Copy link
Contributor

@benpicco benpicco commented Feb 17, 2020 •
edited
Loading

Contribution description

With the updated toolchain, the blacklisting of msp430 might not be needed anymore.
Let's find out!

Testing procedure

Murdock will tell if the blacklisting is still needed.

Issues/PRs references

made possible by RIOT-OS/riotdocker#99
#12457 - oops, this has not been merged yet. Will rebase once it's in.

@benpicco
tests: remove msp430 from blacklists
9d0503e 
With the updated toolchain, blacklisting might not be neccecary anymore.
@benpicco
pkg: remove msp430 from blacklists
b19ae3f 
With the updated toolchain, blacklisting might not be neccecary anymore.
@benpicco benpicco added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation 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 labels Feb 17, 2020
@benpicco benpicco added Area: cpu Area: CPU/MCU ports State: waiting for other PR State: The PR requires another PR to be merged first labels Feb 17, 2020
@miri64 miri64 removed the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Feb 19, 2020
@miri64
Copy link
Member

miri64 commented Feb 19, 2020

#12457 - oops, this has not been merged yet. Will rebase once it's in.

Then let's not bother Murdock with it for now ;-).

@kaspar030
Copy link
Contributor

@benpicco I've added these changes to #12457. Maybe close this?

@benpicco benpicco closed this Feb 26, 2020
@benpicco benpicco deleted the msp430-unblacklist branch February 26, 2020 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aabadie aabadie Awaiting requested review from aabadie aabadie is a code owner

@fjmolinas fjmolinas Awaiting requested review from fjmolinas

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@MichelRottleuthner MichelRottleuthner Awaiting requested review from MichelRottleuthner MichelRottleuthner is a code owner

@miri64 miri64 Awaiting requested review from miri64 miri64 is a code owner

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss

@smlng smlng Awaiting requested review from smlng

Assignees
No one assigned
Labels
Area: build system Area: Build system Area: cpu Area: CPU/MCU ports State: waiting for other PR State: The PR requires another PR to be merged first Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@benpicco @miri64 @kaspar030