BB:APPROVE<br><br>it looks good to me, and works great!<br><br>thanks<br>marius<br><br><br>On 9/11/07, <b class="gmail_sendername">Lukáš Lalinský</b> <<a href="mailto:lalinsky@gmail.com">lalinsky@gmail.com</a>> wrote:
<div><span class="gmail_quote"></span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">On Ut, 2007-09-11 at 17:25 +1000, Martin Pool wrote:<br>
<br>> That looks reasonable to me, except that I'd change the null name<br>> "my_build_ext" to something like "build_ext_if_possible".<br><br>Renamed.<br><br>> By the way, when posting patches, please remember to put '[merge]' in the
<br>> subject, even if you're replying to someone else, so that bundle buggy<br>> will catch it.<br><br>I know, but I didn't intend to have that particular version merged.<br><br>Lukas<br><br><br></blockquote>
</div><br><br clear="all"><br>-- <br><a href="http://bazaar-vcs.org">bazaar-vcs.org</a><br>Because I don't trust Version Control Systems with less than 6953 unit tests.