Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#60450 closed defect (fixed)

bazel: use Java LTS release 11 for minimum version

Reported by: blair (Blair Zajac) Owned by: cjones051073 (Chris Jones)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: bazel

Description

Hi Chris,

Would it be possible to modify the bazel* ports to use Java 11 instead of 12, since Java 11 is an LTS release? I would prefer to have JDK 11 installed on my Mac to be consistent with JDK 11 on my Linux web servers, for the other code I develop.

for bazel-0.25 https://docs.bazel.build/versions/0.25.0/install-ubuntu.html says JDK 8 for bazel-2.0 https://docs.bazel.build/versions/2.0.0/install-ubuntu.html mentions JDK 8

Thoughts?

Change History (5)

comment:1 Changed 5 years ago by cjones051073 (Chris Jones)

Seems reasonable enough. I'll keep the fallbacks as they are, which means the versions the buildbots use will not change.

comment:2 Changed 5 years ago by Chris Jones <jonesc@…>

Resolution: fixed
Status: assignedclosed

In 9046c9812113ea798453e38f19ba27e1314e7ccd/macports-ports (master):

bazel: Make JDK 11 minimum required.
Closes: #60450

comment:3 Changed 5 years ago by blair (Blair Zajac)

Thanks for the quick change.

Can I submit this change for bazel-0.25?

--- a/devel/bazel/Portfile
+++ b/devel/bazel/Portfile
@@ -68,10 +68,6 @@ if { ${name} eq ${subport} } {
     checksums         rmd160  cfff49a4be85f7693ae615c293b13d0a447b80d7 \
                       sha256  7456032199852c043e6c5b3e4c71dd8089c1158f72ec554e6ec1c77007f0ab51 \
                       size    275804130
-    
-    # Required java version.
-    set java_v 12
-    set java_f openjdk12
 
     # min OS
     set min_darwin 14

comment:4 Changed 5 years ago by blair (Blair Zajac)

Or maybe better is to set java_v to 11+ and java_f to 11, thinking that maybe bazel 0.25.x may not build with newer versions in the future.

comment:5 Changed 5 years ago by cjones051073 (Chris Jones)

Fixed, thanks.

Note: See TracTickets for help on using tickets.