On 19 January 2012 18:41, Sunpoet Po-Chuan Hsieh <[email protected]> wrote:
> On Wed, Jan 18, 2012 at 09:35:11PM +0000, Jason Helfman wrote:
>> jgh 2012-01-18 21:35:11 UTC
>>
>> FreeBSD ports repository
>>
>> Modified files:
>> databases/postgresql82-server Makefile pkg-plist-server
>> databases/postgresql84-server Makefile pkg-plist-server
>> databases/postgresql90-server Makefile pkg-plist-server
>> databases/postgresql91-server Makefile pkg-plist-server
>> Log:
>> Fix plist to create directory with proper ownerships PostgreSQL database
>> may start.
>>
>> PR: ports/164273 (critical)
>> Submitted by: Alexander Yerenkow <yerenkow at gmail.com>
>> Approved by: maintainer-timeout: girgen (1 day), portmgr (linimon)
>>
>> Revision Changes Path
>> 1.207 +2 -1 ports/databases/postgresql82-server/Makefile
>> 1.23 +2 -0 ports/databases/postgresql82-server/pkg-plist-server
>> 1.240 +2 -0 ports/databases/postgresql84-server/Makefile
>> 1.37 +2 -0 ports/databases/postgresql84-server/pkg-plist-server
>> 1.233 +2 -0 ports/databases/postgresql90-server/Makefile
>> 1.37 +2 -0 ports/databases/postgresql90-server/pkg-plist-server
>> 1.235 +2 -0 ports/databases/postgresql91-server/Makefile
>> 1.40 +2 -0 ports/databases/postgresql91-server/pkg-plist-server
>> _______________________________________________
>> [email protected] mailing list
>> http://lists.freebsd.org/mailman/listinfo/cvs-ports
>> To unsubscribe, send any mail to "[email protected]"
>
> Hi,
>
> The patch is OK except one thing. I think that directory should be
> ${PREFIX}/${PG_USER}, not ${LOCALBASE}/${PG_USER}.
> Thanks.
You are correct, and I'm sorry I missed that.
For those who are curious, Sunpoet is referring to the line in bsd.port.mk:
homedir=$$(echo $$homedir | sed "s|^/usr/local|${PREFIX}|"); \
Chris
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "[email protected]"
