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

2023-12-01 Thread Andrew Dunstan
On 2023-12-01 Fr 09:04, Andrew Dunstan wrote: On 2023-11-29 We 07:20, Andrew Dunstan wrote: On 2023-11-28 Tu 21:28, Andres Freund wrote: Hi, On 2023-11-23 08:32:21 -0500, Andrew Dunstan wrote: On 2023-11-20 Mo 20:53, Andres Freund wrote: meson: docs: Add {html,man} targets, rename

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

2023-12-01 Thread Andres Freund
Hi, On 2023-12-01 09:04:19 -0500, Andrew Dunstan wrote: > On 2023-11-29 We 07:20, Andrew Dunstan wrote: > > On 2023-11-28 Tu 21:28, Andres Freund wrote: > > > On 2023-11-23 08:32:21 -0500, Andrew Dunstan wrote: > > > > On 2023-11-20 Mo 20:53, Andres Freund wrote: > > > > > meson: docs: Add

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

2023-12-01 Thread Andrew Dunstan
On 2023-11-29 We 07:20, Andrew Dunstan wrote: On 2023-11-28 Tu 21:28, Andres Freund wrote: Hi, On 2023-11-23 08:32:21 -0500, Andrew Dunstan wrote: On 2023-11-20 Mo 20:53, Andres Freund wrote: meson: docs: Add {html,man} targets, rename install-doc-* We have toplevel html, man targets in

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

2023-11-29 Thread Andres Freund
Hi, On 2023-11-29 10:05:26 -0500, Andrew Dunstan wrote: > On 2023-11-29 We 08:49, Tom Lane wrote: > > Andrew Dunstan writes: > > > On 2023-11-28 Tu 21:28, Andres Freund wrote: > > > > I don't really understand why meson compile complains in this case. I > > > > assume > > > > you don't want to

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

2023-11-29 Thread Andrew Dunstan
On 2023-11-29 We 08:49, Tom Lane wrote: Andrew Dunstan writes: On 2023-11-28 Tu 21:28, Andres Freund wrote: I don't really understand why meson compile complains in this case. I assume you don't want to disambiguate as suggested, by building html:alias instead? I've done that as a

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

2023-11-29 Thread Tom Lane
Andrew Dunstan writes: > On 2023-11-28 Tu 21:28, Andres Freund wrote: >> I don't really understand why meson compile complains in this case. I assume >> you don't want to disambiguate as suggested, by building html:alias instead? > I've done that as a temporary fix to get crake out of the hole,

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

2023-11-29 Thread Andrew Dunstan
On 2023-11-28 Tu 21:28, Andres Freund wrote: Hi, On 2023-11-23 08:32:21 -0500, Andrew Dunstan wrote: On 2023-11-20 Mo 20:53, Andres Freund wrote: meson: docs: Add {html,man} targets, rename install-doc-* We have toplevel html, man targets in the autoconf build as well. It'd be odd to have

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

2023-11-28 Thread Andres Freund
Hi, On 2023-11-23 08:32:21 -0500, Andrew Dunstan wrote: > On 2023-11-20 Mo 20:53, Andres Freund wrote: > > meson: docs: Add {html,man} targets, rename install-doc-* > > > > We have toplevel html, man targets in the autoconf build as well. It'd be > > odd > > to have an 'html' target but have the

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

2023-11-27 Thread Andrew Dunstan
On 2023-11-23 Th 08:32, Andrew Dunstan wrote: On 2023-11-20 Mo 20:53, Andres Freund wrote: meson: docs: Add {html,man} targets, rename install-doc-* We have toplevel html, man targets in the autoconf build as well. It'd be odd to have an 'html' target but have the install target be

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

2023-11-23 Thread Andrew Dunstan
On 2023-11-20 Mo 20:53, Andres Freund wrote: meson: docs: Add {html,man} targets, rename install-doc-* We have toplevel html, man targets in the autoconf build as well. It'd be odd to have an 'html' target but have the install target be 'install-doc-html', thus rename the install targets to

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

2023-11-20 Thread Andres Freund
meson: docs: Add {html,man} targets, rename install-doc-* We have toplevel html, man targets in the autoconf build as well. It'd be odd to have an 'html' target but have the install target be 'install-doc-html', thus rename the install targets to match. Reviewed-by: Christoph Berg Reviewed-by: