Skip to content

[2.x] Fix --sbt-version to expand to -Dsbt.version#8446

Merged
eed3si9n merged 2 commits intosbt:developfrom
Achieve3318:fix-sbt-version-flag
Jan 8, 2026
Merged

[2.x] Fix --sbt-version to expand to -Dsbt.version#8446
eed3si9n merged 2 commits intosbt:developfrom
Achieve3318:fix-sbt-version-flag

Conversation

@Achieve3318
Copy link
Copy Markdown
Contributor

Fixes #6645

The --sbt-version flag was incorrectly trying to download an sbt launcher with the specified version, which doesn't exist for many sbt versions.

Changes:

  • Modified --sbt-version handling to directly add -Dsbt.version property
  • Removed sbt_version variable assignment that caused launcher mismatch
  • acquire_sbt_jar now uses default launcher version instead

Now --sbt-version X.Y.Z correctly expands to -Dsbt.version=X.Y.Z without affecting the sbt launcher version used to run the build.

Copy link
Copy Markdown
Member

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

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

Thanks!

@eed3si9n
Copy link
Copy Markdown
Member

eed3si9n commented Jan 8, 2026

The change lgtm. Could you sign the CLA and let me know plz?

@Achieve3318
Copy link
Copy Markdown
Contributor Author

Achieve3318 commented Jan 8, 2026

@eed3si9n
https://www.lightbend.com/contribute/cla/scala
I can't login with github, So I login to CLA manually with my gmail.
Please check.

@eed3si9n
Copy link
Copy Markdown
Member

eed3si9n commented Jan 8, 2026

Let me know if you get a response from the email then. Currenlty CLA check is failing.

@eed3si9n eed3si9n changed the title Fix --sbt-version to expand to -Dsbt.version [2.x] Fix --sbt-version to expand to -Dsbt.version Jan 8, 2026
Fixes sbt#6645

The --sbt-version flag was incorrectly trying to download an sbt launcher
with the specified version, which doesn't exist for many sbt versions.

Changes:
- Modified --sbt-version handling to directly add -Dsbt.version property
- Removed sbt_version variable assignment that caused launcher mismatch
- acquire_sbt_jar now uses default launcher version instead

Now --sbt-version X.Y.Z correctly expands to -Dsbt.version=X.Y.Z without
affecting the sbt launcher version used to run the build.
@Achieve3318 Achieve3318 force-pushed the fix-sbt-version-flag branch from 3755f86 to a0f5c83 Compare January 8, 2026 19:51
@Achieve3318
Copy link
Copy Markdown
Contributor Author

@eed3si9n
I'm ready to sign the CLA, but both CLA signing URLs are showing OAuth redirect errors:

Both give: "The redirect_uri is not associated with this application"

Could you please:

  1. Provide an alternative way to sign the CLA, or
  2. Manually verify my CLA status, or
  3. Point me to a working CLA signing link?

I'm happy to sign however needed. Thanks!

@eed3si9n
Copy link
Copy Markdown
Member

eed3si9n commented Jan 8, 2026

@Achieve3318
Copy link
Copy Markdown
Contributor Author

Achieve3318 commented Jan 8, 2026

@eed3si9n

please check again.
CLA checking will be passed

Thanks

@eed3si9n eed3si9n merged commit 997ad96 into sbt:develop Jan 8, 2026
22 of 23 checks passed
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.

--sbt-version should expand to -Dsbt.version

2 participants