Re: .depend line 264: Inconsistent operator for ipf

On Mon, May 30, 2005 at 11:39:06AM -0700, Kris Kennaway wrote:
> On Mon, May 30, 2005 at 02:22:10PM +0300, Ruslan Ermilov wrote:
> 
> > When someone changes the type of "foo" from file to directory,
> > this is exactly what happens.  sbin/ipf/ipf was a file, which
> > has a recorded dependency in your stale .depend file.  Now,
> > sbin/ipf/ipf is a directory, and it has a :: type of dependency
> > provided by bsd.subdir.mk in sbin/ipf/Makefile, hence the error
> > from make(1).  Just blow away your /usr/obj/usr/src/sbin/ipf.
> > I'm considering changing bsd.subdir.mk to prevent this from
> > happening again and again.
> 
> That would probably be a good idea..this question will surely come up
> again and again each time someone tries to update from 5.x to 6.x.
> 
The patch I'm testing now is very simple:

%%%
Index: bsd.subdir.mk
===================================================================
RCS file: /home/ncvs/src/share/mk/bsd.subdir.mk,v
retrieving revision 1.49
diff -u -r1.49 bsd.subdir.mk
--- bsd.subdir.mk	14 Feb 2005 12:57:51 -0000	1.49
+++ bsd.subdir.mk	30 May 2005 11:21:00 -0000
_at__at_ -57,7 +57,7 _at__at_
 	done
 .endif
 
-${SUBDIR}::
+${SUBDIR}:
 	${_+_}_at_if test -d ${.TARGET}.${MACHINE_ARCH}; then \
 		cd ${.CURDIR}/${.TARGET}.${MACHINE_ARCH}; \
 	else \
%%%


Cheers,
-- 
Ruslan Ermilov
ru_at_FreeBSD.org
FreeBSD committer