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

Broken --workspace doesn't work anymore #72

Closed
szpak opened this issue Sep 3, 2019 · 2 comments
Closed

Broken --workspace doesn't work anymore #72

szpak opened this issue Sep 3, 2019 · 2 comments

Comments

@szpak
Copy link

szpak commented Sep 3, 2019

--workspace doesn't seem to work anymore:

$ . gradle/install-jdk.sh -W ~/custom-java --url "https://api.adoptopenjdk.net/v2/binary/releases/openjdk11?openjdk_impl=openj9&os=linux&arch=x64&release=latest&type=jdk&heap_size=normal"
install-jdk.sh 2019-07-17
Variables:
  feature = <overridden by custom url>
       os = <overridden by custom url>
      url = https://api.adoptopenjdk.net/v2/binary/releases/openjdk11?openjdk_impl=openj9&os=linux&arch=x64&release=latest&type=jdk&heap_size=normal
   status = 302
Downloading JDK from https://api.adoptopenjdk.net/v2/binary/releases/openjdk11?openjdk_impl=openj9&os=linux&arch=x64&release=latest&type=jdk&heap_size=normal...
tar: /home/travis/custom-java: Cannot open

After local testing it seems that --output-directory overrides --directory-prefix introduced in July. It started to be visible as I use locally copied install-jdk.sh script which I just upgraded to get support for newer Java versions.

szpak added a commit to szpak/gradle-pitest-plugin that referenced this issue Sep 3, 2019
OpenJ9 build started to be broken:
sormuras/bach#72
@sormuras
Copy link
Owner

sormuras commented Sep 3, 2019

Thanks for the report.

Do you have an idea how to quickly fix this as well? My current time is limited, so I won't be able to dig into for about two weeks.

@szpak
Copy link
Author

szpak commented Sep 3, 2019

Maybe:

local local="--output-document=${workspace}jdk-${feature}.tar.gz"

plus:

local tar_options="--file ${workspace}jdk-${feature}.tar.gz"

with some trailing "/" adding to workspace if not provided (and if workspace is not "")

However, it's Bash, nothing is easy (and testable)... Take your time, maybe I will be determined enough to get into it before that ;).

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

No branches or pull requests

2 participants