Good point, I will fix that.

/Erik

On 2015-04-15 22:01, David Katleman wrote:
The doc change is fine.

You need a corresponding autoconf update to verify that 4.0 or later is being used on windows builds, to prevent cryptic errors like Phil was running into.

The windows build should fail in configure if it finds a gnumake older than 4.0

        Thanks
                Dave

On 4/15/2015 12:41 AM, Erik Joelsson wrote:
Please review this change to README-builds.html. I have updated all references to the GNU make version to specify that we recommend 4.0 or later on Windows. Cygwin has been shipping with 4.x for a long time now and we have been using it internally since the source code layout change. Using older versions of make, particularly 3.81 version commonly used for JDK 7, has been known to cause trouble with the JDK 9 (post src reshuffle) build. While 3.82.x versions probably still works in many cases, I would rather have us move to 4.x since there are features there we would eventually like to be able to use.

Bug: https://bugs.openjdk.java.net/browse/JDK-8072023
Patch:
diff -r d0321d15e0a0 README-builds.html
--- a/README-builds.html
+++ b/README-builds.html
@@ -41,7 +41,8 @@
The build is now a "<code>configure &amp;&amp; make</code>" style build
                     </li>
                     <li>
-                        Any GNU make 3.81 or newer should work
+ Any GNU make 3.81 or newer should work, except on
+                        Windows where 4.0 or newer is recommended.
                     </li>
                     <li>
The build should scale, i.e. more processors should
@@ -358,8 +359,8 @@
                 For all systems:
                 <ul>
                     <li>
- Be sure the GNU make utility is version 3.81 or newer,
-                        e.g. run "<code>make -version</code>"
+ Be sure the GNU make utility is version 3.81 (4.0 on + windows) or newer, e.g. run "<code>make -version</code>"
                     </li>
                     <li>
                         Install a
@@ -1726,10 +1727,10 @@
             A few notes about using GNU make:
             <ul>
                 <li>
-                    You need GNU make version 3.81 or newer.
-                    If the GNU make utility on your systems is not
-                    3.81 or newer,
-                    see <a href="#buildgmake">"Building GNU make"</a>.
+ You need GNU make version 3.81 or newer. On Windows 4.0 or
+                    newer is recommended.
+ If the GNU make utility on your systems is not of a suitable + version see <a href="#buildgmake">"Building GNU make"</a>.
                 </li>
                 <li>
                     Place the location of the GNU make binary in the


/Erik


Reply via email to