[SCM] mp4v2/master: debian/rules: set ccache variables

[Reinhard Tartler]

On Wed, Aug 10, 2011 at 11:42:43 (CEST), Alessio Treglia wrote:

> On Tue, Aug 9, 2011 at 12:24 AM,  <mfv-guest at users.alioth.debian.org> wrote:
>> The following commit has been merged in the master branch:
>> commit 9567699e623f1d285e3bf0e8277724363af67504
>> Author: Matteo F. Vescovi <m.vescovi at revese.it>
>> Date:   Tue Aug 9 00:21:41 2011 +0200
>>
>>    debian/rules: set ccache variables
>
> Why enabling ccache? AFAIK it's unnecessary on Debian's buildds, it
> could just bring issues.

Indeed, I'd also support to revert this patch, I remember we have been
bitten by this in the past already (but I don't remember exactly which
package it was).

Instead, I'd rather suggest to configure your build environemnt
properly. For example pbuilder allows to set this up in pbuilderrc IIRC.

Moreover, the approach itself is fragile because it breaks with many
build systems. A much better approach is to modify the PATH environment
variable to put '/usr/lib/cache' at the front.

-- 
Gruesse/greetings,
Reinhard Tartler, KeyID 945348A4