[OE-core] tcf-agent, matchbox-wm-2: Correct "proto=git" to "protocol=git".

Andrei Gherzan andrei at gherzan.ro
Mon Jul 9 20:22:04 UTC 2012


On Mon, Jul 9, 2012 at 8:40 PM, Khem Raj <raj.khem at gmail.com> wrote:

> On Mon, Jul 9, 2012 at 10:00 AM, Richard Purdie
> <richard.purdie at linuxfoundation.org> wrote:
> > On Mon, 2012-07-09 at 12:27 +0300, Andrei Gherzan wrote:
> >> On Sat, Jul 7, 2012 at 1:54 PM, Robert P. J. Day
> >> <rpjday at crashcourse.ca> wrote:
> >>
> >>         "proto=git" is not a valid parameter for the git fetcher.
> >>
> >> NOTE: package matchbox-wm-2-0.1+git1
> >> +01fa5465743c9ee43d040350f4405d35293e4869-r1: task do_fetch:
> >> Succeeded
> >>
> >> Anyway, in the git.py, code is searching for "protocol" parameter:
> >> ud.proto = ud.parm['protocol']
> >>
> >> ... so your change seems legit.
> >>
> >> On another hand, svn.py is using:
> >>  proto = ud.parm.get('proto', 'svn')
> >>
> >> A good think would be to use the same parameter names for all
> >> protocols.
> >>
> > I agree but how do we do this without breaking the world? We probably
> > should try and standardise it, probably to "protocol".
>
> so may be change the keyword to protocol and have both for this
> release additionally give a deprecation warning on proto and then
> after 1.3 remove proto
>

Healthy approach. I can submit patches on this topic if nobody did it
already locally.

@g
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20120709/d2a062fd/attachment-0002.html>


More information about the Openembedded-core mailing list