Re: code coverage patch

From: Peter Eisentraut <peter_e(at)gmx(dot)net>
To: Gregory Stark <stark(at)enterprisedb(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org, Michelle Caisse <Michelle(dot)Caisse(at)sun(dot)com>
Subject: Re: code coverage patch
Date: 2008-08-28 13:32:45
Message-ID: 48B6A8FD.6020402@gmx.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Gregory Stark wrote:
> Peter Eisentraut <peter_e(at)gmx(dot)net> writes:
>
>> The reason for that problem is that the shared object needs to be linked
>> with -fprofile-arcs -ftest-coverage. (One of these causes -lgcov to be
>> linked, which includes the missing symbol.) This is not done because the
>> shared object link rules don't use CFLAGS.
>
> Shared object link rules should use another variable (LDFLAGS?) and those
> options should be added that variable as well.

When linking executables, we already use both CFLAGS and LDFLAGS. This
is the standard way in the GNU-enabled world. And it does exactly the
right thing in this gcov case. If we invented another variable, we
would disrupt that system and would further differentiate between
different types of linking, while we should ultimately aim to make it
less different.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2008-08-28 14:00:21 Re: code coverage patch
Previous Message Korry Douglas 2008-08-28 13:08:46 Re: code coverage patch