From: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Date: Mon, 25 Mar 2013 09:34:45 +0000 (+0200)
Subject: buildpackage-rpm: use config file(s) from exported tree-ish
X-Git-Tag: tizen/0.6.15-20140828~98
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=04cf2a51306bebf5c3df369be76d8e5281a0cfbc;p=tools%2Fgit-buildpackage.git

buildpackage-rpm: use config file(s) from exported tree-ish

Use the local per-tree/per-branch config file(s) from the exported
tree-ish instead of the current working copy.

Signed-off-by: Markus Lehtonen <markus.lehtonen@linux.intel.com>
---

diff --git a/gbp/scripts/buildpackage_rpm.py b/gbp/scripts/buildpackage_rpm.py
index 3c6b7c1c..9c86c072 100755
--- a/gbp/scripts/buildpackage_rpm.py
+++ b/gbp/scripts/buildpackage_rpm.py
@@ -174,16 +174,19 @@ def get_tree(repo, tree_name):
     Get/create a tree-ish to be used for exporting and diffing. Accepts
     special keywords for git index and working copies.
     """
-    if tree_name == index_name:
-        # Write a tree of the index
-        tree = repo.write_tree()
-    elif tree_name in wc_names:
-        # Write a tree of the working copy
-        tree = write_wc(repo,
-                        force=wc_names[tree_name]['force'],
-                        untracked=wc_names[tree_name]['untracked'])
-    else:
-        tree = tree_name
+    try:
+        if tree_name == index_name:
+            # Write a tree of the index
+            tree = repo.write_tree()
+        elif tree_name in wc_names:
+            # Write a tree of the working copy
+            tree = write_wc(repo,
+                            force=wc_names[tree_name]['force'],
+                            untracked=wc_names[tree_name]['untracked'])
+        else:
+            tree = tree_name
+    except GitRepositoryError as err:
+        raise GbpError(err)
     if not repo.has_treeish(tree):
         raise GbpError('Invalid treeish object %s' % tree)
 
@@ -316,7 +319,7 @@ def create_packaging_tag(repo, tag, commit, version, options):
                     keyid=options.keyid, commit=commit)
 
 
-def parse_args(argv, prefix):
+def parse_args(argv, prefix, git_treeish=None):
     args = [ arg for arg in argv[1:] if arg.find('--%s' % prefix) == 0 ]
     builder_args = [ arg for arg in argv[1:] if arg.find('--%s' % prefix) == -1 ]
 
@@ -326,7 +329,8 @@ def parse_args(argv, prefix):
             args.append(arg)
 
     try:
-        parser = GbpOptionParserRpm(command=os.path.basename(argv[0]), prefix=prefix)
+        parser = GbpOptionParserRpm(command=os.path.basename(argv[0]),
+                                    prefix=prefix, git_treeish=git_treeish)
     except ConfigParser.ParsingError, err:
         gbp.log.err(err)
         return None, None, None
@@ -449,6 +453,16 @@ def main(argv):
     else:
         repo_dir = os.path.abspath(os.path.curdir)
 
+    # Determine tree-ish to be exported
+    try:
+        tree = get_tree(repo, options.export)
+    except GbpError as err:
+        gbp.log.err('Failed to determine export treeish: %s' % err)
+        return 1
+    # Re-parse config options with using the per-tree config file(s) from the
+    # exported tree-ish
+    options, gbp_args, builder_args = parse_args(argv, prefix, tree)
+
     try:
         # Create base temporary directory for this run
         options.tmp_dir = tempfile.mkdtemp(dir=options.tmp_dir,
@@ -477,9 +491,6 @@ def main(argv):
                 gbp.log.err("You are not on branch '%s' but on '%s'" % (options.packaging_branch, branch))
                 raise GbpError, "Use --git-ignore-branch to ignore or --git-packaging-branch to set the branch name."
 
-        # Determine tree-ish to be exported
-        tree = get_tree(repo, options.export)
-
         # Dump from git to a temporary directory:
         dump_dir = tempfile.mkdtemp(dir=options.tmp_dir,
                                     prefix='dump_tree_')