~bzr-pqm/bzr/bzr.dev

Viewing all changes in revision 5447.4.11.

  • Committer: Vincent Ladeuil
  • Date: 2010-10-04 17:24:52 UTC
  • mto: This revision was merged to the branch mainline in revision 5499.
  • Revision ID: v.ladeuil+lp@free.fr-20101004172452-unu266taysseifa4
Implement ``bzr config --remove <option>``.

* bzrlib/tests/test_config.py:
(TestConfigRemoveOption): Complete the tests.

* bzrlib/tests/blackbox/test_config.py:
(TestConfigRemoveOption.test_unknown_config): Test the cmd_config
option removal.

* bzrlib/config.py:
(IniBasedConfig.get_sections): Add the configuration id to the
returned tuples or we can't implement option removal for
BranchConfig.
(IniBasedConfig.remove_user_option): Default implementation.
(LockableConfig.remove_user_option): Implementation for lockable
configs.
(BranchConfig._get_branch_data_config): Force the id for the
branch data config to avoid forcing it in TreeConfig (oh my...).
(TreeConfig.remove_option): Use a different signature to avoid
confusion with remove_user_option (geez).
(TransportConfig.remove_option): The other side of
remove_user_option for BranchConfig.
(cmd_config._remove_config_option): The working implementation for
cmd config and two unsuccesful
approaches (BranchConfig/TreeConfig/TransportConfig is a mess when
it comes to provide precise access).

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: