Re: Installation instructions update (pg_ctl)

From: Andreas 'ads' Scherbaum <ads(at)pgug(dot)de>
To: Michael Banck <michael(dot)banck(at)credativ(dot)de>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Installation instructions update (pg_ctl)
Date: 2018-10-30 13:52:22
Message-ID: f4f21f14-7523-8be1-37ee-e7c2478a696d@pgug.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 30.10.18 09:04, Michael Banck wrote:
> Hi,
>
> On Tue, Oct 30, 2018 at 12:08:49AM +0100, Andreas 'ads' Scherbaum wrote:
>> The installation instructions (short version) are not consistent with the
>> "initdb" output. The first one still uses "postgres -D", even mentions
>> "check initdb output", but "initdb" emits "pg_ctl" commands.
>>
>> The attached patch updates the short install instructions and replaces
>> "postgres -D" with "pg_ctl" calls.
> Check.
>
>> diff --git a/doc/src/sgml/standalone-install.xml b/doc/src/sgml/standalone-install.xml
>> index 62582effed..b5d2835a47 100644
>> --- a/doc/src/sgml/standalone-install.xml
>> +++ b/doc/src/sgml/standalone-install.xml
>> @@ -50,7 +50,7 @@ in the stand-alone version.
> That one though seems to be what's ending up in INSTALL, according to
> the comment at the beginning of the file.
>
>> <screen>root# <userinput>mkdir /usr/local/pgsql/data</userinput>
>> root# <userinput>chown postgres /usr/local/pgsql/data</userinput>
>> root# <userinput>su - postgres</userinput>
>> -postgres$ <userinput>/usr/local/pgsql/bin/initdb -D /usr/local/pgsql/data</userinput></screen>
>> +postgres$ <userinput>/usr/local/pgsql/bin/pg_ctl -D /usr/local/pgsql/data</userinput></screen>
>> </para>
> I'm confused here, the paragraph reads "Create a database installation
> with the <command>initdb</command>", so I think this hunk is not
> correct and should be removed?

That's indeed one replacement too much.

The attached patch is fixing this.

>> @@ -77,25 +77,21 @@ postgres$ <userinput>/usr/local/pgsql/bin/initdb -D /usr/local/pgsql/data</useri
>> The previous <command>initdb</command> step should have told you how to
>> start up the database server. Do so now. The command should look
>> something like:
>> -<programlisting>/usr/local/pgsql/bin/postgres -D /usr/local/pgsql/data</programlisting>
>> - This will start the server in the foreground. To put the server
>> - in the background use something like:
>> -<programlisting>nohup /usr/local/pgsql/bin/postgres -D /usr/local/pgsql/data \
>> - &lt;/dev/null &gt;&gt;server.log 2&gt;&amp;1 &lt;/dev/null &amp;</programlisting>
>> +<programlisting>/usr/local/pgsql/bin/pg_ctl -D /usr/local/pgsql/data start</programlisting>
>> </para>
>>
>> <para>
>> To stop a server running in the background you can type:
>> -<programlisting>kill `cat /usr/local/pgsql/data/postmaster.pid`</programlisting>
>> +<programlisting>/usr/local/pgsql/bin/pg_ctl -D /usr/local/pgsql/data stop</programlisting>
>> </para>
>> </step>
>>
>> <step>
>> <para>
>> Create a database:
>> -<screen><userinput>createdb testdb</userinput></screen>
>> +<screen><userinput>/usr/local/pgsql/bin/createdb testdb</userinput></screen>
>> Then enter:
>> -<screen><userinput>psql testdb</userinput></screen>
>> +<screen><userinput>/usr/local/pgsql/bin/psql testdb</userinput></screen>
>> to connect to that database. At the prompt you can enter SQL
>> commands and start experimenting.
>> </para>
> Check.

Thank you for the review!

--
Andreas 'ads' Scherbaum
German PostgreSQL User Group
European PostgreSQL User Group - Board of Directors
Volunteer Regional Contact, Germany - PostgreSQL Project

Attachment Content-Type Size
pg_ctl.diff text/x-patch 2.2 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2018-10-30 14:08:34 Re: Super PathKeys (Allowing sort order through precision loss functions)
Previous Message Michael Paquier 2018-10-30 13:48:55 Re: ToDo: show size of partitioned table