hopefully this bundle address all the comments on the list.<br>The changes I made since the last patch are:<br><br>=== modified file &#39;bzrlib/tests/blackbox/test_remove.py&#39;<br>--- bzrlib/tests/blackbox/test_remove.py&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 2007-05-06 01:47:14 +0000
<br>+++ bzrlib/tests/blackbox/test_remove.py&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 2007-06-23 19:45:17 +0000<br>@@ -116,7 +116,7 @@ class TestRemove(ExternalBase):<br>&nbsp;&nbsp;&nbsp;&nbsp; def test_remove_invalid_files(self):<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.build_tree(files)<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; tree = 
self.make_branch_and_tree(&#39;.&#39;)<br>-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.run_bzr(&#39;remove . xyz abc/def&#39;)<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.run_bzr(&#39;remove&#39;, &#39;.&#39;, &#39;xyz&#39;, &#39;abc/def&#39;)<br><br>&nbsp;&nbsp;&nbsp;&nbsp; def test_remove_unversioned_files(self):
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.build_tree(files)<br>@@ -172,7 +172,7 @@ class TestRemove(ExternalBase):<br><br>&nbsp;&nbsp;&nbsp;&nbsp; def test_remove_non_existing_files(self):<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; tree = self._make_add_and_assert_tree([])<br>-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.run_bzr
(&#39;remove b&#39;)<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.run_bzr(&#39;remove&#39;, &#39;b&#39;)<br><br>&nbsp;&nbsp;&nbsp;&nbsp; def test_remove_keep_non_existing_files(self):<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; tree = self._make_add_and_assert_tree([])<br><br>=== modified file &#39;bzrlib/tests/workingtree_implementations/test_remove.py&#39;
<br>--- bzrlib/tests/workingtree_implementations/test_remove.py&nbsp;&nbsp;&nbsp;&nbsp; 2007-06-23 17:59:22 +0000<br>+++ bzrlib/tests/workingtree_implementations/test_remove.py&nbsp;&nbsp;&nbsp;&nbsp; 2007-06-23 19:45:17 +0000<br>@@ -34,11 +34,9 @@ class TestRemove(TestCaseWithWorkingTree
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.build_tree(TestRemove.files)<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return tree<br><br>-&nbsp;&nbsp;&nbsp; def _assertRemoveErrorContainsRe(self, e, file_detail_re):<br>-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; # since assertContainsRe can&#39;t take a DOTALL parameter like<br>
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; # normal re.match, use the following to disregard newlines<br>-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; err = &#39; &#39;.join(str(e).split())<br>-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.assertContainsRe(err, &#39;Can\&#39;t remove changed or unknown&#39;<br>+&nbsp;&nbsp;&nbsp; def _assertRemoveErrorContainsRe(self, exception, file_detail_re):
<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.assertContainsRe(str(exception),<br>+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &#39;(?s)Can\&#39;t remove changed or unknown&#39;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &#39; files:.*&#39; + file_detail_re)<br><br>&nbsp;&nbsp;&nbsp;&nbsp; def test_remove_keep(self):<br><br><br>
<br><div><span class="gmail_quote">On 6/22/07, <b class="gmail_sendername">Martin Pool</b> &lt;<a href="mailto:mbp@canonical.com">mbp@canonical.com</a>&gt; wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Martin Pool has voted +0.<br>Status is now: Waiting<br>Comment:<br>Comments on list.<br><br>For details, see:<br><a href="http://bundlebuggy.aaronbentley.com/request/%3C418c22640705062115x34c0f1fcl91c82a9d2d0659fb%40mail.gmail.com%3E">
http://bundlebuggy.aaronbentley.com/request/%3C418c22640705062115x34c0f1fcl91c82a9d2d0659fb%40mail.gmail.com%3E</a><br><br></blockquote></div><br><br clear="all"><br>-- <br><a href="http://bazaar-vcs.org">bazaar-vcs.org</a>
<br>Because I don&#39;t trust Version Control Systems with less than 6350 unit tests.