<div dir="ltr">2008/7/23 John Arbash Meinel <span dir="ltr">&lt;<a href="mailto:john@arbash-meinel.com">john@arbash-meinel.com</a>&gt;</span><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<div class="Ih2E3d"><br>
Matthew D. Fuller wrote:<br>
&gt; On Wed, Jul 23, 2008 at 12:21:33PM +1000 I heard the voice of<br>
&gt; Rob Weir, and lo! it spake thus:<br>
&gt;&gt; On 21 Jul 2008, Marius Kruger wrote:<br>
&gt;&gt;&gt; I&#39;ve been using `bzr switch` a bit recently, and I&#39;m thinking it<br>
&gt;&gt;&gt; will be nice if it will update the branch&#39;s nick too to be the<br>
&gt;&gt;&gt; nick of the new branch we&#39;re binding to. &nbsp;It makes sense if you<br>
&gt;&gt;&gt; use a checkout as a working branch and switch between other local<br>
&gt;&gt;&gt; branches for current work.<br>
&gt;&gt; Great - lightweight checkouts already work like this, so it would be<br>
&gt;&gt; good to make regular ones consistent.<br>
&gt;<br>
&gt; Even more consistent would be if they properly set the nick on the<br>
&gt; initial checkout too.<br>
&gt;<br>
&gt;<br>
<br>
</div>Or just always returned the nick of the master branch, if that is what we<br>
really want. Then you wouldn&#39;t have to hack around it in every command that<br>
might do something to effect the nick.<br>
<br>
Simply (psuedocode)<br>
<br>
<br>
def _get_nick(self):<br>
 &nbsp;master = self.get_master_branch()<br>
 &nbsp;if master is None:<br>
 &nbsp; &nbsp;# return the local value<br>
 &nbsp; &nbsp;config = self.get_config()<br>
 &nbsp;else:<br>
 &nbsp; &nbsp;config = master.get_config()<br>
 &nbsp;return config.get_nick()<br>
</blockquote></div><br>I dont like the idea of checkouts dynamically using the masters&#39; nick, since for checkouts you might want to change the nick.<br>But when you use switch, you generally want it to look like youre actually working directly on the master branch,<br>
but still having the option of changing it if you like.<br><br>I attached the patch with the bug number in the news.<br><br>regards<br>marius<br>
</div>