-
Notifications
You must be signed in to change notification settings - Fork 39
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
Compile error in build.rs from README.md #73
Comments
Hi, thanks, good catch! I'm hoping to remove some of this complexity soon (#72) which will make it easier to use. Sure, a PR would be great. But if it's not convenient for you I can fix this myself later today. |
* Fix bminixhofer#73 * Propagate error
Probably it would be easier for you to just fix this problem yourself, but I'm keen to contribute to your cool project. 😃 |
It would've probably taken roughly the same time, and there's certainly some symbolic value :) Thanks for contributing! |
Using nlprule and nlprule-build version 0.6.4 and the build.rs script from the current README.md, I get a compile error while building:
I have solved it by adding error propagation to builds.rs:
If you want, I can make a PR.
The text was updated successfully, but these errors were encountered: