From nobody@FreeBSD.org  Tue Feb 26 14:18:43 2008
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 608141065677
	for <freebsd-gnats-submit@FreeBSD.org>; Tue, 26 Feb 2008 14:18:43 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (www.freebsd.org [IPv6:2001:4f8:fff6::21])
	by mx1.freebsd.org (Postfix) with ESMTP id 5498D13C4F2
	for <freebsd-gnats-submit@FreeBSD.org>; Tue, 26 Feb 2008 14:18:43 +0000 (UTC)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (localhost [127.0.0.1])
	by www.freebsd.org (8.14.2/8.14.2) with ESMTP id m1QEFwT4028964
	for <freebsd-gnats-submit@FreeBSD.org>; Tue, 26 Feb 2008 14:15:58 GMT
	(envelope-from nobody@www.freebsd.org)
Received: (from nobody@localhost)
	by www.freebsd.org (8.14.2/8.14.1/Submit) id m1QEFwbM028963;
	Tue, 26 Feb 2008 14:15:58 GMT
	(envelope-from nobody)
Message-Id: <200802261415.m1QEFwbM028963@www.freebsd.org>
Date: Tue, 26 Feb 2008 14:15:58 GMT
From: Richard Tector <richardtector@thekeelecentre.com>
To: freebsd-gnats-submit@FreeBSD.org
Subject: PATCH: Add IPX files to be removed to OptionalObsoleteFiles.inc
X-Send-Pr-Version: www-3.1
X-GNATS-Notify:

>Number:         121121
>Category:       misc
>Synopsis:       PATCH: Add IPX files to be removed to OptionalObsoleteFiles.inc
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    antoine
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Tue Feb 26 14:20:01 UTC 2008
>Closed-Date:    Sun Mar 30 14:03:20 UTC 2008
>Last-Modified:  Sun Mar 30 14:03:20 UTC 2008
>Originator:     Richard Tector
>Release:        FreeBSD 7.0-PRERELEASE
>Organization:
CAPL Limited
>Environment:
FreeBSD daffy.tector.org.uk 7.0-PRERELEASE FreeBSD 7.0-PRERELEASE #0: Mon Feb 25 18:07:37 GMT 2008     richard@daffy.tector.org.uk:/usr/obj/usr/src/sys/DAFFY  amd64
>Description:
The attached patch adds a list of files that can be removed if WITHOUT_IPX has been set for a build with make check=old or delete-old. It removes the libraries, binaries, manpages and includes only installed when the above knob has not been set.

NB. I have also added for removal the compat libraries installed for 32bit systems. Other sections in the patched file do not do this, but it works fine on an i386 system I have.
>How-To-Repeat:

>Fix:
See attached patch



Patch attached with submission follows:

--- tools/build/mk/OptionalObsoleteFiles.inc.old	2008-01-21 18:47:37.000000000 +0000
+++ tools/build/mk/OptionalObsoleteFiles.inc	2008-02-26 14:15:31.000000000 +0000
@@ -578,9 +578,25 @@
 OLD_FILES+=usr/share/man/man8/ippool.8.gz
 .endif
 
-#.if ${MK_IPX} == no
-# to be filled in
-#.endif
+.if ${MK_IPX} == no
+OLD_LIBS+=lib/libipx.so.4
+OLD_FILES+=usr/include/netipx/ipx.h
+OLD_FILES+=usr/include/netipx/ipx_if.h
+OLD_FILES+=usr/include/netipx/ipx_pcb.h
+OLD_FILES+=usr/include/netipx/ipx_var.h
+OLD_FILES+=usr/include/netipx/spx.h
+OLD_FILES+=usr/include/netipx/spx_debug.h
+OLD_FILES+=usr/include/netipx/spx_timer.h
+OLD_FILES+=usr/include/netipx/spx_var.h
+OLD_DIRS+=usr/include/netipx
+OLD_FILES+=usr/lib/libipx.a
+OLD_FILES+=usr/lib/libipx.so
+OLD_FILES+=usr/lib32/libipx.a
+OLD_FILES+=usr/lib32/libipx.so
+OLD_LIBS+=use/lib32/libipx.so.4
+OLD_FILES+=usr/sbin/IPXrouted
+OLD_FILES+=usr/share/man/man8/IPXrouted.8.gz
+.endif
 
 .if ${MK_KERBEROS} == no
 OLD_FILES+=usr/bin/ksu


>Release-Note:
>Audit-Trail:

From: Richard Tector <richardtector@thekeelecentre.com>
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: misc/121121: PATCH: Add IPX files to be removed to OptionalObsoleteFiles.inc
Date: Tue, 26 Feb 2008 14:37:27 +0000

 Please note I missed:
 OLD_LIBS+=usr/lib32/libipx.so.4
 in the patch
 
 Regards,
 
 Richard
Responsible-Changed-From-To: freebsd-bugs->antoine 
Responsible-Changed-By: antoine 
Responsible-Changed-When: Tue Feb 26 17:54:28 UTC 2008 
Responsible-Changed-Why:  
Take: this is in included in a patch I have for PR 120492. 

http://www.freebsd.org/cgi/query-pr.cgi?pr=121121 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: misc/121121: commit references a PR
Date: Fri, 29 Feb 2008 18:59:59 +0000 (UTC)

 antoine     2008-02-29 18:59:28 UTC
 
   FreeBSD src repository
 
   Modified files:
     tools/build/mk       OptionalObsoleteFiles.inc 
   Log:
   Add files to remove when MK_GCOV, MK_IPX or MK_NCP are defined.
   
   Note: it may be a good idea to deduce obsolete usr/lib32/ files from
   obsolete lib/ and usr/lib/ files.
   
   PR:             120492
   PR:             121118
   PR:             121121
   Submitted by:   KAMIYA Satosi, Richard Tector
   Approved by:    rwatson (mentor)
   MFC after:      1 month
   
   Revision  Changes    Path
   1.13      +57 -9     src/tools/build/mk/OptionalObsoleteFiles.inc
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: open->patched 
State-Changed-By: antoine 
State-Changed-When: Fri Feb 29 19:12:27 UTC 2008 
State-Changed-Why:  
Patched in HEAD. 
Note: the include/netipx part was not commited, currently these 
headers are always installed (required by netsmb headers, see XXX 
in src/include/Makefile) 

http://www.freebsd.org/cgi/query-pr.cgi?pr=121121 

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: misc/121121: commit references a PR
Date: Sun, 30 Mar 2008 13:56:28 +0000 (UTC)

 antoine     2008-03-30 13:56:13 UTC
 
   FreeBSD src repository
 
   Modified files:        (Branch: RELENG_7)
     tools/build/mk       OptionalObsoleteFiles.inc 
   Log:
   MFC to RELENG_7:
     Add files to remove when MK_GCOV, MK_IPX or MK_NCP are defined.
   
     PR:             120492
     PR:             121118
     PR:             121121
     Submitted by:   KAMIYA Satosi, Richard Tector
     Approved by:    rwatson (mentor)
     MFC after:      1 month
   
   Revision  Changes    Path
   1.11.2.2  +61 -9     src/tools/build/mk/OptionalObsoleteFiles.inc
 _______________________________________________
 cvs-all@freebsd.org mailing list
 http://lists.freebsd.org/mailman/listinfo/cvs-all
 To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
 
State-Changed-From-To: patched->closed 
State-Changed-By: antoine 
State-Changed-When: Sun Mar 30 14:02:54 UTC 2008 
State-Changed-Why:  
Close: merged to RELENG_7. Thanks! 

http://www.freebsd.org/cgi/query-pr.cgi?pr=121121 
>Unformatted:
