From: Richard Purdie Date: Tue, 19 Nov 2013 13:08:59 +0000 (+0000) Subject: bitbake: git: Use git branch for ancestor checks X-Git-Tag: rev_ivi_2015_02_04~10345 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c496b293526522ddb7604abfcc9c0b9121f76a72;p=scm%2Fbb%2Ftizen-distro.git bitbake: git: Use git branch for ancestor checks Using git merge-base for checking for ancestors is nice but required git 1.8.0 which is not in many distrbutions yet. We therefore revert to a more ugly check using git branch --contains until such times as we can upgrade. (Bitbake rev: 31467c0afe0346502fcd18bd376f23ea76a27d61) Signed-off-by: Richard Purdie --- diff --git a/bitbake/lib/bb/fetch2/git.py b/bitbake/lib/bb/fetch2/git.py index 99230c1..a901a44 100644 --- a/bitbake/lib/bb/fetch2/git.py +++ b/bitbake/lib/bb/fetch2/git.py @@ -283,12 +283,14 @@ class Git(FetchMethod): def _contains_ref(self, tag, branch, d): basecmd = data.getVar("FETCHCMD_git", d, True) or "git" - cmd = "%s merge-base --is-ancestorlog %s %s" % (basecmd, tag, branch) + cmd = "%s branch --contains %s --list %s 2> /dev/null | wc -l" % (basecmd, tag, branch) try: output = runfetchcmd(cmd, d, quiet=True) except bb.fetch2.FetchError: return False - return True + if len(output.split()) > 1: + raise bb.fetch2.FetchError("The command '%s' gave output with more then 1 line unexpectedly, output: '%s'" % (cmd, output)) + return output.split()[0] != "0" def _revision_key(self, url, ud, d, name): """