diff options
Diffstat (limited to 'meta')
-rw-r--r-- | meta/lib/oe/patch.py | 16 | ||||
-rw-r--r-- | meta/lib/oeqa/selftest/cases/bbtests.py | 15 |
2 files changed, 30 insertions, 1 deletions
diff --git a/meta/lib/oe/patch.py b/meta/lib/oe/patch.py index fccbedb519..950fe723dc 100644 --- a/meta/lib/oe/patch.py +++ b/meta/lib/oe/patch.py | |||
@@ -4,6 +4,7 @@ | |||
4 | 4 | ||
5 | import oe.path | 5 | import oe.path |
6 | import oe.types | 6 | import oe.types |
7 | import subprocess | ||
7 | 8 | ||
8 | class NotFoundError(bb.BBHandledException): | 9 | class NotFoundError(bb.BBHandledException): |
9 | def __init__(self, path): | 10 | def __init__(self, path): |
@@ -25,7 +26,6 @@ class CmdError(bb.BBHandledException): | |||
25 | 26 | ||
26 | def runcmd(args, dir = None): | 27 | def runcmd(args, dir = None): |
27 | import pipes | 28 | import pipes |
28 | import subprocess | ||
29 | 29 | ||
30 | if dir: | 30 | if dir: |
31 | olddir = os.path.abspath(os.curdir) | 31 | olddir = os.path.abspath(os.curdir) |
@@ -56,6 +56,7 @@ def runcmd(args, dir = None): | |||
56 | if dir: | 56 | if dir: |
57 | os.chdir(olddir) | 57 | os.chdir(olddir) |
58 | 58 | ||
59 | |||
59 | class PatchError(Exception): | 60 | class PatchError(Exception): |
60 | def __init__(self, msg): | 61 | def __init__(self, msg): |
61 | self.msg = msg | 62 | self.msg = msg |
@@ -298,6 +299,19 @@ class GitApplyTree(PatchTree): | |||
298 | PatchTree.__init__(self, dir, d) | 299 | PatchTree.__init__(self, dir, d) |
299 | self.commituser = d.getVar('PATCH_GIT_USER_NAME') | 300 | self.commituser = d.getVar('PATCH_GIT_USER_NAME') |
300 | self.commitemail = d.getVar('PATCH_GIT_USER_EMAIL') | 301 | self.commitemail = d.getVar('PATCH_GIT_USER_EMAIL') |
302 | if not self._isInitialized(): | ||
303 | self._initRepo() | ||
304 | |||
305 | def _isInitialized(self): | ||
306 | cmd = "git rev-parse --show-toplevel" | ||
307 | (status, output) = subprocess.getstatusoutput(cmd.split()) | ||
308 | ## Make sure repo is in builddir to not break top-level git repos | ||
309 | return status == 0 and os.path.samedir(output, self.dir) | ||
310 | |||
311 | def _initRepo(self): | ||
312 | runcmd("git init".split(), self.dir) | ||
313 | runcmd("git add .".split(), self.dir) | ||
314 | runcmd("git commit -a --allow-empty -m Patching_started".split(), self.dir) | ||
301 | 315 | ||
302 | @staticmethod | 316 | @staticmethod |
303 | def extractPatchHeader(patchfile): | 317 | def extractPatchHeader(patchfile): |
diff --git a/meta/lib/oeqa/selftest/cases/bbtests.py b/meta/lib/oeqa/selftest/cases/bbtests.py index 6779e62103..a74576bc02 100644 --- a/meta/lib/oeqa/selftest/cases/bbtests.py +++ b/meta/lib/oeqa/selftest/cases/bbtests.py | |||
@@ -297,3 +297,18 @@ INHERIT:remove = \"report-error\" | |||
297 | 297 | ||
298 | test_recipe_summary_after = get_bb_var('SUMMARY', test_recipe) | 298 | test_recipe_summary_after = get_bb_var('SUMMARY', test_recipe) |
299 | self.assertEqual(expected_recipe_summary, test_recipe_summary_after) | 299 | self.assertEqual(expected_recipe_summary, test_recipe_summary_after) |
300 | |||
301 | def test_git_patchtool(self): | ||
302 | """ PATCHTOOL=git should work with non-git sources like tarballs | ||
303 | test recipe for the test must NOT containt git:// repository in SRC_URI | ||
304 | """ | ||
305 | test_recipe = "man-db" | ||
306 | self.write_recipeinc(test_recipe, 'PATCHTOOL=\"git\"') | ||
307 | src = get_bb_var("SRC_URI",test_recipe) | ||
308 | gitscm = re.search("git://", src) | ||
309 | self.assertFalse(gitscm, "test_git_patchtool pre-condition failed: {} test recipe contains git repo!".format(test_recipe)) | ||
310 | result = bitbake('man-db -c patch', ignore_status=False) | ||
311 | fatal = re.search("fatal: not a git repository (or any of the parent directories)", result.output) | ||
312 | self.assertFalse(fatal, "Failed to patch using PATCHTOOL=\"git\"") | ||
313 | self.delete_recipeinc(test_recipe) | ||
314 | bitbake('-cclean man-db') | ||