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

build.rs: Fail fast on errors when building libhermit-rs #9

Merged
merged 1 commit into from May 7, 2020

Conversation

jschwe
Copy link
Contributor

@jschwe jschwe commented May 6, 2020

Previously failure of the build command in hermit-sys/build.rs would not lead to immediate failure. The build would only fail in the last step, when the hermit library can't be found. This PR fixes this.
Changes:

  • Asserts that the output status of the build command is "success"
  • Prints the output of the build command so that it can be viewed when building with -vv or if the build command was not successful.

Asserts that the output status of the build command is success
Prints the output of the build command so that it can be viewed when building with -vv or if the build command was not successful
@stlankes
Copy link
Contributor

stlankes commented May 7, 2020

👍

@stlankes stlankes merged commit 4e1697c into hermit-os:devel May 7, 2020
@jschwe jschwe mentioned this pull request May 7, 2020
3 tasks
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.

None yet

2 participants