diff options
| author | Andrew Gierth <rhodiumtoad@postgresql.org> | 2018-09-07 13:51:30 +0100 |
|---|---|---|
| committer | Andrew Gierth <rhodiumtoad@postgresql.org> | 2018-09-07 14:19:14 +0100 |
| commit | 7b6b167fa3bd2f968ad885ca7b024be4122a85f1 (patch) | |
| tree | bd05c6335a7652c43cb0f88ecfe0279f91e7220a /doc/src | |
| parent | 1fea1e325492b10f645809204631426bb6abbe6e (diff) | |
Refactor installation of extension headers.
Commit be54b3777 failed on gmake 3.80 due to a chained conditional,
which on closer examination could be removed entirely with some
refactoring elsewhere for a net simplification and more robustness
against empty expansions. Along the way, add some more comments.
Also make explicit in the documentation and comments that built
headers are not removed by 'make clean', since we don't typically want
that for headers generated by a separate ./configure step, and it's
much easier to add your own 'distclean' rule or use EXTRA_CLEAN than
to try and override a deletion rule in pgxs.mk.
Per buildfarm member prariedog and comments by Michael Paquier, though
all the actual changes are my fault.
Diffstat (limited to 'doc/src')
| -rw-r--r-- | doc/src/sgml/extend.sgml | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/doc/src/sgml/extend.sgml b/doc/src/sgml/extend.sgml index 1b1adae1a63..695e07fb384 100644 --- a/doc/src/sgml/extend.sgml +++ b/doc/src/sgml/extend.sgml @@ -1227,8 +1227,13 @@ include $(PGXS) <term><varname>HEADERS_built</varname></term> <listitem> <para> - files to (optionally build and) install under - <literal><replaceable>prefix</replaceable>/include/server/$MODULEDIR/$MODULE_big</literal> + Files to (optionally build and) install under + <literal><replaceable>prefix</replaceable>/include/server/$MODULEDIR/$MODULE_big</literal>. + </para> + <para> + Unlike <literal>DATA_built</literal>, files in <literal>HEADERS_built</literal> + are not removed by the <literal>clean</literal> target; if you want them removed, + also add them to <literal>EXTRA_CLEAN</literal> or add your own rules to do it. </para> </listitem> </varlistentry> @@ -1244,6 +1249,11 @@ include $(PGXS) in <literal>MODULES</literal> or <literal>MODULE_big</literal>. </para> <para> + Unlike <literal>DATA_built</literal>, files in <literal>HEADERS_built_$MODULE</literal> + are not removed by the <literal>clean</literal> target; if you want them removed, + also add them to <literal>EXTRA_CLEAN</literal> or add your own rules to do it. + </para> + <para> It is legal to use both variables for the same module, or any combination, unless you have two module names in the <literal>MODULES</literal> list that differ only by the presence of a |