Re: RFR: JDK-8074048 : Upgrading to ccache 1.3.10 disables the use of ccache

2015-03-03 Thread Steven Loomis
OK, thanks. Do I need a second or ready to commit? On 3/3/2015 1:14 AM, Erik Joelsson wrote: Still looks good. (We usually skip the generated-configure in the review since it's generated and quite large, at least if we remember to) /Erik On 2015-03-02 18:12, Steven Loomis wrote: up

Re: RFR: JDK-8074048 : Upgrading to ccache 1.3.10 disables the use of ccache

2015-03-02 Thread Steven Loomis
updated webrev (on cr) Bug: https://bugs.openjdk.java.net/browse/JDK-8074048 webrev: http://cr.openjdk.java.net/~srl/8074048/webrev.01/

RFR: JDK-8074048 : Upgrading to ccache 1.3.10 disables the use of ccache

2015-02-27 Thread Steven Loomis
Subject was wrong, this is a RFR on 8074048 On 2/27/2015 9:14 AM, Steven Loomis wrote: Bug:https://bugs.openjdk.java.net/browse/JDK-8074048 webrev: cr.openjdk.java.net has decided it doesn't like me, till it is fixed trivial change is below --- old/common/autoconf/

Re: RFR: JDK-8062223: Upgrading to ccache 1.3.10 disables the use of ccache

2015-02-27 Thread Steven Loomis
Bug:https://bugs.openjdk.java.net/browse/JDK-8074048 webrev: cr.openjdk.java.net has decided it doesn't like me, till it is fixed trivial change is below --- old/common/autoconf/build-performance.m42015-02-27 09:09:47.974447669 -0800 +++ new/common/autoconf/build-perfor

Re: RFR: JDK-8062223: Upgrading to ccache 1.3.10 disables the use of ccache

2015-02-27 Thread Steven Loomis
Sure.. I'll open a new one no problem. -s On 2/27/2015 1:17 AM, Erik Joelsson wrote: Right, didn't think of that case. The patch looks good but it needs a new bug. We cannot submit more changes with the same bugid so reopening doesn't work. /Erik On 2015-02-26 23:51, Stev

Re: RFR: JDK-8062223: Upgrading to ccache 1.3.10 disables the use of ccache

2015-02-26 Thread Steven Loomis
Late to the party, sorry but the fix didn't work for me. Suggested fix below - reopen the bug? https://bugs.openjdk.java.net/browse/JDK-8062223#comment-13613668 http://cr.openjdk.java.net/~erikj/8062223/webrev.root.03/