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.

Sign up for GitHub

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

Jump to bottom

mx fetch-jdk failure with python2 #238

Closed
zakkak opened this issue Jun 22, 2021 · 2 comments
Closed

mx fetch-jdk failure with python2 #238

zakkak opened this issue Jun 22, 2021 · 2 comments

Comments

@zakkak
Copy link
Contributor

zakkak commented Jun 22, 2021

Running mx fetch-jdk without setting MX_PYTHON or MX_PYTHON_VERSION results in the following failure.

$ git clone --depth 1 [email protected]:graalvm/mx
$ ./mx/mx fetch-jdk --jdk-id openjdk8 --to jdk-dl
/tmp/mx/common.json -> "jdks" -> "oraclejdk16" -> "version": value (16.0.1+4) must be a str, not a unicode
@zakkak zakkak mentioned this issue Jun 22, 2021
Closed
@zakkak
Copy link
Contributor Author

zakkak commented Jun 22, 2021 •
edited
Loading

Closing as:

Hi @zakkak , we are close to removing all python2 support from mx so this is going in the wrong direction.

source: #239 (comment)

@zakkak zakkak closed this as completed Jun 22, 2021
@zakkak zakkak mentioned this issue Jun 22, 2021
Closed
zakkak added a commit to zakkak/mandrel that referenced this issue Jun 23, 2021
@zakkak
CI: Force the use of python3
1815c08 
mx is close to removing python2 support and the CI is currently failing
due to graalvm/mx#238
@zakkak zakkak mentioned this issue Jun 23, 2021
Merged
zakkak added a commit to graalvm/mandrel that referenced this issue Jun 23, 2021
@zakkak
CI: Force the use of python3
f9c4edb 
mx is close to removing python2 support and the CI is currently failing
due to graalvm/mx#238
@zakkak zakkak reopened this Jun 23, 2021
@zakkak
Copy link
Contributor Author

zakkak commented Jun 24, 2021

Fixed by fe5916d

@zakkak zakkak closed this as completed Jun 24, 2021
@zakkak zakkak mentioned this issue Nov 29, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make mx fetch-jdk python2 compatible zakkak/mx
1 participant
@zakkak