Re: [PATCH 1/2] gcore: improve shell use

[Keith Seitz <keiths@redhat.com>]

On 3/31/25 11:54 AM, Simon Marchi wrote:
> I think that indeed the line doesn't work as intended then.  I hacked
> gcore to print OPT and OPTARG.  I get:
> 
>      $ ./gcore --version
>      OPT: version
>      OPTARG: version
> 
> OPTARG should be empty here.  If I change
> 
>      OPTARG="${OPTARG#'$OPT'}"
> 
> to
> 
>      OPTARG="${OPTARG#"$OPT"}"
> 
> then I get:
> 
>      $ ./gcore --version
>      OPT: version                                                                                             │
>      OPTARG:
> 
> There is not long option that uses OPTARG, so this bug has no impact,
> but we might as well fix it in this patch.

Yes, I agree. My darn fingers still get confused between python quoting
and shell quoting.

Thank you for pursuing this,
Keith