Greg Stark <st...@mit.edu> writes: > On 12 Mar 2016 10:58 pm, "Tom Lane" <t...@sss.pgh.pa.us> wrote: >> I shall get rid of the const-ness, as well as the lame casting away >> of it, and I think I will also go make buildfarm member longfin use >> "-fno-common". It is truly sad that we apparently have no test >> machine that enforces that const means const ...
> It looks like this would also be useful for catching any mistakes where we > might have defined a variable in two different files or missed an "extern". > Though I'm not sure the failures will be very obvious. The answer is that a failure looks like this: duplicate symbol _FeBeWaitSet in: access/common/printtup.o access/transam/parallel.o duplicate symbol _FeBeWaitSet in: access/common/printtup.o commands/async.o duplicate symbol _FeBeWaitSet in: access/common/printtup.o commands/copy.o duplicate symbol _FeBeWaitSet in: access/common/printtup.o libpq/be-secure.o duplicate symbol _FeBeWaitSet in: access/common/printtup.o libpq/auth.o duplicate symbol _FeBeWaitSet in: access/common/printtup.o libpq/hba.o duplicate symbol _FeBeWaitSet in: access/common/printtup.o libpq/pqcomm.o ... etc etc ... See: http://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=longfin&dt=2016-03-21%2011%3A49%3A13 http://git.postgresql.org/gitweb/?p=postgresql.git;a=commitdiff;h=7fa0064092e135415a558dc3c4d7393d14ab6d8e regards, tom lane -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers