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

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

[Richard Purdie]

There is other code which can want to run ls-remote style commands with 
different parameters so split out the function.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/bitbake/lib/bb/fetch2/git.py b/bitbake/lib/bb/fetch2/git.py
index f7c26b3..1edcee4 100644
--- a/bitbake/lib/bb/fetch2/git.py
+++ b/bitbake/lib/bb/fetch2/git.py
@@ -317,22 +317,30 @@ class Git(FetchMethod):
         """
         return "git:" + ud.host + ud.path.replace('/', '.') + ud.unresolvedrev[name]
 
-    def _latest_revision(self, ud, d, name):
+    def __lsremote(self, ud, d, search):
         """
-        Compute the HEAD revision for the url
+        Run git ls-remote with the specified search string
         """
         if ud.user:
             username = ud.user + '@'
         else:
             username = ""
 
-        cmd = "%s ls-remote %s://%s%s%s refs/heads/%s refs/tags/%s^{}" % \
-              (ud.basecmd, ud.proto, username, ud.host, ud.path, ud.unresolvedrev[name], ud.unresolvedrev[name])
+        cmd = "%s ls-remote %s://%s%s%s %s" % \
+              (ud.basecmd, ud.proto, username, ud.host, ud.path, search)
         if ud.proto.lower() != 'file':
             bb.fetch2.check_network_access(d, cmd)
         output = runfetchcmd(cmd, d, True)
         if not output:
             raise bb.fetch2.FetchError("The command %s gave empty output unexpectedly" % cmd, ud.url)
+        return output
+
+    def _latest_revision(self, ud, d, name):
+        """
+        Compute the HEAD revision for the url
+        """
+        search = "refs/heads/%s refs/tags/%s^{}" % (ud.unresolvedrev[name], ud.unresolvedrev[name])
+        output = self.__lsremote(ud, d, search)
         return output.split()[0]
 
     def _build_revision(self, ud, d, name):

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

[Chris Larson]

[-- Attachment #1: Type: text/plain, Size: 603 bytes --]

On Fri, Feb 28, 2014 at 10:22 AM, Richard Purdie <
richard.purdie@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 :)
-- 
Christopher Larson
clarson at kergoth dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics

[-- Attachment #2: Type: text/html, Size: 1020 bytes --]

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

[Richard Purdie]

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@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