Skip to content

Build on jdk Set release target to 8 for jdk11 compiler#313

Closed
pacomendes wants to merge 2 commits intoapache:masterfrom
pacomendes:CURATOR-523
Closed

Build on jdk Set release target to 8 for jdk11 compiler#313
pacomendes wants to merge 2 commits intoapache:masterfrom
pacomendes:CURATOR-523

Conversation

@pacomendes
Copy link
Copy Markdown

Test running ci build on openjdk11 with release 8

@asdf2014
Copy link
Copy Markdown
Member

ping @Randgalt @leventov

Comment thread .travis.yml
language: java
jdk:
- oraclejdk8
- openjdk11
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can test on both?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@pacomendes Would you please replace it with

jdk:
  - openjdk8
  - openjdk11

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This will likely fail on openjdk8 as the -release option is not recognised.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@pacomendes Raising a new PR to solve this problem #321

@martin-g
Copy link
Copy Markdown
Member

This PR is obsolete and can be closed!
The project uses Github Actions now!

@tisonkun
Copy link
Copy Markdown
Member

tisonkun commented Sep 6, 2022

We already build for JDK8 and JDK11 on master.

@tisonkun tisonkun closed this Sep 6, 2022
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.

5 participants