Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*

Поиск
Список
Период
Сортировка
От Andres Freund
Тема Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*
Дата
Msg-id 20231201221255.d7meekaycfrncorl@awork3.anarazel.de
обсуждение исходный текст
Ответ на Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*  ("Tristan Partin" <tristan@neon.tech>)
Список pgsql-hackers
Hi,

On 2023-12-01 15:55:29 -0600, Tristan Partin wrote:
> Commits look fine to me, but I hate the new target names...

You shouldn't ever need to use them anywhere - that's what the alias is for...

Happy to go another route if you have a suggestion.


> >  +    for name, v in targets_info_byname.items():
> >  +        if len(targets_info_byname[name]) > 1:
> 
> My only comment is that you could reverse the logic and save yourself an
> indentation.
> 
> - if len(targets_info_byname[name]) > 1:
> + if len(targets_info_byname[name]) <= 1:
> +     continue
> 
> But whatever you want.

Makes sense.



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

Предыдущее
От: "Tristan Partin"
Дата:
Сообщение: Re: Clean up some signal usage mainly related to Windows
Следующее
От: Sacha Hottinger
Дата:
Сообщение: AW: Building PosgresSQL with LLVM fails on Solaris 11.4