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

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Fri Jan 30 11:01:37 UTC 2015


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.

> 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



More information about the build-dev mailing list