RFR: JDK-8073328: Incremental build of gensrc broken
erik.joelsson at oracle.com
Tue Feb 17 15:23:00 UTC 2015
I realize that I missed a couple of my own notes on the original review
for JDK-8073152. Here is an update webrev where I address those points
* Remove unused variable CHARSET_EXTENDED
* Remove use of undefined variable $(CHARSET_EXTENDED_DATA)
However, I opted not to add all the java source files in the source
directories $(CHARSET_EXTENDED_JAVA_DIR) and $(CHARSET_EXTSRC_DIR) as
prerequisites. I don't think it would be correct.
On 2015-02-17 15: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/
>  http://openjdk.java.net/groups/build/doc/code-conventions.html
More information about the build-dev