Re: pgsql: Doc: remove some long-obsolete information from installation.sgm

From: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Doc: remove some long-obsolete information from installation.sgm
Date: 2019-09-06 11:53:20
Message-ID: 4b44a874-f3c3-75b4-3caa-d3257608f807@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On 2019-09-01 23:34, Tom Lane wrote:
> Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com> writes:
>>> Doc: remove some long-obsolete information from installation.sgml.
>
>> - It is a fully native build and uses no additional software like
>> - MinGW. A ready-made installer is available on the main
>> - PostgreSQL web site.
>> + The Visual C++ build is recommended, as it is fully native and uses no
>> + additional software like MinGW. A ready-made installer is available on
>> + the main PostgreSQL web site.
>
>> Is this correct? MinGW builds are also native builds. The old wording
>> was shaky but the new wording (inserting "... is recommended, as ...")
>> makes it sound like that there is something wrong with the MinGW build
>> compared to the MSVC build.
>
> So did the old wording, it seemed to me. I'm happy to remove the sentence
> altogether if there's no reason to be pushing people away from MinGW.
> Should we also rethink the sentence I added at the top of the chapter,
> pointing Windows people to chapter 17?

Yeah, I suggest this:

diff --git a/doc/src/sgml/installation.sgml b/doc/src/sgml/installation.sgml
index 1f2f7d9957..6e79133681 100644
--- a/doc/src/sgml/installation.sgml
+++ b/doc/src/sgml/installation.sgml
@@ -21,10 +21,6 @@ <title>Installation from Source Code</title>
distribution. If you are installing a pre-packaged distribution,
such as an RPM or Debian package, ignore this chapter
and read the packager's instructions instead.
- Also, this chapter does not describe the preferred way to install
- <productname>PostgreSQL</productname> on Microsoft
- Windows<phrase condition="standalone-ignore"> (for that, see
- <xref linkend="install-windows"/>)</phrase>.
</para>

<sect1 id="install-short">
@@ -2484,9 +2480,7 @@ <title>MinGW/Native Windows</title>
The MinGW build procedure uses the normal build system described in
this chapter; the Visual C++ build works completely differently
and is described in <xref linkend="install-windows"/>.
- The Visual C++ build is recommended, as it is fully native and uses no
- additional software like MinGW. A ready-made installer is available on
- the main PostgreSQL web site.
+ A ready-made installer is available on the main PostgreSQL web site.
</para>

<para>

--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Robert Haas 2019-09-06 13:11:57 pgsql: When performing a base backup, check for read errors.
Previous Message Peter Eisentraut 2019-09-06 09:47:48 Re: pgsql: Fix compiler warning