BUG #3879: OS X Start Script should not `cd /Users/postgres`

From: "David Wheeler" <david(at)kineticode(dot)com>
To: pgsql-bugs(at)postgresql(dot)org
Subject: BUG #3879: OS X Start Script should not `cd /Users/postgres`
Date: 2008-01-16 00:28:04
Message-ID: 200801160028.m0G0S41G086834@wwwmaster.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs


The following bug has been logged online:

Bug reference: 3879
Logged by: David Wheeler
Email address: david(at)kineticode(dot)com
PostgreSQL version: 8.3RC1
Operating system: Mac OS X 10.5.1
Description: OS X Start Script should not `cd /Users/postgres`
Details:

The Mac OS X start script (contrib/start-scripts/osx/PostgreSQL) should not
have the recently-added lines `cd /Users/postgres` for the following
reasons:

* Not all systems use that username.
* Even those that use the "postgres" username may not have a home
directory.
* changing to the home directory appears to have no effect whatsoever.

Here is the patch:

--- contrib/start-scripts/osx/PostgreSQL.orig 2008-01-15 16:24:54.000000000
-0800
+++ contrib/start-scripts/osx/PostgreSQL 2008-01-15 16:25:11.000000000
-0800
@@ -84,7 +84,6 @@
StartService () {
if [ "${POSTGRESQL:=-NO-}" = "-YES-" ]; then
ConsoleMessage "Starting PostgreSQL database server"
- cd /Users/postgres
if [ "${ROTATELOGS}" = "1" ]; then
sudo -u $PGUSER sh -c "${DAEMON} -D '${PGDATA}' 2>&1 |
${LOGUTIL} '${PGLOG}' ${ROTATESEC} &"
else
@@ -95,7 +94,6 @@

StopService () {
ConsoleMessage "Stopping PostgreSQL database server"
- cd /Users/postgres
sudo -u $PGUSER $PGCTL stop -D "$PGDATA" -s -m fast
}

@@ -103,7 +101,6 @@
if [ "${POSTGRESQL:=-NO-}" = "-YES-" ]; then
ConsoleMessage "Restarting PostgreSQL database server"
# should match StopService:
- cd /Users/postgres
sudo -u $PGUSER $PGCTL stop -D "$PGDATA" -s -m fast
# should match StartService:
if [ "${ROTATELOGS}" = "1" ]; then

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Tomasz Ostrowski 2008-01-16 13:43:29 Re: BUG #3872: backslash_quote = off doesn't work any more
Previous Message Guillaume Smet 2008-01-15 22:05:11 Re: BUG #3876: Problems migrating databases