tech-toolchain archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/usr.bin/make

On Sun, Oct 25, 2020 at 05:37:36PM +0000, Simon J. Gerraty wrote:
 > Modified Files:
 > 	src/usr.bin/make: main.c
 > 	src/usr.bin/make/unit-tests: varmod-match-escape.exp
 > Log Message:
 > Skip reading .MAKE.DEPENDFILE if set to
 > "/dev/null" or anything starting with "no".
 > Ref:

I object to this, particularly the "no" part but also making /dev/null

This is an unnecessary workaround for a bug in freebsd's build

If there's going to be a change, it should be removing the :T from the
generated reference; there is no reason to remove the directory name
from a user-supplied .MAKE.DEPENDFILE, and the default doesn't have a
directory part. That would also remove the problem freebsd's seeing,
and would be, y'know, actually correct.

(also the application of :T is undocumented as well as wrong)

kept for reference:

 > To generate a diff of this commit:
 > cvs rdiff -u -r1.388 -r1.389 src/usr.bin/make/main.c
 > cvs rdiff -u -r1.1 -r1.2 src/usr.bin/make/unit-tests/varmod-match-escape.exp

David A. Holland

Home | Main Index | Thread Index | Old Index