Subject: bin/21117: kdump does not compile without "make clean" after recent change
To: None <gnats-bugs@gnats.netbsd.org>
From: None <kre@cs.mu.OZ.AU>
List: netbsd-bugs
Date: 04/13/2003 07:04:36
>Number:         21117
>Category:       bin
>Synopsis:       kdump does not compile without "make clean" after recent change
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    bin-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sat Apr 12 17:06:00 PDT 2003
>Closed-Date:
>Last-Modified:
>Originator:     Robert Elz
>Release:        NetBSD 1.6Q	(--current of 2003-04-13)
>Organization:
	Prince of Songkla University
>Environment:
	
	
System: NetBSD fuchsia.cs.mu.OZ.AU 1.6P NetBSD 1.6P (FUCHSIA) #35: Wed Mar 5 13:50:03 ICT 2003 root@:/usr/obj/sys/FUCHSIA i386
Architecture: i386
Machine: i386
>Description:
	sys/sockio.h recently had 2 ioctl definitions removed from it.
	After that, usr.bin/kdump does not compile, without doing a
	"make clean" as its constructed ioctl.c refers to the deleted
	ioctls but doesn't depend on sockio.h so doesn't get rebuilt
>How-To-Repeat:
	Have a pre-built set of .o files, etc, and "build.sh -U" after
	the (less than a day old) update to sockio.h
>Fix:
	Add sockio.h to the list of (currently just 2) include files
	that ioctl.c depends upon in usr.bin/kdump/Makefile.ioctl-c
	All other .h files included by sys/ioctl.h and sys/ioctl_compat.h
	should also be added.

	An alternative migt be to massage the .depend file in some way
	so that ioctl.c depends upon all of that stuff instead of
	ioctl.o - then the list of which include files require ioctl.c
	to be rebuilt could be maintained automatically.  If that is
	done, make sure it happens in Makefile.ioctl-c so usr.bin/ktruss
	benefits from the same fix.

	And, shouldn't the deleted ioctl's have been added to
	ioctl_compat.h anyway, so kdump can decode them for
	backwards compatability with old ktrace.out files, or is
	that not considered worth the bother?
>Release-Note:
>Audit-Trail:
>Unformatted: