Re: Fix the version of required jdk

[Erik Joelsson]

Pushed: https://bugs.openjdk.java.net/browse/JDK-8151685

On 2016-03-11 09:20, Erik Joelsson wrote:

Hello Yuji,

Your second patch looks fine. I will sponsor the fix.

/Erik

On 2016-03-11 08:25, KUBOTA Yuji wrote:

Hi everyone

Sorry, I pasted a wrong patch at previous mail. The right patch is as below.

diff --git a/common/autoconf/generated-configure.sh
b/common/autoconf/generated-configure.sh
--- a/common/autoconf/generated-configure.sh
+++ b/common/autoconf/generated-configure.sh
@@ -4053,7 +4053,7 @@
      devkit)
        PKGHANDLER_COMMAND="sudo apt-get install build-essential" ;;
      openjdk)
-      PKGHANDLER_COMMAND="sudo apt-get install openjdk-7-jdk" ;;
+      PKGHANDLER_COMMAND="sudo apt-get install openjdk-8-jdk" ;;
      alsa)
        PKGHANDLER_COMMAND="sudo apt-get install libasound2-dev" ;;
      cups)
@@ -4076,7 +4076,7 @@
      devkit)

PKGHANDLER_COMMAND="sudo yum groupinstall \"Development Tools\"" ;;

      openjdk)
-      PKGHANDLER_COMMAND="sudo yum install java-1.7.0-openjdk" ;;
+      PKGHANDLER_COMMAND="sudo yum install java-1.8.0-openjdk-devel" ;;
      alsa)
        PKGHANDLER_COMMAND="sudo yum install alsa-lib-devel" ;;
      cups)
@@ -4941,7 +4941,7 @@
  #CUSTOM_AUTOCONF_INCLUDE

  # Do not change or remove the following line, it is needed for
consistency checks:
-DATE_WHEN_GENERATED=1457632864
+DATE_WHEN_GENERATED=1457680853

###############################################################################
  #
diff --git a/common/autoconf/help.m4 b/common/autoconf/help.m4
--- a/common/autoconf/help.m4
+++ b/common/autoconf/help.m4
@@ -106,7 +106,7 @@
      devkit)
        PKGHANDLER_COMMAND="sudo apt-get install build-essential" ;;
      openjdk)
-      PKGHANDLER_COMMAND="sudo apt-get install openjdk-7-jdk" ;;
+      PKGHANDLER_COMMAND="sudo apt-get install openjdk-8-jdk" ;;
      alsa)
        PKGHANDLER_COMMAND="sudo apt-get install libasound2-dev" ;;
      cups)
@@ -129,7 +129,7 @@
      devkit)

PKGHANDLER_COMMAND="sudo yum groupinstall \"Development Tools\"" ;;

      openjdk)
-      PKGHANDLER_COMMAND="sudo yum install java-1.7.0-openjdk" ;;
+      PKGHANDLER_COMMAND="sudo yum install java-1.8.0-openjdk-devel" ;;
      alsa)
        PKGHANDLER_COMMAND="sudo yum install alsa-lib-devel" ;;
      cups)

Thanks,
Yuji

2016-03-11 15:48 GMT+09:00 KUBOTA Yuji <kubota.y...@gmail.com>:

Hi everyone.

configure of jdk9 (and build-infra/jdk9) requires boot jdk version is 8 or 9.

But help message requires jdk7. Especially, yum_help requires JRE.
So I create a small patch to fix, please someone review it?

I do not have an account of openjdk, so I want a sponsor.

diff --git a/common/autoconf/generated-configure.sh
b/common/autoconf/generated-configure.sh
--- a/common/autoconf/generated-configure.sh
+++ b/common/autoconf/generated-configure.sh
@@ -4053,7 +4053,7 @@
      devkit)
        PKGHANDLER_COMMAND="sudo apt-get install build-essential" ;;
      openjdk)
-      PKGHANDLER_COMMAND="sudo apt-get install openjdk-7-jdk" ;;
+      PKGHANDLER_COMMAND="sudo apt-get install openjdk-8-jdk" ;;
      alsa)
        PKGHANDLER_COMMAND="sudo apt-get install libasound2-dev" ;;
      cups)
@@ -4076,7 +4076,7 @@
      devkit)

PKGHANDLER_COMMAND="sudo yum groupinstall \"Development Tools\"" ;;

      openjdk)
-      PKGHANDLER_COMMAND="sudo yum install java-1.7.0-openjdk" ;;

+ PKGHANDLER_COMMAND="sudo yum install java-1.8.0-openjdk-devel" ;;

      alsa)
        PKGHANDLER_COMMAND="sudo yum install alsa-lib-devel" ;;
      cups)

Thanks,
Yuji