Merge ~nacc/git-ubuntu:lp1734137-add-quilt-gbp-API into git-ubuntu:master

[Nish Aravamudan]

Make jenkins happy.

[Server Team CI bot (server-team-bot)]

PASSED: Continuous integration, rev:7a04bc99041ee22a807bbfb7e0f20bfd9d83741d
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/265/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Style Check
    SUCCESS: Unit Tests
    SUCCESS: Integration Tests
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/265/rebuild

review: Approve (continuous-integration)

[Server Team CI bot (server-team-bot)]

PASSED: Continuous integration, rev:256caffcacb85d2df2d9ec0ff760d6985e13bd7e
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/267/
Executed test runs:
    SUCCESS: Checkout
    SUCCESS: Style Check
    SUCCESS: Unit Tests
    SUCCESS: Integration Tests
    IN_PROGRESS: Declarative: Post Actions

Click here to trigger a rebuild:
https://jenkins.ubuntu.com/server/job/git-ubuntu-ci/267/rebuild

review: Approve (continuous-integration)

[Robie Basak (racb)]

review: Approve

diff --git a/gitubuntu/git_repository.py b/gitubuntu/git_repository.py
index f974f88..f3bd406 100644
--- a/gitubuntu/git_repository.py
+++ b/gitubuntu/git_repository.py
@@ -21,7 +21,13 @@ import gitubuntu.lint
 from gitubuntu.__main__ import top_level_defaults
 import gitubuntu.build
 from gitubuntu.dsc import component_tarball_matches
-from gitubuntu.run import run, runq, decode_binary
+from gitubuntu.run import (
+   decode_binary,
+   run,
+   runq,
+   run_gbp,
+   run_quilt,
+)
 try:
     pkg = 'python3-debian'
     import debian.changelog
@@ -916,7 +922,7 @@ class GitUbuntuRepository:
             tarballs.append(
                 os.path.join(os.path.pardir, potential_main_tarballs[0])
             )
