RFR: JDK-8027566 Remove the old build system
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Fri Nov 8 08:10:20 UTC 2013
Bug: https://bugs.openjdk.java.net/browse/JDK-8027566
This is the aggregation of the three previous preliminary reviews posted.
Since there previous webrevs did not result in any suggestions of code
changes, the actual code is basically the same here as in the three
preliminary, with two exceptions.
First, the addition of a new version of the compare script
(common/bin/compare.sh) which was needed to properly compare the build
results with the previous output. (Prior to that fix, the compare script
assumed you were comparing with the build bits from the old build system.)
Second, I fixed an issue with bootcycle-images, which I didn't detect
earlier.
Also, with the broken webrev script fixed, it is easier to spot if a
file was just moved, or if it also was modified. (This was not clear
from the early partial reviews.)
We have verified, using the compare script, that the resulting build
bits do not differ from what was build before this patch was applied.
This comparison is still not finished for a few platforms, but we have
tested the majority and found no regressions. We will continue testing
the last platforms, but we don't expect any surprises.
WebRev:
http://cr.openjdk.java.net/~ihse/JDK-8027566-remove-old-build-system/webrev.01
For reference, here are the three previous webrevs, were the separate
steps in the removal process can be traced:
Part 1: Removes (conservatively) the majority of the old build, and
replaces the top-level Makefile.
http://cr.openjdk.java.net/~ihse/remove-old-build-part-1/webrev.02/
Part 2: Prepares make/* and makefiles/* for merging, and removes the
last bits of the old build system.
http://cr.openjdk.java.net/~ihse/remove-old-build-part-2/webrev.02/
Part 3: Moves the files in makefiles/* to make/*.
http://cr.openjdk.java.net/~ihse/remove-old-build-part-3/webrev.06/
/Magnus
More information about the build-dev
mailing list