Thanks for the review Chris.
For the record, here’s the updated webrev:
http://cr.openjdk.java.net/~vinnie/8027567/webrev.01/
On 30 Oct 2013, at 16:57, Chris Hegarty wrote:
> The source changes look fine to me, I'm happy to be listed as a reviewer for
> this. ( generated-configure.sh, I jus
The source changes look fine to me, I'm happy to be listed as a reviewer
for this. ( generated-configure.sh, I just assume is ok once
common/autoconf/autogen.sh is run ).
-Chris.
On 30/10/2013 16:24, Vincent Ryan wrote:
Sure. I’ll re-run autogen.sh and submit an updated webrev.
Thanks.
On 3
Sure. I’ll re-run autogen.sh and submit an updated webrev.
Thanks.
On 30 Oct 2013, at 16:15, Erik Joelsson wrote:
> I'm leaving for the day pretty soon, so feel free to push this to tl if you
> have time.
>
> If you do, don't forget to run common/autoconf/autogen.sh and to also submit
> the
I'm leaving for the day pretty soon, so feel free to push this to tl if
you have time.
If you do, don't forget to run common/autoconf/autogen.sh and to also
submit the closed generated-configure.
/Erik
On 2013-10-30 17:07, Vincent Ryan wrote:
Thanks Erik. We’re seeing the issue in TL so it’
Thanks Erik. We’re seeing the issue in TL so it’d be great to get this fixed
today.
Would you like to take ownership of this issue or shall I push your fix to TL?
On 30 Oct 2013, at 16:00, Erik Joelsson wrote:
> I found a solution that's also more readable. Posting inline since it's so
> small
I found a solution that's also more readable. Posting inline since it's
so small a change:
diff -r 4f2011496393 common/autoconf/basics.m4
--- a/common/autoconf/basics.m4
+++ b/common/autoconf/basics.m4
@@ -514,7 +514,7 @@
if test "x$IS_GNU_MAKE" = x; then
AC_MSG_NOTICE([Found potenti
It seems the problem is with the grep tool used to parse the version
string. /usr/xpg4/bin/grep doesn't handle '\(3\.8[12]\)\|\(4\.\)' the
same as gnu grep. In jprt it finds /usr/sfw/bin/ggrep which works
better. I will see if I can figure out something that works with both.
/Erik
On 2013-10-
Thanks Chris.
So reverting to the previous version of builds.m4 will fix this issue on
Solaris but will undo
the fix for 8026528 on Windows. Maybe Eric can advise.
On 30 Oct 2013, at 15:05, Chris Hegarty wrote:
> Hi Vinnie,
>
> I have seen this issue myself, kind of funny ;-)
>
> ...
> co
Hi Vinnie,
I have seen this issue myself, kind of funny ;-)
...
configure: Found GNU make at /java/devtools/i386/bin/make, however this
is not version 3.81 or later. (it is: GNU Make 3.81). Ignoring.
configure: error: Cannot find GNU make 3.81 or newer! Please put it in
the path, or add e.g. M
Please review this fix to correct the JDK8 build Configure script.
It reverts a recent change to common/autoconf/basics.m4 that was causing a
build failure on Solaris.
Bug: https://bugs.openjdk.java.net/browse/JDK-8027567
Webrev: http://cr.openjdk.java.net/~vinnie/8027567/
10 matches
Mail list logo
