Skip to content

Conversation

@astro
Copy link
Contributor

@astro astro commented May 10, 2018

These characters occur in the .svd file for NXP LPC13xx.

@jamesmunns
Copy link
Member

Thanks @astro!

bors r+

bors bot added a commit that referenced this pull request May 11, 2018
215: Add `'/'`, `' '` to `BLACKLIST_CHARS` r=jamesmunns a=astro

These characters occur in [the .svd file for NXP LPC13xx](https://github.com/posborne/cmsis-svd/blob/master/data/NXP/LPC13xx_svd_v1.svd).

Co-authored-by: Astro <astro@spaceboyz.net>
@jamesmunns
Copy link
Member

Looks like Travis tests passed (https://travis-ci.org/japaric/svd2rust/builds/377690793?utm_source=github_status&utm_medium=notification), lets see what our friend bors is waiting on, and I expect to merge this.

@astro you may want to consider enabling some of these tests in the CI now that they are not expected to fail.

@bors
Copy link
Contributor

bors bot commented May 11, 2018

Timed out

@Emilgardis
Copy link
Member

Emilgardis commented May 11, 2018

The same thing with timing out happened in #192, something weird is happening with bors

@japaric
Copy link
Member

japaric commented May 11, 2018

@Emilgardis I think the problem is that bors is waiting for appveyor to pass but appveyor is never running because the appveyor.yaml file was never updated to test the staging and trying branches ... 😅. I'll push a appveyor.yaml fix.

Feel free to merge this manually since the tests passed.

@Emilgardis Emilgardis merged commit f271405 into rust-embedded:master May 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants