From: | Peter Eisentraut <peter(at)eisentraut(dot)org> |
---|---|
To: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Put genbki.pl output into src/include/catalog/ directly |
Date: | 2024-02-08 07:58:53 |
Message-ID: | 21b74bdc-183d-4dd5-9c27-9378d178f459@eisentraut.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
With the makefile rules, the output of genbki.pl was written to
src/backend/catalog/, and then the header files were linked to
src/include/catalog/.
This patch changes it so that the output files are written directly to
src/include/catalog/. This makes the logic simpler, and it also makes
the behavior consistent with the meson build system. For example, a
file like schemapg.h is now mentioned only in
src/include/catalog/{meson.build,Makefile,.gitignore}
where before it was mentioned in (checks ...)
src/backend/catalog/.gitignore
src/backend/catalog/Makefile
src/include/Makefile
src/include/catalog/.gitignore
src/include/catalog/meson.build
Also, the list of catalog files is now kept in parallel in
src/include/catalog/{meson.build,Makefile}, while before the makefiles
had it in src/backend/catalog/Makefile.
I think keeping the two build systems aligned this way will be useful
for longer-term maintenance.
(There are other generated header files that are linked in a similar way
and could perhaps be simplified. But they don't all work the same way.
Some of the scripts also generate .c files, for example, so they need to
put some stuff under src/backend/. So I restricted this patch to
src/{backend,include}/catalog/, especially because it would be good to
keep the catalog lists aligned.)
Attachment | Content-Type | Size |
---|---|---|
0001-Put-genbki.pl-output-into-src-include-catalog-direct.patch | text/plain | 15.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2024-02-08 08:06:39 | Re: Testing autovacuum wraparound (including failsafe) |
Previous Message | Michael Paquier | 2024-02-08 07:39:23 | Re: A comment in DropRole() contradicts the actual behavior |