chicken-hackers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] handle misquoting of target in build-scripts


From: felix . winkelmann
Subject: Re: [PATCH] handle misquoting of target in build-scripts
Date: Tue, 01 Nov 2022 10:52:52 +0100

> On Mon, Oct 31, 2022 at 05:46:55PM +0100, felix.winkelmann@bevuta.com wrote:
> > Thanks, pushed. I had to hand-apply the patch as git-am didn't like it. I 
> > also
> > fixed a scrutinizer warning.
>
> I suppose the warning was about target-librarian-options not being a
> list?  Unfortunately, splicing in these options wouldn't work properly -
> they're taken from a baked in foreign value which may contain spaces.
>
> We'll have to figure out a way to keep certain arguments "as-is".
> Attached is a pretty straightforward proposal to make this work.

Thanks - pushed.

>
> PS: I noticed two more mistakes that have nothing to do with this - just
> the platform was still being passed in to "filelist" and was *not* being
> passed in to print-build-command.  I've fixed these already and pushed
> them as trivial patches.

Excellent.


cheers,
felix




reply via email to

[Prev in Thread] Current Thread [Next in Thread]