RFR: JDK-8073328: Incremental build of gensrc broken
erik.joelsson at oracle.com
Tue Feb 17 15:38:47 UTC 2015
Right, thanks for catching that. The OS specific file is not there for
every OS. This patch should fix it. I'm testing all platforms now.
On 2015-02-17 16:24, Daniel Fuchs wrote:
> Hi Erik,
> I tried to import your patch, and now the build fails with the
> following error:
> make: *** No rule to make target
> needed by
> So it looks that something is still wrong :-(
> best regards,
> -- daniel
> On 17/02/15 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 messages.
>> 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