This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Test for profiling support (_mcount/gprof)
- From: Florian Weimer <fweimer at redhat dot com>
- To: Andreas Schwab <schwab at suse dot de>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Tue, 15 Aug 2017 14:45:11 +0200
- Subject: Re: [PATCH] Test for profiling support (_mcount/gprof)
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx08.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx08.extmail.prod.ext.phx2.redhat.com; spf=fail smtp.mailfrom=fweimer at redhat dot com
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com B632DC053FA3
- References: <4d15086e-9e54-014b-1a50-16534f287e24@redhat.com> <mvm4lt9ui9h.fsf@suse.de>
On 08/15/2017 02:28 PM, Andreas Schwab wrote:
> On Aug 15 2017, Florian Weimer <fweimer@redhat.com> wrote:
>
>> diff --git a/Makeconfig b/Makeconfig
>> index 80aed2a987..19b559701a 100644
>> --- a/Makeconfig
>> +++ b/Makeconfig
>> @@ -465,7 +465,8 @@ else # not build-pie-default
>> +link-before-libc = $(CC) -nostdlib -nostartfiles -o $@ \
>> $(sysdep-LDFLAGS) $(LDFLAGS) $(LDFLAGS-$(@F)) \
>> $(combreloc-LDFLAGS) $(relro-LDFLAGS) $(hashstyle-LDFLAGS) \
>> - $(addprefix $(csu-objpfx),$(start-installed-name)) \
>> + $(or $(CRT-$(@F)), $ \
>> + $(addprefix $(csu-objpfx),$(start-installed-name))) \
>
> $(or) is a 3.81 feature, we only require 3.79. You can use $(firstword)
> instead.
Oh, good point. $(firstword …) does not really align with the
subsequent $(addprefix …) usage because that has the implication that
$(start-installed-name) contains multiple words. We can turn
$(addprefix $(csu-objpfx),$(start-installed-name))
into
$(csu-objpfx)$(start-installed-name)
though; it should work on all in-tree architectures. Then $(firstword
…) should have the right semantics.
But can we require make 3.81 instead? It was released in April 2006,
after all. That does not mean we'll start using controversial 3.81
features such as $(eval …).
> The lone $ looks like a typo.
Right. Fixed.
Thanks,
Florian