[MERGE] Fix case-sensitivity detection

Aaron Bentley aaron.bentley at utoronto.ca
Wed Nov 28 14:29:48 GMT 2007


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


Alexander Belchenko wrote:
> Aaron Bentley пишет:
> +    def test_apply_case_conflict(self):
> +        tree = self.make_branch_and_tree('tree')
> +        # Don't try this at home, kids!
> +        # Force the tree to report that it is case insensitive, for
> conflict
> +        # resolution tests
> 
> <-- it seems you forget to force insensitiveness here, as comment claims
> I.e. the line is missed:

Here's an updated patch.

I've removed the incorrect comments for test_apply_case_conflicts, and
also added an additional pair of files ('dir/dirfile' and
'dir/dirFiLe'), to ensure that when files are created inside their
containing directory, creating files with duplicate names doesn't cause
conflicts.

> Also when you're comment on my previous attempt to introduce case
> sensitiveness detection you said test for _detect_case_handling()
> should be in workingtree_implementation group. But you're not
> provide any test for it. Is it intended or you just forget about it?

I've now provided tests for WorkingTree.case_sensitive, which is the
public API.  (presumably a tree could use some mechanism other than
_detect_case_handling() to set it.)

I'm pretty confident these updated tests will pass on win32, but I don't
have a machine handy to test with.  Could you please run them, Alexander?

Aaron
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFHTXtc0F+nu1YWqI0RAmDIAJ0Vw89xt4ziAeKiZwAVrrz+2P5sRgCffTmw
XaY9Jx9V28i7R7oovcJeh+s=
=t0ac
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: case-sensitive-2.patch
Type: text/x-patch
Size: 15987 bytes
Desc: not available
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20071128/83115eee/attachment-0001.bin 


More information about the bazaar mailing list