Re: [PATCH v2 3/5] completion: add and use __git_compute_first_level_config_vars_for_section

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 14, 2024, 00:24 UTC
Message-ID: <xmqqa5o328r8.fsf@gitster.g>
In-Reply-To: <996fab96-82b1-0b54-3a09-0ecc18d68a11@gmail.com>

Philippe Blain <levraiphilippeblain@gmail.com> writes:

Show 10 quoted lines

>> A silly question (primarily because I do not much use the indirect
>> reference construct ${!name}).  Does the assignment with printf need
>> to spell out the long variable name with "_${section}"?  Can it be
>> 
>>     printf -v "$this_section" ...
>> 
>> instead, as we already have the short-hand for it?
>
> No, unfortunately neither "$this_section" nor "${!this_section}"
> work, so we must use the long name.

Hmph, this does not match my experiment, though. What am I doing wrong?

        bash$ vname=foo
        bash$ foo=bar
        bash$ set | grep foo
        foo=bar
        vname=foo
        bash$ printf -v "$vname" "%d" 1234
        bash$ set | grep foo
        foo=1234
        vname=foo
        bash$ echo $BASH_VERSION
        5.2.21(1)-release