"bzr config" ignores policy:appendpath
Bug #671050 reported by
John A Meinel
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Bazaar |
Fix Released
|
High
|
Vincent Ladeuil |
Bug Description
The new "bzr config" command is pretty handy, but it seems to be missing policy: handling.
Specifically, I have:
[/path/to/dev]
push_location = lp:~jameinel
push_location:
[/path/to/dev/bzr]
push_location = lp:~jameinel/bzr
push_location:
And it works correctly for "bzr push", and "bzr info" gives:
$ bzr info
...
push branch: lp:///~jameinel/bzr/2.3-gcb-peak-mem
...
However "bzr config push_location":
$ bzr config push_location
locations:
push_location = lp:///~jameinel/bzr
push_location = lp:~jameinel
Related branches
lp:~vila/bzr/671050-config-policy
- Martin Pool: Approve
-
Diff: 178 lines (+69/-3)4 files modifiedbzrlib/config.py (+13/-1)
bzrlib/tests/blackbox/test_config.py (+32/-0)
bzrlib/tests/test_config.py (+20/-2)
doc/en/release-notes/bzr-2.3.txt (+4/-0)
Changed in bzr: | |
assignee: | nobody → Vincent Ladeuil (vila) |
Changed in bzr: | |
status: | Confirmed → In Progress |
Changed in bzr: | |
milestone: | none → 2.3b4 |
Changed in bzr: | |
status: | In Progress → Fix Released |
To post a comment you must log in.
That's weird, it doesn't display anything about push_location: appendpath ?
I see no reason to display lp:///~jameinel/bzr instead of lp:~jameinel either, wrong copy/paste ?
Now, for locations.conf where several sections mention the same option, the section name needs to be displayed...
I was hoping to leave that un-addressed for a while, but I forgot about this case, so the section name should be displayed too.
But I think the right display for your case should be:
$ bzr config --all push_location location: appendpath = True location: appendpath = True
locations:
[/path/to/dev/bzr]
push_location = lp:~jameinel/bzr
push_
[/path/to/dev]
push_location = lp:~jameinel
push_