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

[#6631] Enable launcher script use --sbt-boot #6632

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

er1c
Copy link
Contributor

@er1c er1c commented Aug 21, 2021

This addresses #6631 to use --sbt-boot directory to download sbt-launch.jar or sbtn

Also added some minor SbtScriptTest output cleanup.

@er1c
Copy link
Contributor Author

er1c commented Aug 21, 2021

Looks like I broke --sbt-jar ooops

sbt Outdated
@@ -620,7 +644,6 @@ map_args () {
-no-share|--no-share) options=( "${options[@]}" "${noshare_opts[@]}" ) && shift ;;
-no-global|--no-global) options=( "${options[@]}" "-Dsbt.global.base=$(pwd)/project/.sbtboot" ) && shift ;;
-ivy|--ivy) require_arg path "$1" "$2" && options=( "${options[@]}" "-Dsbt.ivy.home=$2" ) && shift 2 ;;
-sbt-boot|--sbt-boot) require_arg path "$1" "$2" && options=( "${options[@]}" "-Dsbt.boot.directory=$2" ) && shift 2 ;;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not quite sure what the difference between map_args and process_args is, but trying to set the sbt_boot var here didn't work for some reason

@er1c er1c force-pushed the runner-script-sbt-boot branch 3 times, most recently from 3a17541 to a3945a2 Compare August 21, 2021 22:21
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.

None yet

1 participant