Re: recent -current usr.bin/make changes break pkgsrc again

[David Holland <dholland-current%netbsd.org@localhost>]

On Tue, Jan 20, 2009 at 06:53:32PM +0000, David Laight wrote:
 > > > Plausibly all of defined(), make(), exists(), target() and command()
 > > > should just return 'false'.
 > > 
 > > Nah, those should be errors.
 > 
 > Quite possibly, but not historically :-(
 > I think the old code backed up the input parser by the number of
 > characters in 'defined' (etc) then applied the default function.
 > However the argument parser (for .if) doesn't count () so gets
 > a mismatch of the end of the name...
 > 
 > I also suspect that:
 >     .if exists(${FC_PATH})
 > first got checked as:
 >     .if exists()
 > Then as (backing up 6 bytes):
 >     .ifdef _PATH})
 > 
 > And hence a likely 'false' - but I still don't quite see what
 > removed the ')'.
 > 
 > make needs a command line flag to report 'dubious syntax' errors.

It should default to on, then, though. What you've just described is
rather uncomfortably like demons flying out of my nose.

-- 
David A. Holland
dholland%netbsd.org@localhost