From: Markus Lehtonen <markus.lehtonen@linux.intel.com>
Date: Mon, 9 Jul 2012 13:10:37 +0000 (+0300)
Subject: Introduce PkgPolicy.is_valid_orig_archive()
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=78bb757baa7052da548abf38de77f03f09003d9c;p=tools%2Fgit-buildpackage.git

Introduce PkgPolicy.is_valid_orig_archive()

A new method for checking if the given filename is usable as a orig
(upstream) source archive.

Also, takes this method into use in the UpstreamSource class.

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

diff --git a/gbp/pkg/__init__.py b/gbp/pkg/__init__.py
index 3f125e1b..47b167ae 100644
--- a/gbp/pkg/__init__.py
+++ b/gbp/pkg/__init__.py
@@ -133,6 +133,14 @@ class PkgPolicy(object):
             raise NotImplementedError("Class needs to provide upstreamversion_re")
         return True if cls.upstreamversion_re.match(version) else False
 
+    @classmethod
+    def is_valid_orig_archive(cls, filename):
+        "Is this a valid orig source archive"
+        (base, arch_fmt, compression) =  parse_archive_filename(filename)
+        if arch_fmt == 'tar' and compression:
+            return True
+        return False
+
     @staticmethod
     def guess_upstream_src_version(filename, extra_regex=r''):
         """
@@ -266,16 +274,7 @@ class UpstreamSource(object):
             self._orig = False
             return
 
-        parts = self._path.split('.')
-        try:
-            if parts[-1] == 'tgz':
-                self._orig = True
-            elif parts[-2] == 'tar':
-                if (parts[-1] in compressor_opts or
-                    parts[-1] in compressor_aliases):
-                        self._orig = True
-        except IndexError:
-            self._orig = False
+        self._orig = self._pkg_policy.is_valid_orig_archive(os.path.basename(self.path))
 
     def is_orig(self):
         """