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

Scala CLI does not support dependencyModules #2792

Closed
tgodzik opened this issue Mar 11, 2024 · 4 comments
Closed

Scala CLI does not support dependencyModules #2792

tgodzik opened this issue Mar 11, 2024 · 4 comments
Labels
bsp Issues tied to the implementation of BSP (Build Server Protocol) bug Something isn't working
Projects

Comments

@tgodzik
Copy link
Member

tgodzik commented Mar 11, 2024

Version(s)
v1.2.0

Describe the bug

2024.03.10 14:35:44 INFO  Connected to Scala CLI server v1.1.3
2024.03.10 14:35:44 INFO  Scala CLI: Fatal error has occured within bloop bsp server, method: buildTargetDependencyModules, with params: DependencyModulesParams [
2024.03.10 14:35:44 INFO  Scala CLI:   targets = ArrayList (
2024.03.10 14:35:44 INFO  Scala CLI:     BuildTargetIdentifier [
2024.03.10 14:35:44 INFO  Scala CLI:       uri = "file:/home/runner/work/metals/metals/tests/unit/target/e2e/file-watcher/didChangeWatchedFiles-notification/.scala-build/?id=didChangeWatchedFiles-notification_f4dd477a3a"
2024.03.10 14:35:44 INFO  Scala CLI:     ],
2024.03.10 14:35:44 INFO  Scala CLI:     BuildTargetIdentifier [
2024.03.10 14:35:44 INFO  Scala CLI:       uri = "file:/home/runner/work/metals/metals/tests/unit/target/e2e/file-watcher/didChangeWatchedFiles-notification/.scala-build/?id=didChangeWatchedFiles-notification_f4dd477a3a-test"
2024.03.10 14:35:44 INFO  Scala CLI:     ]
2024.03.10 14:35:44 INFO  Scala CLI:   )
2024.03.10 14:35:44 INFO  Scala CLI: ]. Shutting down the server:
2024.03.10 14:35:44 INFO  Scala CLI:  org.eclipse.lsp4j.jsonrpc.MessageIssueException: Message could not be parsed.
2024.03.10 14:35:44 INFO  Scala CLI: 	at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.handleResponseIssues(RemoteEndpoint.java:358)
2024.03.10 14:35:44 INFO  Scala CLI: 	at org.eclipse.lsp4j.jsonrpc.RemoteEndpoint.handle(RemoteEndpoint.java:317)
2024.03.10 14:35:44 INFO  Scala CLI: 	at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.handleMessage(StreamMessageProducer.java:198)
2024.03.10 14:35:44 INFO  Scala CLI: 	at org.eclipse.lsp4j.jsonrpc.json.StreamMessageProducer.listen(StreamMessageProducer.java:94)
2024.03.10 14:35:44 INFO  Scala CLI: 	at org.eclipse.lsp4j.jsonrpc.json.ConcurrentMessageProcessor.run(ConcurrentMessageProcessor.java:113)
2024.03.10 14:35:44 INFO  Scala CLI: 	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539)
2024.03.10 14:35:44 INFO  Scala CLI: 	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
2024.03.10 14:35:44 INFO  Scala CLI: 	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
2024.03.10 14:35:44 INFO  Scala CLI: 	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
2024.03.10 14:35:44 INFO  Scala CLI: 	at java.base/java.lang.Thread.run(Thread.java:840)
2024.03.10 14:35:44 INFO  Scala CLI: 

To Reproduce
Try to send dependencyModules request, connected to scalameta/metals#6213

It's wrongly set to true in

capabilities.setDependencyModulesProvider(true)

Expected behaviour
dependencyModulesProvider is set to true only if underlying Bloop supports it.

@tgodzik tgodzik added the bug Something isn't working label Mar 11, 2024
@Gedochao Gedochao added the bsp Issues tied to the implementation of BSP (Build Server Protocol) label Mar 11, 2024
@Gedochao Gedochao added this to To do in Issue Board via automation Mar 12, 2024
@fthomas
Copy link

fthomas commented Mar 15, 2024

This is somewhat related to this issue. Bloop supports dependencyModules since scalacenter/bloop#2197. Is there some ETA when this will be available in Scala CLI?

@tgodzik
Copy link
Member Author

tgodzik commented Mar 16, 2024

Alex fixed the release, so I will port that over for now (I need to reabsorb the fork at some point)

@tgodzik
Copy link
Member Author

tgodzik commented Mar 27, 2024

I am running the release in https://github.com/scala-cli/bloop-core/actions/runs/8457371951, once it's out we'll bump bloop in Scala CLI and do a release.

@tgodzik
Copy link
Member Author

tgodzik commented Mar 28, 2024

Closed by #2818

@tgodzik tgodzik closed this as completed Mar 28, 2024
Issue Board automation moved this from To do to Done Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bsp Issues tied to the implementation of BSP (Build Server Protocol) bug Something isn't working
Projects
Development

No branches or pull requests

3 participants