Skip to content

Conversation

@hamzaremmal
Copy link
Member

Before we used to rely on the presence of the scala-library:2.13.x in the classpath to copy the binaries. This was not the case for a long time now and we can correctly not depend on it in the bootstrapped library.

// because we allow cross-compilation.
versionScheme := Some("always"),
crossPaths := false,
crossVersion := CrossVersion.disabled,
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sjrd seems we have broken the artifact name on October 8th when we added the DottyJSPlugin to this project. This here fixes it but is it the correct way?

See the nightly releases: https://repo.scala-lang.org/ui/repos/tree/General/local-maven-nightlies/org/scala-js/scalajs-scalalib_2.13_sjs1_3/3.8.0-RC1-bin-20251008-6194563-NIGHTLY.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to be right way, yes.

…-language-server`

sbt doesn't evict a library dependency if it has the same dependency as a project
…js1_3`

This broke when we added the a dependency to the `DottyJSPlugin`
@hamzaremmal hamzaremmal force-pushed the stdlib-bootstrapped-deps branch from 7d82fbf to e1953d8 Compare November 12, 2025 14:07
@hamzaremmal hamzaremmal marked this pull request as ready for review November 12, 2025 14:16
@hamzaremmal hamzaremmal requested a review from Gedochao November 12, 2025 14:24
@Gedochao Gedochao added the backport:nominated If we agree to backport this PR, replace this tag with "backport:accepted", otherwise delete it. label Nov 12, 2025
@Gedochao Gedochao added this to the 3.8.0 milestone Nov 12, 2025
@hamzaremmal hamzaremmal merged commit b10f71b into scala:main Nov 12, 2025
53 checks passed
@hamzaremmal hamzaremmal deleted the stdlib-bootstrapped-deps branch November 12, 2025 16:24
@WojciechMazur WojciechMazur added backport:accepted This PR needs to be backported, once it's been backported replace this tag by "backport:done" and removed backport:nominated If we agree to backport this PR, replace this tag with "backport:accepted", otherwise delete it. labels Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:accepted This PR needs to be backported, once it's been backported replace this tag by "backport:done"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants