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

Поиск
Список
Период
Сортировка
От Tristan Partin
Тема Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*
Дата
Msg-id CXDBYZZR3T3Q.3CQUPEYXLP4NL@neon.tech
обсуждение исходный текст
Ответ на Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*  (Andres Freund <andres@anarazel.de>)
Ответы Re: pgsql: meson: docs: Add {html,man} targets, rename install-doc-*  (Andres Freund <andres@anarazel.de>)
Список pgsql-hackers
Commits look fine to me, but I hate the new target names... Luckily,
I just use plain ninja, so I don't interact with that.

>  +    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.

--
Tristan Partin
Neon (https://neon.tech)



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

Предыдущее
От: Nathan Bossart
Дата:
Сообщение: Re: [PATCH] pg_convert improvement
Следующее
От: "Tristan Partin"
Дата:
Сообщение: Re: [RFC] Clang plugin for catching suspicious typedef casting