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
"javafx-controls-17-ea+9-${javafx.platform}.jar" is not found #6564
Comments
Same problem here with sbt 1.5.3 and 17ea+14 - is there a workaround? |
Not that I know of |
There are Coursier fields called A workaround probably would look like using |
To a potential contributor: |
@eed3si9n note that sbt does not have a problem with 17-ea+8, the problem only starts with 17-ea+9 and above (tested until 17-ea+14). Edit, indeed they introduced more platforms, thought I have the feeling this shouldn't be the problem but it could be somehow related |
There has also been a change in this file: This is where I've done some experiments and it seems like the system property |
There are two new properties used to define First, But there is also a new property named
This non-activation may be a bug of coursier. If not, a solution is to define this property via For example, it works with mill by using a
The first part is just providing system properties but then I add |
Hah! Hello everyone affected by this issue :) I've been investigating it and arrived at the same conclusion as @s-bernard - coursier's property activation logic is slightly incorrect in this particular case. I hope this PR will fix it: coursier/coursier#2176 |
Nice! |
@keynmol Is there a way to use the pre-release v2.0.16-200-ge888c6dea of coursier in sbt? |
I believe there was a way, but I've never used it: https://get-coursier.io/docs/sbt-coursier#sbt-13x That said, I'm pretty sure either @alexarchambault or @eed3si9n have a way of doing it, so let's see if they can shed some light :) |
sbt uses lm-coursier-shaded_2.12 (https://repo1.maven.org/maven2/io/get-coursier/lm-coursier-shaded_2.12/). I don't think you can use the latest pre-releases unless you're willing to build and publish the artifact locally or something. |
@eed3si9n thanks for the clarification, in this case I'll see if I can help to get the fix into a new release. @alexarchambault can I somehow help to get the coursier release 2.0.17 out and make it available to sbt? |
I'm not sure what fixed it but I just upgraded to |
seems working for most JavaFx modules in JavaFX 18+, however as soon
|
@mindlid do we have reason to believe that the root cause is a bug in sbt and/or in Coursier? |
@SethTisue the only facts i can list: on Mac/Apple Silicon |
steps
Here are two different versions you can try to pull down and clone. This branch has the bug, note the version bump to
17-ea+9
https://github.com/Christewart/bitcoin-s-core/tree/2021-06-23-javafx-classifer-bug
problem
sbt does not seem to resolve classifiers correctly on 1.5.4, and here is the proof that it doesn't work on CI:
https://github.com/bitcoin-s/bitcoin-s/runs/2609905220#step:5:508
expectations
it resolves.
note
master branch runs on
17-ea+8
which seems to work ok.here are the links to the artifacts published on maven:
17-ea+8
https://repo1.maven.org/maven2/org/openjfx/javafx-base/17-ea+8/17-ea+9
(does not work when attempting to resolve dep!) https://repo1.maven.org/maven2/org/openjfx/javafx-base/17-ea+9/The text was updated successfully, but these errors were encountered: