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
