with patch<br><br><div class="gmail_quote">2009/1/25 Marius Kruger <span dir="ltr">&lt;<a href="mailto:amanic@gmail.com">amanic@gmail.com</a>&gt;</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
hi,<br><br>Don&#39;t raise a ValueError when the user passes dotted revnos in the wrong order to log<br>&nbsp;&nbsp;&nbsp; (eg. `bzr log -r 1.1.1..1`), by comparing revision ids in stead of revnos.<br>&nbsp;&nbsp;&nbsp; This is needed because for some reason we always get a revno=1 for dotted revnos,<br>

&nbsp;&nbsp;&nbsp; which was allowed past this validation.<br><br>I added a assertRevnos() to the blackbox test but didn&#39;t refactor the other tests<br>to use it because of all the other log changes in BundleBuddy and I didn&#39;t want<br>

to cause unnecessary conflicts. (I can do that when all the other log changes have landed)<br><br>regards<br><font color="#888888">marius<br>
</font></blockquote></div><br><br clear="all"><br>-- <br>&lt;| regards<br>U| Marius<br>H| &lt;&gt;&lt; <br>Z| <a href="http://amanica.blogspot.com/">http://amanica.blogspot.com/</a><br>&lt;| <br>E| <a href="http://bazaar-vcs.org/">http://bazaar-vcs.org/</a><br>
&lt;| because I don&#39;t trust version control systems with less than 14777 unit tests<br>