-            cmd = ['gbp', 'buildpackage', '--git-builder=/bin/true',
+            args = ['buildpackage', '--git-builder=/bin/true',
                    '--git-pristine-tar', '--git-ignore-branch',
                    '--git-upstream-tag=%s/upstream/%s/%%(version)s%s' %
                    (namespace, dist, ext)]
@@ -933,11 +939,11 @@ class GitUbuntuRepository:
             tarballs.extend(map(lambda x : os.path.join(os.path.pardir, x),
                 list(potential_component_tarballs.values()))
             )
-            cmd.extend(map(lambda x : '--git-component=%s' % x,
+            args.extend(map(lambda x : '--git-component=%s' % x,
                 list(potential_component_tarballs.keys()))
             )
             with self.pristine_tar_branches(dist, namespace):
-                run(cmd)
+                run_gbp(args, env=self.env)
             return tarballs
 
         raise PristineTarNotFoundError(
@@ -1970,8 +1976,8 @@ class GitUbuntuRepository:
         # differentiate between applied an unapplied
         with self.temporary_worktree(commit_hash):
             try:
-                run(
-                    ['quilt', 'push', '-a'],
+                run_quilt(
+                    ['push', '-a'],
                     env=self.quilt_env_from_treeish_str(commit_hash),
                 )
                 # False if in an unapplied state, which is signified by
@@ -1996,8 +2002,8 @@ class GitUbuntuRepository:
                         raise e
 
                     try:
-                        run(
-                            ['quilt', 'push', '-a'],
+                        run_quilt(
+                            ['push', '-a'],
                             env=self.quilt_env_from_treeish_str(commit_hash),
                         )
                         # False if in an unapplied state
@@ -2092,8 +2098,8 @@ class GitUbuntuRepository:
                 )
 
                 # generate the equivalent .pc directory
-                run(
-                    ['quilt', 'push', '-a'],
+                run_quilt(
+                    ['push', '-a'],
                     env=self.quilt_env_from_treeish_str(commit_hash),
                     rcs=[2],
                 )
@@ -2276,14 +2282,14 @@ with other relevant fields (see below for details).
             'debian/changelog',
         ):
             with self.temporary_worktree(commit_hash):
-                run(
+                run_gbp(
                     [
-                        'gbp',
                         'dch',
                         '--snapshot',
                         '--ignore-branch',
                         '--since=%s' % str(parent_ref),
-                    ]
+                    ],
+                    env=self.env,
                 )
                 return self.dir_to_tree('.')
 
diff --git a/gitubuntu/importer.py b/gitubuntu/importer.py
index 4f7d1c9..c998cf2 100644
--- a/gitubuntu/importer.py
+++ b/gitubuntu/importer.py
@@ -53,7 +53,13 @@ from gitubuntu.git_repository import (
     PristineTarError,
     is_dir_3_0_quilt,
 )
-from gitubuntu.run import decode_binary, run, runq
+from gitubuntu.run import (
+    decode_binary,
+    run,
+    runq,
+    run_gbp,
+    run_quilt,
+)
 from gitubuntu.source_information import (
     GitUbuntuSourceInformation,
     NoPublicationHistoryException,
@@ -696,17 +702,17 @@ def import_orig(repo, dsc, namespace, version, dist):
             ext = os.path.splitext(dsc.orig_tarball_path)[1]
 
             # make this non-fatal if upstream already exist as tagged?
-            cmd = ['gbp', 'import-orig', '--no-merge',
+            args = ['import-orig', '--no-merge',
                  '--upstream-branch', 'do-not-push',
                  '--pristine-tar', '--no-interactive',
                  '--no-symlink-orig',
                  '--upstream-tag=%s/upstream/%s/%%(version)s%s' %
                      (namespace, dist, ext)]
-            cmd.extend(map(lambda x: '--component=%s' % x,
+            args.extend(map(lambda x: '--component=%s' % x,
                 list(orig_component_paths.keys()))
             )
-            cmd.extend([orig_tarball_path,])
-            run(cmd, env=repo.env)
+            args.extend([orig_tarball_path,])
+            run_gbp(args, env=repo.env)
     except CalledProcessError as e:
         raise GitUbuntuImportOrigError(
             'Unable to import tarballs: %s' % orig_paths
@@ -793,18 +799,18 @@ def import_patches_applied_tree(repo, dsc_pathname):
         while True:
             try:
                 os.chdir(extracted_dir)
-                run(
-                    ['quilt', 'push'],
+                run_quilt(
+                    ['push'],
                     env=repo.quilt_env_from_treeish_str(import_tree_hash),
                     verbose_on_failure=False,
                 )
-                patch_name, _ = run(
-                    ['quilt', 'top'],
+                patch_name, _ = run_quilt(
+                    ['top'],
                     env=repo.quilt_env_from_treeish_str(import_tree_hash),
                 )
                 patch_name = patch_name.strip()
-                header, _  = run(
-                    ['quilt', 'header'],
+                header, _  = run_quilt(
+                    ['header'],
                     env=repo.quilt_env_from_treeish_str(import_tree_hash),
                 )
                 header = header.strip()
diff --git a/gitubuntu/run.py b/gitubuntu/run.py
index 8fb4d85..5ff9ccd 100644
--- a/gitubuntu/run.py
+++ b/gitubuntu/run.py
@@ -141,3 +141,41 @@ def run_lxc(args, env=None, **kwargs):
         pass
 
     return run(cmd, env=env_unset_SNAP, **kwargs)
+
+def run_quilt(args, env=None, **kwargs):
+    """execute quilt without reading any configuration files
+
+    Params:
+    args: A list of string arguments to pass to quilt.
+    env: A namespace object to use as the environment. If not specified,
+    the calling environment is inherited.
+    kwargs: passed directly to run().
+
+    Returns:
+    A tuple of stdout, stderr from the quilt process.
+    Raises a subprocess.CalledProcessError exception on error.
+    """
+    cmd = ['quilt', '--quiltrc', '-'] + args
+    return run(cmd, env=env, **kwargs)
+
+def run_gbp(args, env=None, **kwargs):
+    """execute gbp without reading any configuration files
+
+    Params:
+    args: A list of string arguments to pass to gbp.
+    env: A namespace object to use as the environment. If not specified,
+    the calling environment is inherited.
+    kwargs: passed directly to run().
+
+    Returns:
+    A tuple of stdout, stderr from the gbp process.
+    Raises a subprocess.CalledProcessError exception on error.
+    """
+    cmd = ['gbp'] + args
+    if env:
+        env_no_gbp_conf = env.copy()
+    else:
+        env_no_gbp_conf = os.environ.copy()
+    env_no_gbp_conf['GBP_CONF_FILES'] = '/dev/null'
+
+    return run(cmd, env=env_no_gbp_conf, **kwargs)