From thomas@laurel.tmseck.homedns.org  Fri Apr 23 10:03:32 2004
Return-Path: <thomas@laurel.tmseck.homedns.org>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id AA26516A4CE
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 23 Apr 2004 10:03:32 -0700 (PDT)
Received: from smtp1.netcologne.de (smtp1.netcologne.de [194.8.194.112])
	by mx1.FreeBSD.org (Postfix) with ESMTP id 4177943D2F
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 23 Apr 2004 10:03:32 -0700 (PDT)
	(envelope-from thomas@laurel.tmseck.homedns.org)
Received: from laurel.tmseck.homedns.org (xdsl-195-14-221-230.netcologne.de [195.14.221.230])
	by smtp1.netcologne.de (Postfix) with SMTP id CE46038B18
	for <FreeBSD-gnats-submit@freebsd.org>; Fri, 23 Apr 2004 19:03:28 +0200 (MEST)
Received: (qmail 14027 invoked by uid 1001); 23 Apr 2004 17:03:34 -0000
Message-Id: <20040423170334.14026.qmail@laurel.tmseck.homedns.org>
Date: 23 Apr 2004 17:03:34 -0000
From: Thomas-Martin Seck <tmseck@netcologne.de>
Reply-To: Thomas-Martin Seck <tmseck@netcologne.de>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: [Maintainer] www/squid: fix plist generation, cleanup pkg-deinstall
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         65918
>Category:       ports
>Synopsis:       [Maintainer] www/squid: fix plist generation, cleanup pkg-deinstall
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    krion
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          maintainer-update
>Submitter-Id:   current-users
>Arrival-Date:   Fri Apr 23 10:10:19 PDT 2004
>Closed-Date:    Wed Apr 28 12:49:58 PDT 2004
>Last-Modified:  Wed Apr 28 12:49:58 PDT 2004
>Originator:     Thomas-Martin Seck
>Release:        FreeBSD 4.9-STABLE i386
>Organization:
a private site in Germany
>Environment:
FreeBSD ports repository as of 23 April 2004.

	
>Description:
- Add a missing "%D" when generating the package list entry for
  etc/squid/errors (spotted by bento's "files and directories not in
  plist" check)
- Probably bump PORTREVISION (I leave the final decision to the committer)
- Clean up pkg-deinstall:
  + remove an unnecessary variable
  + replace rmdir -p with two distinct rmdir calls since we
    do not want to delete $PKG_PREFIX too if it happens to be empty
	
>How-To-Repeat:
	
>Fix:
Apply this patch:

Index: Makefile
===================================================================
--- Makefile	(.../branches/freebsd)	(revision 42)
+++ Makefile	(.../trunk)	(revision 42)
@@ -29,7 +29,7 @@
 
 PORTNAME=	squid
 PORTVERSION=	2.5.5
-PORTREVISION=	5
+PORTREVISION=	6
 CATEGORIES=	www
 MASTER_SITES=	\
 		ftp://ftp.squid-cache.org/pub/%SUBDIR%/ \
@@ -314,7 +314,7 @@
 	    >>${TMPPLIST}
 	@${ECHO_CMD} "@dirrm etc/squid/errors/${d}" >>${TMPPLIST}
 .endfor
-	@${ECHO_CMD} "@unexec rmdir etc/squid/errors 2>/dev/null || true" \
+	@${ECHO_CMD} "@unexec rmdir %D/etc/squid/errors 2>/dev/null || true" \
 	    >>${TMPPLIST}
 
 changeuser:
Index: pkg-deinstall
===================================================================
--- pkg-deinstall	(.../branches/freebsd)	(revision 42)
+++ pkg-deinstall	(.../trunk)	(revision 42)
@@ -4,7 +4,6 @@
 #
 
 PATH=/bin:/usr/bin:/usr/sbin
-pkgname=$1
 
 case $2 in
 DEINSTALL)
@@ -16,10 +15,11 @@
 POST-DEINSTALL)
 	rmdir ${PKG_PREFIX}/etc/squid 2>/dev/null
 	rmdir ${PKG_PREFIX}/squid/cache 2>/dev/null
-	rmdir -p ${PKG_PREFIX}/squid/logs 2>/dev/null
-	echo "===> post-deinstallation information for ${pkgname}"
+	rmdir ${PKG_PREFIX}/squid/logs 2>/dev/null
+	rmdir ${PKG_PREFIX}/squid 2>/dev/null
+	echo "===> post-deinstallation information for $1"
 	echo ""
-	echo "     Please note that ${pkgname} was not completely removed"
+	echo "     Please note that $1 was not completely removed"
 	echo "     from this system."
 	echo ""
 	echo "     The cache and log directories, squid's user account,"

	


>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->krion 
Responsible-Changed-By: krion 
Responsible-Changed-When: Fri Apr 23 22:22:07 PDT 2004 
Responsible-Changed-Why:  
I'll take it 

http://www.freebsd.org/cgi/query-pr.cgi?pr=65918 
State-Changed-From-To: open->closed 
State-Changed-By: krion 
State-Changed-When: Wed Apr 28 12:49:50 PDT 2004 
State-Changed-Why:  
Committed, thanks! 

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