[bug 248932] reconfigure --standalone creates incompatibe repository
Martin von Gagern
Martin.vGagern at gmx.net
Fri Apr 17 14:03:11 BST 2009
Hi!
Find attached a draft towards fixing bug #248932
https://bugs.launchpad.net/bzr/+bug/248932
Problem: reconfigure --standalone creates repositories of default type,
which may differ in rich root support, among others.
Solution: Instead the new repository should use the same format as the
original shared repository, imho.
The patch introduces a blackbox unit test to expose the problem, as well
as a quick and dirty hack to solve it. This is not considered a merge
proposal, but rather a draft to start discussing.
Problems with my approach, and possible solutions:
1. Disregards encapsulation by directly accessing
- BzrDir._format and
- Repository._format
a) introduce getter/setter methods
b) introduce properties
c) Pass Repository to BzrDir to have it copy the format
2. Modifies a BzrDirFormat object which might be shared
a) clone object
b) ensure the object is never reused elsewhere
3. Bad exception handling for
- read-only BzrDirFormat.repository_format and
- unset BzrDir._format
a) identify expected problem situations and deal with them
Please comment on these problems and tell me what you think.
I would have pushed a branch if I hadn't been biten by bug 356699
(preventing upload on a slow connection) and bug 343960 (which prevented
execution of bzr in the first place on a remote fast connection).
Instead you have to make do with the attached merge directive.
Greetings,
Martin
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: bug248932a.patch
Url: https://lists.ubuntu.com/archives/bazaar/attachments/20090417/cf4dea3a/attachment-0001.diff
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 261 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090417/cf4dea3a/attachment-0001.pgp
More information about the bazaar
mailing list