Hello Kevin,

Am 16.06.21 um 18:03 schrieb Kevin Locke:
> On Wed, 2021-06-16 at 09:22 -0600, Kevin Locke wrote:
>> I've attached a patch which changes the quoting to fix the issue with
>> the hope that you may find it more acceptable than Christophe's patch.
> 
> Please ignore the previous patch.  It was sent with insufficient testing
> and did not handle the quoting in ${TB_GDB_DEFAULT_OPTS} in the way it
> appears to be intended.
> 
> I've attached v2 of the patch which quotes ${TB_ARGS[@]} using
> `printf ' %q'` then uses eval to perform word splitting on the combined
> command string.  In my testing it handled spaces and special characters
> in ${TB_GDB_DEFAULT_OPTS} and arguments following -g in the way that I
> would expect.  See what you think.

thanks, looks good to me, will get included within the next upload.

-- 
Regards
Carsten

Reply via email to