Re: Re: INSTALL file

From: David Steele <david(at)pgmasters(dot)net>
To: Andreas 'ads' Scherbaum <ads(at)pgug(dot)de>
Cc: Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Michael Paquier <michael(at)paquier(dot)xyz>, Stephen Frost <sfrost(at)snowman(dot)net>, Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Re: INSTALL file
Date: 2019-03-21 18:06:36
Message-ID: 3562fd9a-ab3b-cb1c-88a1-ded6d21e441e@pgmasters.net
Views: Raw Message | Whole Thread | Download mbox
Thread:
Lists: pgsql-hackers

Hi Andreas,

On 2/15/19 11:59 AM, Peter Eisentraut wrote:
> On 14/02/2019 20:13, Andres Freund wrote:
>> On 2019-02-04 11:02:44 +0900, Michael Paquier wrote:
>>> +1. I have just looked at the patch, and my take is that listing all
>>> the ways to build Postgres directly in the README is just a
>>> duplication of what we already have in the documentation. So I would
>>> just rip out all that and keep a simple link to the documentation.
>>
>> Well, the documentation cannot be built without the dependencies, and
>> not everyone has convenient internet access. I'm not sure what the
>> solution to that is, but somehow consolidating that information in the,
>> by now pretty standardized, location of INSTALL seems pretty reasonable
>> to me.
>
> (I suppose you meant README here. The information is already in INSTALL.)
>
> But the proposed additions don't actually mention the required
> dependencies to build the documentation, so it wouldn't even achieve
> that goal.
>
> And if you don't have internet access, how did you get the git checkout?
>
> The use case here seems pretty narrow.

Any thoughts on Peter and Andres' comments?

Regards,
--
-David
david(at)pgmasters(dot)net

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2019-03-21 18:15:57 Re: Pluggable Storage - Andres's take
Previous Message David Steele 2019-03-21 18:01:15 Re: Re: [proposal] Add an option for returning SQLSTATE in psql error message