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

[Chris Rees]

On 19 January 2012 18:41, Sunpoet Po-Chuan Hsieh <sunpoet at freebsd.org> 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
>> _______________________________________________
>> cvs-ports at freebsd.org mailing list
>> http://lists.freebsd.org/mailman/listinfo/cvs-ports
>> To unsubscribe, send any mail to "cvs-ports-unsubscribe at freebsd.org"
>
> 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