cvs commit: src Makefile.inc1

Alexander Leidinger Alexander at
Wed May 16 09:29:17 UTC 2007

Quoting Dag-Erling Smørgrav <des at> (from Wed, 16 May 2007  
10:25:14 +0200):

> Alexander Leidinger <Alexander at> writes:
>> Dag-Erling Smørgrav <des at> writes:
>> > Alexander Leidinger <Alexander at> writes:
>> > > The original code hat it before, but you CD into DESTDIR, so it is not
>> > > necessary anymore.
>> > Unless someone goofs and adds an absolute path to
>> > which is why I very deliberately left "${DESTDIR}/" in front of the path
>> > in the rm command, although I could probably replace it with "./".
>> That would make it more consistent.
> No, wait, there is another reason I had forgotten: the path passed to rm
> is shown to the user, so it should be the full path.

Ah, yes... ok, why not remove the cd DESTDIR and use DESTDIR instead  
of the ./? Or add a comment explaining why DESTDIR is used for the rm.


Bombeck's Rule of Medicine:
	Never go to a doctor whose office plants have died.    Alexander @ PGP ID = B0063FE7       netchild @  : PGP ID = 72077137

More information about the cvs-all mailing list