RFR: JDK-8073328: Incremental build of gensrc broken
Alan.Bateman at oracle.com
Tue Feb 17 15:23:08 UTC 2015
On 17/02/2015 14:56, Erik Joelsson wrote:
> Please review this fix for the incremental build behavior of the
> gensrc build step. After JDK-8073152 the charsets would
> unconditionally get regenerated each time make was invoked. The actual
> culprit was a stray backslash which prevented the 'touch' on the next
> line from executing.
> I took the liberty of cleaning up some more, adding to the
> prerequisites lists of the rules in question and fixing whitespace
> according to our guideline . The whitespace changes aren't visible
> in the webrev.
> I also quieted down the gensrc tool for charsets by adding the
> LOG_INFO macro. This will hide the output for the default warn level.
> To see it, run make with "LOG=info". For this to work I had to modify
> the tool to print its unconditional output to stdout instead of
> stderr. We want to keep stderr open to the log for any actual failure
> Bug: https://bugs.openjdk.java.net/browse/JDK-8073328
> Webrev: http://cr.openjdk.java.net/~erikj/8073328/webrev.jdk.01/
The updated dependences, the removal of the trailing slash (easy to miss
that one), and adding LOG_INFO look okay to me.
More information about the build-dev