From owner-svn-src-stable@FreeBSD.ORG Tue May 13 06:09:29 2014 Return-Path: Delivered-To: svn-src-stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 90353733; Tue, 13 May 2014 06:09:29 +0000 (UTC) Received: from mail108.syd.optusnet.com.au (mail108.syd.optusnet.com.au [211.29.132.59]) by mx1.freebsd.org (Postfix) with ESMTP id 4E65C2FDC; Tue, 13 May 2014 06:09:28 +0000 (UTC) Received: from c122-106-147-133.carlnfd1.nsw.optusnet.com.au (c122-106-147-133.carlnfd1.nsw.optusnet.com.au [122.106.147.133]) by mail108.syd.optusnet.com.au (Postfix) with ESMTPS id C00C51A330E; Tue, 13 May 2014 16:09:21 +1000 (EST) Date: Tue, 13 May 2014 16:09:20 +1000 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Glen Barber Subject: Re: svn commit: r265917 - in stable/10: share/man/man4 sys/dev/bce sys/dev/bxe sys/modules/bce In-Reply-To: <20140512161817.GK1258@hub.FreeBSD.org> Message-ID: <20140513153541.S3047@besplex.bde.org> References: <201405121552.s4CFqnbX009177@svn.freebsd.org> <20140512161817.GK1258@hub.FreeBSD.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.1 cv=eojmkOZX c=1 sm=1 tr=0 a=7NqvjVvQucbO2RlWB8PEog==:117 a=PO7r1zJSAAAA:8 a=s01DRGYkj-EA:10 a=kj9zAlcOel0A:10 a=JzwRw_2MAAAA:8 a=84xCnt9HLjLxYQgaN1YA:9 a=CjuIK1q_8ugA:10 Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-10@freebsd.org, David C Somayajulu X-BeenThere: svn-src-stable@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: SVN commit messages for all the -stable branches of the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 13 May 2014 06:09:29 -0000 On Mon, 12 May 2014, Glen Barber wrote: > On Mon, May 12, 2014 at 03:52:49PM +0000, David C Somayajulu wrote: >> Log: >> MFC r265703 >> Modify Copyright information and other strings to reflect >> Qlogic Corporation's purchase of Broadcom's NetXtreme business. >> Added clean option to Makefile >> > >> Modified: stable/10/sys/modules/bce/Makefile >> ============================================================================== >> --- stable/10/sys/modules/bce/Makefile Mon May 12 14:46:32 2014 (r265916) >> +++ stable/10/sys/modules/bce/Makefile Mon May 12 15:52:49 2014 (r265917) >> @@ -5,4 +5,9 @@ SRCS= opt_bce.h if_bce.c miibus_if.h mii >> >> #CFLAGS += -DBCE_DEBUG=0 >> >> +clean: >> + rm -f opt_bdg.h device_if.h bus_if.h pci_if.h export_syms >> + rm -f *.o *.kld *.ko >> + rm -f @ machine x86 miibus_if.h miidevs.h opt_bce.h >> + >> .include > > Shouldn't these be listed in CLEANFILES ? Most of them already do. Most of them are automatically generated, and a small part of this is to put automatically generated files in CLEANFILES. The existence of the clean target is a larger bug. It defeats the default clean target which handles CLEANFILES and other things, but not all of the above. Some of them belong in the cleandir target. That is not defeated. Before this commit, the clean target did: % rm -f export_syms if_bce.ko if_bce.kld if_bce.o opt_bce.h miibus_if.h pci_if.h bus_if.h device_if.h miidevs.h and the cleandir target did this plus: % rm -f @ machine x86 % rm -f .depend GPATH GRTAGS GSYMS GTAGS % if [ -L /usr/src/sys/modules/bce/obj ]; then rm -f /usr/src/sys/modules/bce/obj; fi The new clean target does exactly the same as the old one plus a strict subset of the cleandir target, except: - it adds opt_budgerigar.h (sic), which apparently still is neither generated nor used by bce. - it removes *.o, *.kld and *.ko instead of if_bce.ko if_bce.kld if_bce.o. The default rule is careful to avoid using wildcards so as to not remove anything not generated by the makefile. Removing @, machine and x86 in clean instead of in cleandir is minor breakage. .depend and obj are still not removed in clean. Handling obj in a home made rule would be very difficult, since the obj dir can be almost anywhere (and not named obj) and it might be a directory instead of a symlink. Bruce