Re: Where can I find the doxyfile?

From: John Morris <john(dot)morris(at)crunchydata(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Peter Eisentraut <peter(at)eisentraut(dot)org>, vignesh C <vignesh21(at)gmail(dot)com>, Bohdan Mart <mart(dot)bogdan(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>, Bruce Momjian <bruce(at)momjian(dot)us>, "postgres(at)coyotebush(dot)net" <postgres(at)coyotebush(dot)net>, Stefan Kaltenbrunner <stefan(at)kaltenbrunner(dot)cc>
Subject: Re: Where can I find the doxyfile?
Date: 2024-02-09 00:43:52
Message-ID: CYXP222MB09477D80058BE7A1EF88BA4AA0442@CYXP222MB0947.NAMP222.PROD.OUTLOOK.COM
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

>> We also have a handful of .cpp files.
Fortunately, our .cpp files are simple and do not include C++ specific features.
We shouldn’t have any problem adding .cpp to the filter list.

At some point it will be necessary to support general C++, but I wasn’t planning to do it yet.

>> This file isn't really following postgres coding style - any reason not to do so?
I’ll update it to Postgres commenting. The filter started as a standalone project.

>> What do you mean with "build time error messages" specifically? Why would we
>> want to output anything at build time (rather than configure time)?
“build time error messages” occur after typing “ninja”.

Documentation is usually produced long after meson setup messages have been forgotten.
It is much more useful to see “Please install the doxygen program” after typing “ninja doxygen”
than to see “target not supported”, or to have to look through the setup log.

It’s personal preference, but I think the sgml docs should display a similar message.

>> I'd add "native: true", it'd not work to find the program in the target
>> environment of a cross build.
Got it. Thanks.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2024-02-09 00:54:53 Re: Make COPY format extendable: Extract COPY TO format implementations
Previous Message Andres Freund 2024-02-09 00:43:07 Re: confusing / inefficient "need_transcoding" handling in copy