RFR: JDK-8027584 disable ccache by default

[Dmitry Samersoff]

Magnus,

Glad to see ccache turned off by default but I'm second to David.

Please, keep --disable-ccache flag

-Dmitry

On 2014-01-29 13:03, David Holmes wrote:
> Hi Magnus,
> 
> I don't object to ccache being off by default but I would like the
> --disable-ccache flag to remain otherwise I will have to edit a bunch of
> build scripts and will need different versions for 9 vs 8. So please
> maintain backward compatibility here.
> 
> Aside: it would make most sense to me for there to be both --enable-x
> and --disable-x flags regardless of what the default might be.
> 
> Thanks,
> David
> 
> On 29/01/2014 9:31 AM, Magnus Ihse Bursie wrote:
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8027584
>>
>> While ccache can be used to speed up recompilations, there are also
>> several issues with ccache. Old versions does not work correctly for the
>> jdk (we try to detect that). It does not work well on other platforms
>> than Linux; on Windows it does not work at all. Nevertheless is ccache
>> enabled on Windows if it is installed, which it apparently tends to be
>> with Cygwin nowadays, requiring the use of --disable-ccache to avoid
>> compilation failure.
>>
>> In many circumstances ccache also does not provide any performance
>> benefit. For instance, new compilations gets a performance hit. Only
>> recompilations benefits.
>>
>> It is better to turn this upside down. This patch makes ccache disabled
>> by default, on all platforms. If you want ccache and are sure you are in
>> a situation that benefits from it, then you can enable it. It also
>> removes the output in configure hinting that ccache should be used.
>>
>> WebRev:
>> http://cr.openjdk.java.net/~ihse/JDK-8027584-disable-ccache-by-default/webrev.01
>>
>>
>>
>> /Magnus


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the source code.