[Box Backup-dev] Makefile.extra?

James O'Gorman boxbackup-dev@fluffy.co.uk
Fri, 24 Feb 2006 13:27:07 +0000


Ben Summers wrote:
> 
> On 24 Feb 2006, at 12:46, James O'Gorman wrote:
> 
>> Ben Summers wrote:
>>>> b) If they're not automatically generated, is it safe to just patch all
>>>> these files?
>>> Yes.
>>> What should we do for the next release to tidy this up?
>>
>> One thing I did notice was some inconsistency - sometimes "perl" was 
>> used, sometimes "/usr/bin/perl", sometimes "/usr/local/bin/perl".
> 
> Probably a result of lots of different people touching it! Although 
> there shouldn't be a /usr/local/bin/perl there.

You're right - I must've run the grep after I'd already patched some 
things without realising.

> Sounds like the best thing to do is to autoconf it, and use 
> Makefile.extra.in instead, which allows the prefix to be included. Then 
> change all the C++ to use a #define for the location too.
> 
> Are you up for doing that? I know it won't help your current porting 
> work, but would make it easier for next time.

I can certainly give it a go!

FreeBSD is in a ports freeze at the moment anyway (preparing for 5.5/6.1 
releases) so my updates wouldn't go in for at least a month, which gives 
me time to look at doing this.

James