Skip to content

Fix prebuilt launcher URL of sbtn on Windows#197

Merged
alexarchambault merged 1 commit intocoursier:mainfrom
scalacenter:sbtn-descriptor
Jan 27, 2023
Merged

Fix prebuilt launcher URL of sbtn on Windows#197
alexarchambault merged 1 commit intocoursier:mainfrom
scalacenter:sbtn-descriptor

Conversation

@julienrf
Copy link
Copy Markdown
Contributor

No description provided.

Comment thread apps/resources/sbtn.json
"x86_64-pc-linux": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-x86_64-pc-linux",
"aarch64-pc-linux": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-aarch64-pc-linux",
"x86_64-apple-darwin": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-x86_64-apple-darwin",
"aarch64-apple-darwin": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-aarch64-apple-darwin",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Note that that launcher (macOS on aarch64) is not yet produced by the sbt release workflow.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Tracked by sbt/sbt#7143

Comment thread apps/resources/sbtn.json
"aarch64-pc-linux": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-aarch64-pc-linux",
"x86_64-apple-darwin": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-x86_64-apple-darwin",
"aarch64-apple-darwin": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-aarch64-apple-darwin",
"x86_64-pc-win32": "zip+https://github.com/sbt/sbt/releases/download/v${version}/sbt-${version}.zip!sbt/bin/sbtn-x86_64-pc-win32.exe"
Copy link
Copy Markdown
Contributor Author

@julienrf julienrf Jan 27, 2023

Choose a reason for hiding this comment

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

The previous URL was not working on Windows because it was missing the .exe extension in the file name.

@alexarchambault alexarchambault merged commit fd91e1a into coursier:main Jan 27, 2023
@julienrf julienrf deleted the sbtn-descriptor branch January 27, 2023 14:50
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