From utisoft@hotmail.com  Mon Jun  6 19:03:36 2011
Return-Path: <utisoft@hotmail.com>
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id E88241065670
	for <FreeBSD-gnats-submit@freebsd.org>; Mon,  6 Jun 2011 19:03:36 +0000 (UTC)
	(envelope-from utisoft@hotmail.com)
Received: from blu0-omc1-s14.blu0.hotmail.com (blu0-omc1-s14.blu0.hotmail.com [65.55.116.25])
	by mx1.freebsd.org (Postfix) with ESMTP id 9F6AE8FC16
	for <FreeBSD-gnats-submit@freebsd.org>; Mon,  6 Jun 2011 19:03:36 +0000 (UTC)
Received: from BLU0-SMTP131 ([65.55.116.8]) by blu0-omc1-s14.blu0.hotmail.com with Microsoft SMTPSVC(6.0.3790.4675);
	 Mon, 6 Jun 2011 12:03:35 -0700
Received: from zeus.bayofrum.net ([90.221.7.100]) by BLU0-SMTP131.phx.gbl over TLS secured channel with Microsoft SMTPSVC(6.0.3790.4675);
	 Mon, 6 Jun 2011 12:03:34 -0700
Received: from zeus.bayofrum.net (crees@localhost [127.0.0.1])
	by zeus.bayofrum.net (8.14.4/8.14.4) with ESMTP id p56J0dlt056306
	(version=TLSv1/SSLv3 cipher=DHE-DSS-AES256-SHA bits=256 verify=NO)
	for <FreeBSD-gnats-submit@freebsd.org>; Mon, 6 Jun 2011 20:00:40 +0100 (BST)
	(envelope-from crees@zeus.bayofrum.net)
Received: (from crees@localhost)
	by zeus.bayofrum.net (8.14.4/8.14.4/Submit) id p56J0cuT056303;
	Mon, 6 Jun 2011 20:00:38 +0100 (BST)
	(envelope-from crees)
Message-Id: <BLU0-SMTP13110406874DE5B1C2FD1F0A9600@phx.gbl>
Date: Mon, 6 Jun 2011 20:00:38 +0100
From: Chris Rees <utisoft@gmail.com>
Sender: <utisoft@hotmail.com>
Reply-To: Chris Rees <utisoft@gmail.com>
To: FreeBSD-gnats-submit@freebsd.org
Cc:
Subject: Fix port: databases/postgresql91-server should use USERS
X-Send-Pr-Version: 3.113
X-GNATS-Notify:

>Number:         157667
>Category:       ports
>Synopsis:       Fix port: databases/postgresql91-server should use USERS
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    girgen
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Mon Jun 06 19:10:07 UTC 2011
>Closed-Date:    Wed Jul 20 12:11:18 UTC 2011
>Last-Modified:  Wed Jul 20 12:11:18 UTC 2011
>Originator:     Chris Rees
>Release:        FreeBSD 8.2-RELEASE-p1 i386
>Organization:
bayofrum
>Environment:
System: FreeBSD zeus.bayofrum.net 8.2-RELEASE-p1 FreeBSD 8.2-RELEASE-p1 #0: Sat Apr 30 15:09:06 BST 2011 root@zeus.bayofrum.net:/usr/obj/usr/src/sys/ZEUS i386


	
>Description:
	

	This port uses pkg-install to create new users instead of using USERS= and GROUPS=

>How-To-Repeat:
	
>Fix:

	

	- Use USERS and GROUPS
	- Remove quotes from BROKEN to make portlint happy

	Submitted by: Chris Rees (utisoft@gmail.com)


--- databases-postgresql91-server.patch begins here ---
Index: Makefile
===================================================================
RCS file: /exports/cvsroot-freebsd/ports/databases/postgresql91-server/Makefile,v
retrieving revision 1.217
diff -u -r1.217 Makefile
--- Makefile	28 Apr 2011 07:46:15 -0000	1.217
+++ Makefile	2 Jun 2011 13:47:19 -0000
@@ -35,7 +35,6 @@
 
 PG_USER?=	pgsql
 PG_GROUP?=	pgsql
-PG_UID?=	70
 
 LDFLAGS+=	-L${LOCALBASE}/lib
 INCLUDES+=	-I${LOCALBASE}/include
