Re: New developer papercut - Makefile references INSTALL

Поиск
Список
Период
Сортировка
От Josef Šimánek
Тема Re: New developer papercut - Makefile references INSTALL
Дата
Msg-id CAFp7QwpnFEf4Kdy8pvXBECcPMHoYLsbKKx-72H0NuhhPMLi9dQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: New developer papercut - Makefile references INSTALL  (samay sharma <smilingsamay@gmail.com>)
Ответы Re: New developer papercut - Makefile references INSTALL  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
pá 21. 1. 2022 v 1:29 odesílatel samay sharma <smilingsamay@gmail.com> napsal:
>
>
>
> On Wed, Jan 19, 2022 at 4:58 PM Tim McNamara <tim@mcnamara.nz> wrote:
>>
>> Hello,
>>
>> I encountered a minor road bump when checking out the pg source today. The Makefile's all target includes the
followinghelp message if GNUmakefile isn't available: 
>>
>>   echo "You need to run the 'configure' program first. See the file"; \
>>   echo "'INSTALL' for installation instructions." ; \
>>
>> After consulting README.git, it looks as though INSTALL isn't created unless the source is bundled into a release or
snapshottarball. I'm happy to submit a patch to update the wording, but wanted to check on the preferred approach. 
>>
>> Perhaps this would be sufficient?
>>
>>   echo "You need to run the 'configure' program first. See the file"; \
>>   echo "'INSTALL' for installation instructions, or visit" ; \
>>   echo "<https://www.postgresql.org/docs/devel/installation.html>" ; \
>>
>> -Tim
>
>
> I noticed a similar thing in the README of the github repository. It asks to see the INSTALL file for build and
installationinstructions but I couldn't find that file and that confused me. This might confuse other new developers as
well.So, maybe we should update the text in the README too? 

There is README.git explaining this. README itself is meant to be used
for distributed source code. You can generate INSTALL locally for
example by running make dist (INSTALL will be present in
postgresql-15devel directory).

Anyway I do agree this is confusing. Maybe we can actually rename
README.git to README and current README to README.dist or similar.
README.dist can be copied to distribution package as README during
Makefile magic.

I can try to provide a patch if welcomed.

> Regards,
> Samay



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Michael Paquier
Дата:
Сообщение: Re: Extend compatibility of PostgreSQL::Test::Cluster
Следующее
От: Etsuro Fujita
Дата:
Сообщение: Re: postgres_fdw: incomplete subabort cleanup of connections used in async execution