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

[Erik Joelsson]

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/

/Erik

>> Also made it a fatal error to try to use ccache if the compiler does 
>> not support the option as it's unclear if it really works correctly 
>> then. On the other hand, that option has been around for a long time 
>> so it will never fail.
> Sounds good.
>
> /Magnus