Re: Fix the version of required jdk

2016-03-11 Thread KUBOTA Yuji
Hi Erik, Thank you for quickly push! - Yuji On 2016-03-11, Erik Joelsson wrote: > 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 0

Re: Fix the version of required jdk

2016-03-11 Thread 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 a

Re: Fix the version of required jdk

2016-03-11 Thread Erik Joelsson
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.s

Re: Fix the version of required jdk

2016-03-10 Thread Volker Simonis
Hi Yuji, your fix looks reasonable. But as the name implies, 'generated-configure.sh' is a generated file. A complete patch should change the source files (in this case common/autoconf/help.m4) and recreate 'generated-configure.sh' by calling common/autoconf/autogen.sh (you should use autoconf 2.6

Re: Fix the version of required jdk

2016-03-10 Thread KUBOTA Yuji
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 @@

Fix the version of required jdk

2016-03-10 Thread KUBOTA Yuji
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/