diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-05-07 13:44:09 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-05-07 13:44:09 -0400 |
| commit | 513ff52e81675f26ceb91e8301019546439d73c9 (patch) | |
| tree | 004397a3b54561b06d6fcb696b10b99c4fa0f664 | |
| parent | f34f0e4c58a31e5edd3aa8a23e171fbcf7e01ff2 (diff) | |
| download | postgresql-513ff52e81675f26ceb91e8301019546439d73c9.tar.gz postgresql-513ff52e81675f26ceb91e8301019546439d73c9.zip | |
Suppress compiler warnings when building with --enable-dtrace.
Most versions of "dtrace -h" drop const qualifiers from the declarations
of probe functions (though macOS gets it right). This causes compiler
warnings when we pass in pointers to const. Repair by extending our
existing post-processing of the probes.h file. To do so, assume that all
"char *" arguments should be "const char *"; that seems reasonably safe.
Thomas Munro
Discussion: https://postgr.es/m/CAEepm=2j1pWSruQJqJ91ZDzD8w9ZZDsM4j2C6x75C-VryWg-_w@mail.gmail.com
| -rw-r--r-- | src/backend/utils/Makefile | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/backend/utils/Makefile b/src/backend/utils/Makefile index 966e3bc2ed3..e797539d09c 100644 --- a/src/backend/utils/Makefile +++ b/src/backend/utils/Makefile @@ -49,10 +49,15 @@ ifneq ($(enable_dtrace), yes) probes.h: Gen_dummy_probes.sed endif +# We editorialize on dtrace's output to the extent of changing the macro +# names (from POSTGRESQL_foo to TRACE_POSTGRESQL_foo) and changing any +# "char *" arguments to "const char *". probes.h: probes.d ifeq ($(enable_dtrace), yes) $(DTRACE) -C -h -s $< -o $@.tmp - sed -e 's/POSTGRESQL_/TRACE_POSTGRESQL_/g' $@.tmp >$@ + sed -e 's/POSTGRESQL_/TRACE_POSTGRESQL_/g' \ + -e 's/( *char \*/(const char */g' \ + -e 's/, *char \*/, const char */g' $@.tmp >$@ rm $@.tmp else sed -f $(srcdir)/Gen_dummy_probes.sed $< >$@ |