@@ -62,6 +61,8 @@
 USE_PGSQL=	yes
 WANT_PGSQL_VER=	${PORTVERSION:C/([0-9][0-9]*)\.([0-9][0-9]*).*/\1\2/g}
 SUB_FILES+=	502.pgsql
+USERS=		${PG_USER}
+GROUPS=		${PG_GROUP}
 .endif
 
 .if !defined(SLAVE_ONLY)
@@ -173,7 +174,7 @@
 
 .  if defined(WITH_MIT_KRB5)
 .    if exists(/usr/lib/libkrb5.so) || exists(/usr/bin/krb5-config)
-BROKEN=		"You must remove heimdal's /usr/bin/krb5-config and /usr/lib/libkrb5.so*, and set NO_KERBEROS=true in /etc/make.conf to build successfully with MIT-KRB"
+BROKEN=		You must remove heimdal's /usr/bin/krb5-config and /usr/lib/libkrb5.so*, and set NO_KERBEROS=true in /etc/make.conf to build successfully with MIT-KRB
 .    endif
 # Allow defining a home built MIT Kerberos by setting KRB5_HOME
 .    if defined(KRB5_HOME) && exists(${KRB5_HOME}/lib/libgssapi_krb5.a) && exists(${KRB5_HOME}/bin/krb5-config)
@@ -309,20 +310,10 @@
 .  endif
 .  if exists(${FILESDIR}/pkg-install${PKGNAMESUFFIX}.in)
 SUB_FILES+=     pkg-install${PKGNAMESUFFIX}
-SUB_LIST=       PG_GROUP=$(PG_GROUP) \
-                PG_USER=$(PG_USER) \
-                PG_UID=$(PG_UID)
-PLIST_SUB+=     PG_USER=$(PG_USER)
 PKGINSTALL=     ${WRKDIR}/pkg-install${PKGNAMESUFFIX}
 .  endif
 .endif
 
-.if defined(SERVER_ONLY)
-pre-su-install:
-	@ ${SETENV} PKG_PREFIX=${PREFIX} PG_USER=$(PG_USER) PG_GROUP=$(PG_GROUP) \
-		${SH} ${PKGINSTALL} ${PORTNAME} PRE-INSTALL
-.endif
-
 .if !defined(NO_BUILD)
 post-patch:
 	@${REINPLACE_CMD} s/@PTHREAD_LIBS@// ${WRKSRC}/src/Makefile.global.in
Index: pkg-plist-server
===================================================================
RCS file: /exports/cvsroot-freebsd/ports/databases/postgresql91-server/pkg-plist-server,v
retrieving revision 1.32
diff -u -r1.32 pkg-plist-server
--- pkg-plist-server	28 Apr 2011 07:46:15 -0000	1.32
+++ pkg-plist-server	4 Jun 2011 13:49:46 -0000
@@ -756,9 +756,8 @@
 share/postgresql/tsearch_data/synonym_sample.syn
 share/postgresql/tsearch_data/thesaurus_sample.ths
 @dirrmtry share/postgresql/tsearch_data
-@unexec echo "Warning: If you will *NOT* use this package anymore, please remove the pgsql user manually."
 @dirrmtry share/postgresql
 @dirrmtry etc/periodic/daily
 @dirrmtry etc/periodic
 @dirrmtry lib/postgresql
-@dirrmtry %%PG_USER%%
+@dirrmtry pgsql
Index: files/pkg-install-server.in
===================================================================
RCS file: /exports/cvsroot-freebsd/ports/databases/postgresql91-server/files/pkg-install-server.in,v
retrieving revision 1.1
diff -u -r1.1 pkg-install-server.in
--- files/pkg-install-server.in	18 Apr 2011 23:34:27 -0000	1.1
+++ files/pkg-install-server.in	2 Jun 2011 13:46:03 -0000
@@ -3,9 +3,6 @@
 # $FreeBSD: ports/databases/postgresql91-server/files/pkg-install-server.in,v 1.1 2011/04/18 23:34:27 girgen Exp $
 
 PATH=/bin:/usr/bin:/usr/sbin
-PG_USER=%%PG_USER%%
-PG_GROUP=%%PG_GROUP%%
-PG_UID=%%PG_UID%%
 
 backupwarning() { echo "
 
@@ -23,43 +20,6 @@
 }
 
 case $2 in
