Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 20 Jan 2012 02:41:46 +0800
From:      Sunpoet Po-Chuan Hsieh <sunpoet@FreeBSD.org>
To:        Jason Helfman <jgh@FreeBSD.org>
Cc:        cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org
Subject:   Re: cvs commit: ports/databases/postgresql82-server Makefile pkg-plist-server ports/databases/postgresql84-server Makefile pkg-plist-server ports/databases/postgresql90-server Makefile pkg-plist-server ports/databases/postgresql91-server Makefile pkg-plist-server
Message-ID:  <20120119184146.GB85006@bonjour.sunpoet.net>
In-Reply-To: <201201182135.q0ILZBXQ073169@repoman.freebsd.org>
References:  <201201182135.q0ILZBXQ073169@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--X1bOJ3K7DJ5YkBrT
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jan 18, 2012 at 09:35:11PM +0000, Jason Helfman wrote:
> jgh         2012-01-18 21:35:11 UTC
>=20
>   FreeBSD ports repository
>=20
>   Modified files:
>     databases/postgresql82-server Makefile pkg-plist-server=20
>     databases/postgresql84-server Makefile pkg-plist-server=20
>     databases/postgresql90-server Makefile pkg-plist-server=20
>     databases/postgresql91-server Makefile pkg-plist-server=20
>   Log:
>   Fix plist to create directory with proper ownerships PostgreSQL database
>   may start.
>  =20
>   PR:     ports/164273 (critical)
>   Submitted by: Alexander Yerenkow <yerenkow at gmail.com>
>   Approved by:    maintainer-timeout: girgen (1 day), portmgr (linimon)
>  =20
>   Revision  Changes    Path
>   1.207     +2 -1      ports/databases/postgresql82-server/Makefile
>   1.23      +2 -0      ports/databases/postgresql82-server/pkg-plist-serv=
er
>   1.240     +2 -0      ports/databases/postgresql84-server/Makefile
>   1.37      +2 -0      ports/databases/postgresql84-server/pkg-plist-serv=
er
>   1.233     +2 -0      ports/databases/postgresql90-server/Makefile
>   1.37      +2 -0      ports/databases/postgresql90-server/pkg-plist-serv=
er
>   1.235     +2 -0      ports/databases/postgresql91-server/Makefile
>   1.40      +2 -0      ports/databases/postgresql91-server/pkg-plist-serv=
er
> _______________________________________________
> cvs-ports@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/cvs-ports
> To unsubscribe, send any mail to "cvs-ports-unsubscribe@freebsd.org"

Hi,

The patch is OK except one thing. I think that directory should be
${PREFIX}/${PG_USER}, not ${LOCALBASE}/${PG_USER}.
Thanks.

Regards,
sunpoet

--X1bOJ3K7DJ5YkBrT
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.18 (FreeBSD)

iQIcBAEBAgAGBQJPGGPpAAoJENxEJH7MV+NrmZwP/1UFO88ztRMAqwzMEuJFgGwA
YfDGysq2DFifQoqgpBpv4jMdaM8Hq9p/KlzZCfqrbWGoWyo8QRhHsFMnuMqNTO7U
mUm4RfhNRwztHaYX1c7Py1vY+R+gOPb0a4iqI5LGzi/57/ZLH6QAZhTbXmdno6wQ
AIsBz+OR2cZmO8xBKRwtAcuMdBtP+kcTvpBuvMdZfKOUxP7VV8mwrubyy2rk0CEk
wycUDHa1st/cy7gnTcs7wwwfEZBAYL/sFy0pgbFRQvP2s/H2k/2jP2d7oSTA9d9w
y0WtMCUDRkAT9UvqVfjgnhXA5UBzb16U+iswpyB/FUJ1Jzq6JUSv44ZlLuD/sD2l
vNXPmlcKUKCfalxohBv3Wiz8XGIvRt3W3xEq+cyEXPPm/cm5rX/hedle2/woyHef
wPMHYnCj/8PHS/FnQavrEBnkQipXJA8uqYQ0HG35vmGokmvhozCMckx1UavB6nKp
PCY+IgyTDxpKgxXUQ9t4LaN7NZtg23VtY0BWjIzYKF81tt+bO0des2vHvjmdTsFE
ykR0CewWsTz9ayObFxsa27im0pd5wqKPe9BxvIAst+BhUQTPkFE1bErdx7tFM5/R
1Tz2tq0SMVtaj2XxcAP3TTStQ9MI001dyqlsNDbbQK8lw8ECHpmh7LzRJQQPf7/B
Q2ckrsyS4P/Hva7EXQlw
=CcGk
-----END PGP SIGNATURE-----

--X1bOJ3K7DJ5YkBrT--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120119184146.GB85006>