ports/153028: [PATCH] textproc/py-4suite-xml leaves extra files when deinstalled

[NIcola Vitale]

>Number:         153028
>Category:       ports
>Synopsis:       [PATCH] textproc/py-4suite-xml leaves extra files when deinstalled
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Sat Dec 11 22:00:28 UTC 2010
>Closed-Date:
>Last-Modified:
>Originator:     NIcola Vitale
>Release:        FreeBSD 8.1-RELEASE-p2 i386
>Organization:
n/a
>Environment:
FreeBSD mnemosine.domo.sva 8.1-RELEASE-p2 FreeBSD 8.1-RELEASE-p2 #12: Sun Dec  5 11:10:55 CET 2010     stan at mnemosine.domo.sva:/usr/obj/usr/src/sys/MNEMOSINE  i386

>Description:
textproc/py-4suite-xml leaves extra files under ${DOCSDIR} when installed with NOPORTDOCS defined (see the last lines of this log: http://goo.gl/q1CZ2 ).
>How-To-Repeat:
cd /usr/ports/textproc/py-4suite-xml && make install -DNOPORTDOCS
>Fix:


Patch attached with submission follows:

diff -ruN py-4suite-xml/files/patch-Ft-Lib-DistExt-InstallText.py py-4suite-xml.fix-NOPORTDOCS/files/patch-Ft-Lib-DistExt-InstallText.py
--- py-4suite-xml/files/patch-Ft-Lib-DistExt-InstallText.py	1970-01-01 01:00:00.000000000 +0100
+++ py-4suite-xml.fix-NOPORTDOCS/files/patch-Ft-Lib-DistExt-InstallText.py	2010-12-11 22:17:29.000000000 +0100
@@ -0,0 +1,16 @@
+--- Ft/Lib/DistExt/InstallText.py	2006-08-28 00:06:47.000000000 +0200
++++ Ft/Lib/DistExt/InstallText.py.port	2010-12-11 22:17:13.000000000 +0100
+@@ -26,10 +26,10 @@
+         self.set_undefined_options('install',
+                                    ('install_docs', 'install_dir'),
+                                    ('force', 'force'))
+-
++        portdocs = not os.environ.has_key('NOPORTDOCS')
+         self.files = [ f for f in self.distribution.doc_files
+-                       if isinstance(f, Structures.File) ]
+-        if self.distribution.license_file:
++                       if isinstance(f, Structures.File) and portdocs]
++        if self.distribution.license_file and portdocs:
+             self.files.append(Structures.File(self.distribution.license_file))
+         return
+ 


>Release-Note:
>Audit-Trail:
>Unformatted: