James Carlson wrote: > Charles Binford writes: >> <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"> > > Tryin' to make it hard ... > >> If lint is invoked with the >> "-I/cb/admONlint/proto/root_i386/usr/include" option, I'm broke. If I >> remove that -I it all works fine.<br> > > That sounds like our old friend: CR 4393089. > > It's oft-reported. See CR 4875943 for a more modern copy.
Of course. So now those that failed to remember the past (or, more accurately, that never knew about it in the first place) have repeated it. As they were doomed to do. The ENVCPPFLAGS[1-4] are designed to pick stuff up out of parent proto, and aren't a good place to fix this. Charles, you can try something like this: In usr/src/Makefile.master, change the definition of CPPFLAGS.master to read > CPPFLAGS.master=$(DTEXTDOM) $(DTS_ERRNO) $(LINTONLYCPPFLAGS) \ > $(ENVCPPFLAGS1) $(ENVCPPFLAGS2) $(ENVCPPFLAGS3) $(ENVCPPFLAGS4) ...and then in your makefile, add: > lint := LINTONLYCPPFLAGS= -I$(SPRO_VROOT)/prod/include/lint Frankly, it's ugly, and it may break with compiler changes. And probably the definition of LINTONLYCPPFLAGS belongs in Makefile.master, for just that reason, but since it should be conditionally assigned only for the lint target, that seems somewhat crappy. At least right now, it's safe; the only thing therein is note.h and sys/note.h. And I checked that it works as expected. Can't really win here, other than perhaps to just use the deprecated-but-still-functional comment forms. Doesn't help with sys/queue.h. Or turn off CONSTCOND in your LINTFLAGS, like James McPherson suggested originally. --Mark
