Re: RFR: JDK-8172577: Builds for OS X after build 149 does not include Java Mission Control.app

2017-01-11 Thread Tim Bell
Erik: This patch fixes bug in the bundle logic, specifically the part that tries to work around files with spaces in them. In one of my recent changes in Bundles.gmk, I managed to break this completely. Bug: https://bugs.openjdk.java.net/browse/JDK-8172577 Webrev: http://cr.openjdk.java.net/~e

Re: RFR(xxs): 8172579: 8168503 broke AIX build

2017-01-11 Thread Volker Simonis
Hi Thomas, why not simply using GNU grep? We already insist for so many other tools on the GNU version that this wouldn't be a big thing. And as far as I remember, GNU grep is already installed on most of our machines anyway. You just have to place /opt/freeware/bin in front of your PATH. Your f

RFR: JDK-8172577: Builds for OS X after build 149 does not include Java Mission Control.app

2017-01-11 Thread Erik Joelsson
Hello, This patch fixes bug in the bundle logic, specifically the part that tries to work around files with spaces in them. In one of my recent changes in Bundles.gmk, I managed to break this completely. Bug: https://bugs.openjdk.java.net/browse/JDK-8172577 Webrev: http://cr.openjdk.java.net

RFR(xxs): 8172579: 8168503 broke AIX build

2017-01-11 Thread Thomas Stüfe
Dear all, please take a look at this tiny fix: Bug: https://bugs.openjdk.java.net/browse/JDK-8172579 Webrev: http://cr.openjdk.java.net/~stuefe/webrevs/8172579-colon-8168503-broke-AIX-build/webrev/common/autoconf/hotspot.m4.udiff.html JDK-8168503 exposed a bug in AIX grep. AIX grep, when called

Re: RFR: JDK-8172562 Changing log level on Javadoc causes total rebuild

2017-01-11 Thread Tim Bell
Hello Magnus Looks good to me as well. Tim On 01/11/17 06:41, Erik Joelsson wrote: Looks good. /Erik On 2017-01-11 14:03, Magnus Ihse Bursie wrote: This patch makes debugging Javadoc builds a bit easier. It makes sure debug flags are not stored in vardeps files, and it also sets the -verb

Re: RFR: JDK-8172562 Changing log level on Javadoc causes total rebuild

2017-01-11 Thread Erik Joelsson
Looks good. /Erik On 2017-01-11 14:03, Magnus Ihse Bursie wrote: This patch makes debugging Javadoc builds a bit easier. It makes sure debug flags are not stored in vardeps files, and it also sets the -verbose flag on LOG=trace. Bug: https://bugs.openjdk.java.net/browse/JDK-8172562 WebRev:

RFR: JDK-8172562 Changing log level on Javadoc causes total rebuild

2017-01-11 Thread Magnus Ihse Bursie
This patch makes debugging Javadoc builds a bit easier. It makes sure debug flags are not stored in vardeps files, and it also sets the -verbose flag on LOG=trace. Bug: https://bugs.openjdk.java.net/browse/JDK-8172562 WebRev: http://cr.openjdk.java.net/~ihse/JDK-8172562-javadoc-debuggability/w

Re: Using hard links for debug builds

2017-01-11 Thread Volker Simonis
Hi Ioi, I think the problem with links (both, hard and soft) is that they are not supported on all filesystems. The same applies to the proposed "cp --reflink". So you would at least have to check if it works during the configuration step. Regards, Volker On Wed, Jan 11, 2017 at 1:57 AM, Ioi La