You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
Passing the Java property into Scala CLI should also pass it into BSP.
To Reproduce
First run of Scala CLI should create a BSP setup in which -Dfoo=bar properties are also set as Java properties for the Scala CLI BSP command.
scala-cli -Dfoo=bar run .
so it should create scala-cli.json with the following content:
If we only support the first run, then we will run into bad DX in the following scenario:
user starts to play with the first version of their script: scala-cli run ., the scala-cli.json gets generated
user recognizes, that they want to add a critical JVM property (for example: -Djavax.net.ssl.trustStore=mycompanycerts in order for scala-cli to be able to fetch some dependencies from their company-internal Nexus repository)
does scala-cli.json stay as it was? Because if it does, then the instance of scala-cli is unable to fetch the required dependencies because of a javax.net.ssl.SSLHandshakeException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target exception (caused by the lack of the required certificate from the default cacerts/truststore file). Should the user be forced to search the web for a solution and then to update the scala-cli.json file by hand?
Gedochao
added
the
bsp
Issues tied to the implementation of BSP (Build Server Protocol)
label
Jun 17, 2024
Version(s)
0.2.1
Describe the bug
Passing the Java property into Scala CLI should also pass it into BSP.
To Reproduce
First run of Scala CLI should create a BSP setup in which
-Dfoo=bar
properties are also set as Java properties for the Scala CLI BSP command.so it should create
scala-cli.json
with the following content:Expected behaviour
It should also work with the
setup-ide
command.The text was updated successfully, but these errors were encountered: