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

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Tue Feb 10 09:56:08 UTC 2015


On 2015-02-05 15:31, Erik Joelsson wrote:
>
> On 2015-01-30 12:01, Magnus Ihse Bursie wrote:
>> On 2015-01-30 11:51, Erik Joelsson wrote:
>>> Hello,
>>>
>>> New webrev: http://cr.openjdk.java.net/~erikj/8062223/webrev.root.02/
>>>
>>> Fixed the anti pattern.
>>
>> Sort of. :-) You still ignore the flag silently if it is not 
>> supported. I would prefer if a AC_MSG_WARN or AC_MSG_NOTICE in that 
>> case. If the user has explicitely given an argument, I think they 
>> deserves to know that is was to no effect. But then again, that's no 
>> regression so I accept if you will not fix it now.
>>
> You are right, I think I got it right this time:
> http://cr.openjdk.java.net/~erikj/8062223/webrev.root.03/
Thank you! :-)

Looks good.

/Magnus



More information about the build-dev mailing list