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.
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 |