base, parser, fetch: add nogix in SRCS git option in spec file to control use gix fetch src

Author: eatradish

acbs/base.py

@@ -6,7 +6,7 @@
 
 
 class ACBSSourceInfo(object):
-    def __init__(self, type: str, url: str, revision=None, branch=None, depth=None) -> None:
+    def __init__(self, type: str, url: str, revision=None, branch=None, depth=None, no_gix=False) -> None:
         self.type = type
         self.url = url
         self.revision: Optional[str] = revision
@@ -21,9 +21,10 @@ def __init__(self, type: str, url: str, revision=None, branch=None, depth=None)
         self.copy_repo: bool = False
         # this is a tristate: 0 - off; 1 - on (non-recursive); 2 - recursive
         self.submodule: int = 2
+        self.no_gix: bool = no_gix
 
     def __repr__(self) -> str:
-        return '<ACBSSourceInfo {type}: {url}:{branch}@{revision} integrity: {checksum}>'.format(type=self.type, url=self.url, branch=self.branch, revision=self.revision, checksum=self.chksum)
+        return '<ACBSSourceInfo {type}: {url}:{branch}@{revision} integrity: {checksum} no_gix: {no_gix}>'.format(type=self.type, url=self.url, branch=self.branch, revision=self.revision, checksum=self.chksum, no_gix=self.no_gix)
 
 
 class ACBSPackageInfo(object):

acbs/fetch.py

@@ -119,21 +119,28 @@ def blob_processor(package: ACBSPackageInfo, index: int, source_name: str) -> No
     return tarball_processor_innner(package, index, source_name, False)
 
 
-def git_fetch(info: ACBSSourceInfo, source_location: str, name: str) -> Optional[ACBSSourceInfo]:
-    try:
-        # Try use gix
-        return gix_fetch(info, source_location, name)
-    except:
-        # Fallback to git
-        full_path = os.path.join(source_location, name)
-        if not os.path.exists(full_path):
-            subprocess.check_call(['git', 'clone', '--bare', info.url, full_path])
-        else:
-            logging.info('Updating repository...')
-            subprocess.check_call(
+def git_fetch_fallback(info: ACBSSourceInfo, source_location: str, name: str) -> Optional[ACBSSourceInfo]:
+    if info.no_gix:
+        return git_fetch(info, source_location, name)
+    else:
+        try:
+            # Try use gix
+            return gix_fetch(info, source_location, name)
+        except:
+            # Fallback to git
+            return git_fetch(info, source_location, name)
+
+
+def git_fetch(info, source_location, name):
+    full_path = os.path.join(source_location, name)
+    if not os.path.exists(full_path):
+        subprocess.check_call(['git', 'clone', '--bare', info.url, full_path])
+    else:
+        logging.info('Updating repository...')
+        subprocess.check_call(
                 ['git', 'fetch', 'origin', '+refs/heads/*:refs/heads/*', '--prune'], cwd=full_path)
-        info.source_location = full_path
-        return info
+    info.source_location = full_path
+    return info
 
 
 def gix_hack(path: str):
@@ -334,7 +341,7 @@ def fossil_processor(package: ACBSPackageInfo, index: int, source_name: str) ->
 
 
 handlers: Dict[str, pair_signature] = {
-    'GIT': (git_fetch, git_processor),
+    'GIT': (git_fetch_fallback, git_processor),
     'SVN': (svn_fetch, svn_processor),
     'BZR': (bzr_fetch, bzr_processor),
     'HG': (hg_fetch, hg_processor),

acbs/parser.py

@@ -28,6 +28,7 @@ def parse_url_schema(url: str, checksum: str) -> ACBSSourceInfo:
     url_split = url.split('::', 2)
     schema = ''
     url_plain = ''
+    no_gix= ''
     if len(url_split) < 2:
         url_plain = url
         if re.search(tarball_pattern, url_plain):
@@ -51,6 +52,7 @@ def parse_url_schema(url: str, checksum: str) -> ACBSSourceInfo:
     acbs_source_info.chksum = (
         chksum_[0], chksum_[1]) if checksum != 'SKIP' else ('none', '')
     acbs_source_info.url = url_plain
+    acbs_source_info.no_gix = no_gix
     return acbs_source_info
 
 
@@ -75,6 +77,9 @@ def parse_fetch_options(options: str, acbs_source_info: ACBSSourceInfo):
             if translated is None:
                 raise ValueError(f'Invalid submodule directive: {v}')
             acbs_source_info.submodule = translated
+        elif k == 'nogix':
+            acbs_source_info.no_gix = v.strip() == 'true'
+        
     return acbs_source_info