Skip to content

Conversation

@jberkel
Copy link
Contributor

@jberkel jberkel commented Nov 28, 2025

The new given syntax as described in SIP-64 has been released as non-experimental in the Scala 3.6.x series. The default version of scalameta used in pants doesn't support this syntax yet, so you'll get compile errors during the dependency analysis stage (which is also very confusing, since a user might assume it happened during the compilation stage).

This PR updates scalameta to a more recent version which supports this syntax.

Failing test case (without the scalameta update):

pants.engine.process.ProcessExecutionFailure: Process 'Scala source dependency analysis failed.' failed with exit code 899.
stdout:

stderr:
__source_to_analyze/Source.scala:8: error: ; expected but : found
    given intOrder: Ord[Int]:
                            ^
	at scala.meta.internal.parsers.Reporter.syntaxError(Reporter.scala:16)

@jberkel
Copy link
Contributor Author

jberkel commented Nov 28, 2025

Looks like this is already being addressed in #22767

@sureshjoshi
Copy link
Member

Yeah, and as I mentioned over there - I'm working on Dependabotting some of it, so the PR might linger.

But thanks for this one! Once we come up with some reasonable approach, we can close both of these PRs at the same time.

@jberkel
Copy link
Contributor Author

jberkel commented Dec 17, 2025

No worries, I'm manually overriding the dependencies for now in pants.toml. The test might be useful to pick from this PR. Is it worth having dependabot support for this? I don't think there'll be many updates, only for major Scala syntax changes, which don't occur that often.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants