J. Mallett said: > * From "Steven G. Kargl" <[EMAIL PROTECTED]> > > The first patch fixes install(1). The reamining patches > > correctly document the breakage of rev 1.55 of xinstall.c
I should have stated that if the first patch isn't acceptable, then the remaining patches should be applied. > > The last patch is wrong, the others are good, though a warning about the > overriding of flags might be nice until this is ACCEPTED behaviour by the > user community. > > The third patch is wrong because there are things other than -C that one > might want to override INSTALL with, for example an INSTALL that uses a > replacement program that static relinks an executable being installed > to a partition where its dynamic dependencies are not satisfied. No. The third patch is correct (see below); otherwise I could set INSTALL to " install -C" QED. > > I've known people to do similar. > > > --- share/mk/sys.mk.orig Fri Jun 7 18:05:26 2002 > > +++ share/mk/sys.mk Fri Jun 7 18:06:13 2002 > > @@ -67,7 +67,7 @@ > > .endif > > EFLAGS ?= > > > > -INSTALL ?= install > > +INSTALL = install > > > > LEX ?= lex > > LFLAGS ?= > > -- Steve http://troutmask.apl.washington.edu/~kargl/ To Unsubscribe: send mail to [EMAIL PROTECTED] with "unsubscribe freebsd-current" in the body of the message
