Skip to content

Conversation

@nicolasstucki
Copy link
Contributor

@nicolasstucki nicolasstucki commented Nov 25, 2022

[skip test]
[skip test_windows_fast]
[skip test_sbt]
[skip mima]
[skip docs]

@nicolasstucki nicolasstucki self-assigned this Nov 25, 2022
@nicolasstucki nicolasstucki force-pushed the upgrade-java-in-community-build branch 2 times, most recently from 56adb20 to 621ba3e Compare November 25, 2022 16:44
@nicolasstucki
Copy link
Contributor Author

community_build_a and community_build_b fail on Java 16. I will open an issue with the description of the failures.


steps:
- name: Set JDK 17 as default
run: echo "/usr/lib/jvm/java-17-openjdk-amd64/bin" >> $GITHUB_PATH
Copy link
Member

Choose a reason for hiding this comment

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

I don't think Java 17 is installed on the docker image we're currently running, so this might not do anything

Copy link
Member

Choose a reason for hiding this comment

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

The Dockerfile was updated in October 2021: lampepfl/dotty-ci#19, but the image we're running is from March 2021: https://github.com/lampepfl/dotty/blob/main/.github/workflows/ci.yaml#L37

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 changed it to JDK 16. We can update it when we update the image.

How do we update that image?

Copy link
Member

Choose a reason for hiding this comment

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

To push a new image, see instructions in https://github.com/lampepfl/dotty-ci#dotty-docker-image, then the ci.yaml file can be updated to point to the new one.

@nicolasstucki nicolasstucki force-pushed the upgrade-java-in-community-build branch from 621ba3e to 67a511f Compare November 28, 2022 13:16
)"

steps:
- name: Set JDK 15 as default
Copy link
Member

Choose a reason for hiding this comment

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

I don't think the image has JDK 15 either, just JDK 16 and JDK 8 as the default, so this probably doesn't do anything.

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

Labels

None yet

2 participants