<div class="gmail_quote">On Tue, Jul 10, 2012 at 7:08 PM, Richard Purdie <span dir="ltr">&lt;<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="HOEnZb"><div class="h5">On Tue, 2012-07-10 at 18:04 +0300, Andrei Gherzan wrote:<br>
&gt; On Tue, Jul 10, 2012 at 4:33 PM, Richard Purdie<br>
&gt; &lt;<a href="mailto:richard.purdie@linuxfoundation.org">richard.purdie@linuxfoundation.org</a>&gt; wrote:<br>
&gt;         On Tue, 2012-07-10 at 14:15 +0300, Andrei Gherzan wrote:<br>
&gt;         &gt; Andrei Gherzan (5):<br>
&gt;         &gt;   fetch2/bzr.py: Use &quot;protocol&quot; parameter and warn user if<br>
&gt;         recipe is<br>
&gt;         &gt;     using &quot;proto&quot;<br>
&gt;         &gt;   fetch2/git.py: Use &quot;protocol&quot; parameter and warn user if<br>
&gt;         recipe is<br>
&gt;         &gt;     using &quot;proto&quot;<br>
&gt;         &gt;   fetch2/hg.py: Use &quot;protocol&quot; parameter and warn user if<br>
&gt;         recipe is<br>
&gt;         &gt;     using &quot;proto&quot;<br>
&gt;         &gt;   fetch2/osc.py: Use &quot;protocol&quot; parameter and warn user if<br>
&gt;         recipe is<br>
&gt;         &gt;     using &quot;proto&quot;<br>
&gt;         &gt;   fetch2/svn.py: Use &quot;protocol&quot; parameter and warn user if<br>
&gt;         recipe is<br>
&gt;         &gt;     using &quot;proto&quot;<br>
&gt;         &gt;<br>
&gt;         &gt;  bitbake/lib/bb/fetch2/bzr.py |    7 ++++++-<br>
&gt;         &gt;  bitbake/lib/bb/fetch2/git.py |    4 ++++<br>
&gt;         &gt;  bitbake/lib/bb/fetch2/hg.py  |    7 ++++++-<br>
&gt;         &gt;  bitbake/lib/bb/fetch2/osc.py |    7 ++++++-<br>
&gt;         &gt;  bitbake/lib/bb/fetch2/svn.py |    7 ++++++-<br>
&gt;         &gt;  5 files changed, 28 insertions(+), 4 deletions(-)<br>
&gt;<br>
&gt;         How about putting this check into __init__.py and then just<br>
&gt;         setting the<br>
&gt;         protocol parameter instead if its used? We can then change the<br>
&gt;         backends<br>
&gt;         just to use procotol right now.<br>
&gt;<br>
&gt;<br>
&gt; I can add the warning in init but how would i define the local proto<br>
&gt; value to the one defined with parameter &quot;proto&quot;?<br>
<br>
</div></div>In FetchData::__init__(), add<br>
<br>
        proto = self.parm.get(&quot;proto&quot;, None)<br>
        if proto and &quot;protocol&quot; not in self.param:<br>
            self.parm[&quot;protocol&quot;] = proto<br>
<br>
before urldata_init?<br><br></blockquote><div>Yes. It makes sense this way. V2 is on it&#39;s way.</div><div><br></div><div>@g </div></div><br>