Re: [PATCH] Fix variable assignments in function invocations

[Eric Blake <eblake@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 2379 bytes --]

On 01/09/2015 10:17 AM, Harald van Dijk wrote:
> Hello all,
> 
> A long-standing problem with dash has been how it deals with variable
> assignments in function invocations, and several packages are affected
> by it, two I've come across recently being autogen and pkg-config (only
> their test suites, luckily).
> 
> A short test script:
> 
>   f() {
>     echo inside f, VAR is $VAR
>     sh -c 'echo inside sh called from f, VAR is $VAR'
>   }
> 
>   VAR=value f

This behavior is tricky.  Here's the latest POSIX wording:
http://austingroupbugs.net/view.php?id=654#c1559

      * If the command name is a function that is not a standard
        utility implemented as a function, variable assignments shall
        affect the current execution environment during the execution
        of the function. It is unspecified:

          - Whether or not the variable assignments persist
            after the completion of the function

          - Whether or not the variables gain the export
            attribute during the execution of the function

          - Whether or not export attributes gained as a result of the
            variable assignments persist after the completion of
            the function (if variable assignments persist after the
            completion of the function)

So the existing dash behavior is compliant, even if different from bash.

> 
> Quoting SUSv4 Shell Command Language 2.9.1 Simple Commands:
> 
>   If no command name results, variable assignments shall affect the
>   current execution environment. Otherwise, the variable assignments
>   shall be exported for the execution environment of the command and
>   shall not affect the current execution environment (except for
>   special built-ins).

This is the text that was rendered obsolete by the above POSIX bug 654.

> Fixing this seems trivial, see the attachment, and the test suites of
> both autogen and pkg-config pass with this change. Does this look correct?

I have no opinion on whether to take the patch in order to behave more
like bash, or whether to tell script-writers to fix their script to
avoid unspecified behavior because dash is already compliant in
providing a different behavior than bash.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]