[bitbake-devel] [PATCH] fetch/git: Separate out an ls-remote function

Richard Purdie richard.purdie at linuxfoundation.org
Sun Mar 2 17:26:48 UTC 2014


On Sat, 2014-03-01 at 21:50 -0700, Chris Larson wrote:
> 
> On Fri, Feb 28, 2014 at 10:22 AM, Richard Purdie
> <richard.purdie at linuxfoundation.org> wrote:
>         -    def _latest_revision(self, ud, d, name):
>         +    def __lsremote(self, ud, d, search):
> 
> 
> Are we sure we want to use __ here? Double-underscore will trigger
> python internal name mangling. While using it doesn't break anything,
> it doesn't really buy us much either vs _, and makes inspection of the
> class/object rather ugly, IMO :)
> 
Agreed, I changed it before merging.

Cheers,

Richard





More information about the bitbake-devel mailing list