[PATCH v3 1/8] tests: Allow tests to append tunables

Yury Khrustalev yury.khrustalev@arm.com
Mon Apr 20 13:52:46 GMT 2026


On Thu, Apr 16, 2026 at 04:16:43PM +0100, Yury Khrustalev wrote:
> On Thu, Apr 16, 2026 at 04:44:10PM +0200, Andreas Schwab wrote:
> > On Apr 16 2026, Yury Khrustalev wrote:
> > 
> > > +# Concatenate elements in the list using colon.
> > > +# Two spaces after 'subst' are intentional.
> > > +emptystring :=
> > > +spacestring := $(emptystring) # one space
> > > +define join-with-col
> > > +$(subst $(spacestring),:,$(strip $(1)))
> > > +endef
> > 
> > We already use $(subst $(empty) ,:,...) in several places.
> > 
> 
> There are indeed a few examples of this, thanks for pointing it out.
> 
> We could perhaps introduce
> 
>   define colon-join
>   $(subst $(empty) ,:,$(strip $(1)))
>   endef
> 
> and replace all of them with
> 
>   $(call colon-join,$(FOO))
> 
> which will be shorter than
> 
>   $(subst $(empty) ,:,$(strip $(FOO)))
> 
> I can add this change as a patch to this patch series.
> I'm open to suggestions for a better name for this function.
>

Actually, after looking into it I think that we shouldn't add any new
macros and instead use '$(subst $(empty) ,:,$(strip $(FOO)))' in this
patch series.



More information about the Libc-alpha mailing list