Re: pgsql: Mention "pg_ctl stop" in pgindent README instructions.

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Mention "pg_ctl stop" in pgindent README instructions.
Date: 2011-06-10 05:55:29
Message-ID: 4DF1B1D1.4040007@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On 06/09/2011 11:28 PM, Bruce Momjian wrote:
> Andrew Dunstan wrote:
>>
>>
>>
>> On 06/09/2011 08:51 PM, Bruce Momjian wrote:
>>> Mention "pg_ctl stop" in pgindent README instructions.
>>>
>> This change make no sense at all. Why would you be running 'pg_ctl stop'
>> when there is nothing to suggest that there's a postgres running. And
>> even if there is there's not the slightest reason for stopping it unless
>> it's in the same location you're installing to. Certainly having this
>> line in the README with no comment about when you might want it is going
>> to be highly confusing at best. I think you need to rethink this.
> Well, I have a Postgres running, and it the install goes into the same
> directory, so my regression tests failed, so I documented the command to
> shut down the postmaster.

Then the answer is "don't do that." Surely it's far saner to run
configure --prefix=/path/to/temploc and then go from there. Disturbing a
running postmaster for this purpose is crazy. BTW, for years I've used a
gadget to set up every piece of development work with its own install
location and port so that this sort of problem doesn't happen. It's
fairly trivial but I'll gladly share it with you. I rather imagined
every serious postgres developer would have such a thing.

cheers

andrew

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2011-06-10 14:18:31 Re: pgsql: Mention "pg_ctl stop" in pgindent README instructions.
Previous Message Bruce Momjian 2011-06-10 03:28:24 Re: pgsql: Mention "pg_ctl stop" in pgindent README instructions.