Code review comment for lp:~vila/bzr/config-expand-options

Revision history for this message
Vincent Ladeuil (vila) wrote :

>>>>> John A Meinel <email address hidden> writes:

<snip/>
    > I'm a bit concerned about scope here. For example if in locations.conf I do:

    > [/one/branch/path]
    > foo = hello
    > bar = {foo}/2

    > [/another/branch/path]
    > bar = {foo}/2

A trickier example would be:

    > [/project]
    > foo = hello
    > bar = {foo}/2

    > [/project/branch]
    > bar = {foo}/3

And this case should be valid.

Tests coming soon...

« Back to merge proposal