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
Add scopes to the tasks defined by the sbt plugin #714
Conversation
@@ -235,18 +236,18 @@ object ScalaNativePluginInternal { | |||
logger.running(compilec) | |||
val result = Process(compilec, cwd) ! logger | |||
if (result != 0) { | |||
println("Failed to compile native library runtime code.") | |||
sys.error("Failed to compile native library runtime code.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this is a fix of #692
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup, I mentioned it in the commit message, but forgot to in the PR description. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that sys.error
will be deprecated in 2.13: scala/scala#5677. It's better to use throw new RuntimeException
instead.
The changes look good to me! 👍 |
3566d1a
to
1413c57
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 👍
Done! |
Maybe there is a regression or change specification of |
@xuwei-k Do you have an example of project that shows this behavior? I can't reproduce it on the |
It was incorrectly unscoped since scala-native#714
* Scope Scala Native's tasks to `Compile` and `Test` * Report native lib compilation failures * Adapt scripted tests to scoping changes * Remove unused imports Fixes scala-native#692
It was incorrectly unscoped since scala-native#714
|
@xuwei-k Can you please file an issue for this? |
The changes to the scripted tests may be seen as a problem, but they are actually correct. The following build definition would fail at runtime, saying that there are undefined settings:
Also, fixes #692