-PRE-INSTALL)
-	backupwarning
-	PGUSER=${PGUSER:-${PG_USER}}
-	PGGROUP=${PGGROUP:-${PG_GROUP}}
-	DB_DIR=${PKG_PREFIX}/${PGUSER}
-	UID=${PG_UID}
-	GID=${PG_UID}
-
-	if pw group show "${PGGROUP}" 2>/dev/null; then
-		echo "You already have a group \"${PGGROUP}\", so I will use it."
-	else
-		if pw groupadd ${PGGROUP} -g ${GID}; then
-			echo "Added group \"${PGGROUP}\"."
-		else
-			echo "Adding group \"${PGGROUP}\" failed..."
-			exit 1
-		fi
-	fi
-
-	if pw user show "${PGUSER}" 2>/dev/null; then
-		echo "You already have a user \"${PGUSER}\", so I will use it."
-	else
-		if pw useradd ${PGUSER} -u ${UID} -g ${PGGROUP} -h - \
-			-d ${DB_DIR} -c "PostgreSQL Daemon"
-		then
-			echo "Added user \"${PGUSER}\"."
-		else
-			echo "Adding user \"${PGUSER}\" failed..."
-			exit 1
-		fi
-	fi
-
-	if ! [ -x ~${PGUSER} ] ; then 
-		install -m 755 -o ${PGUSER} -g ${PGGROUP} -d ${DB_DIR}
-	fi
-	;;
-
 BACKUPWARNING)
 	backupwarning
 	;;
--- databases-postgresql91-server.patch ends here ---



-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.

>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->girgen 
Responsible-Changed-By: edwin 
Responsible-Changed-When: Mon Jun 6 19:10:22 UTC 2011 
Responsible-Changed-Why:  
Over to maintainer (via the GNATS Auto Assign Tool) 

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

From: dfilter@FreeBSD.ORG (dfilter service)
To: bug-followup@FreeBSD.org
Cc:  
Subject: Re: ports/157667: commit references a PR
Date: Sat, 11 Jun 2011 23:14:45 +0000 (UTC)

 girgen      2011-06-11 23:14:30 UTC
 
   FreeBSD ports repository
 
   Modified files:
     databases            Makefile 
     databases/postgresql91-server Makefile distinfo 
                                   pkg-plist-client 
                                   pkg-plist-contrib 
                                   pkg-plist-server 
     databases/postgresql91-server/files 502.pgsql.in 
                                         pkg-install-server.in 
                                         postgresql.in 
   Added files:
     databases/postgresql91-client Makefile 
     databases/postgresql91-contrib Makefile pkg-descr 
   Log:
   Add postgresql 9.1 beta1.
   
   See http://www.postgresql.org/about/news.1313 for more info.
   
   Also, use USERS knob instead of explicitally creating the pgsql user
   while still accepting alternative names, using [1] with some added
   magic.
   
   PR:     157667 [1]
   
   Revision  Changes    Path
   1.983     +3 -0      ports/databases/Makefile
   1.1       +23 -0     ports/databases/postgresql91-client/Makefile (new)
   1.1       +32 -0     ports/databases/postgresql91-contrib/Makefile (new)
   1.1       +23 -0     ports/databases/postgresql91-contrib/pkg-descr (new)
   1.218     +44 -34    ports/databases/postgresql91-server/Makefile
   1.83      +2 -2      ports/databases/postgresql91-server/distinfo
   1.3       +11 -8     ports/databases/postgresql91-server/files/502.pgsql.in
   1.2       +1 -41     ports/databases/postgresql91-server/files/pkg-install-server.in
   1.6       +2 -2      ports/databases/postgresql91-server/files/postgresql.in
   1.22      +23 -1     ports/databases/postgresql91-server/pkg-plist-client
   1.15      +124 -77   ports/databases/postgresql91-server/pkg-plist-contrib
   1.33      +4 -2      ports/databases/postgresql91-server/pkg-plist-server
 _______________________________________________
 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->closed 
State-Changed-By: girgen 
State-Changed-When: Wed Jul 20 12:10:38 UTC 2011 
State-Changed-Why:  
It is using USERS since beta2. Thanks for pointing it out. 

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