vcs_* stuff should not raise BuildExceptions

diff --git a/fdroidserver/checkupdates.py b/fdroidserver/checkupdates.py
index 292a495ef822996039a0090d008c76b311241ddf..a292c7bd6f5f6ec8455b1512c9ffb52431cd4db8 100644 (file)
--- a/fdroidserver/checkupdates.py
+++ b/fdroidserver/checkupdates.py
@@ -32,7 +32,7 @@ import logging
 
 import common
 import metadata
-from common import BuildException, VCSException
+from common import VCSException
 from metadata import MetaDataException
 
 
@@ -156,9 +156,6 @@ def check_tags(app, pattern):
             return (hver, hcode, htag)
         return (None, "Couldn't find any version information", None)
 
-    except BuildException as be:
-        msg = "Could not scan app {0} due to BuildException: {1}".format(app['id'], be)
-        return (None, msg, None)
     except VCSException as vcse:
         msg = "VCS error while scanning app {0}: {1}".format(app['id'], vcse)
         return (None, msg, None)
@@ -235,9 +232,6 @@ def check_repomanifest(app, branch=None):
 
         return (version, vercode)
 
-    except BuildException as be:
-        msg = "Could not scan app {0} due to BuildException: {1}".format(app['id'], be)
-        return (None, msg)
     except VCSException as vcse:
         msg = "VCS error while scanning app {0}: {1}".format(app['id'], vcse)
         return (None, msg)
@@ -266,9 +260,6 @@ def check_repotrunk(app, branch=None):
 
         ref = vcs.getref()
         return (ref, ref)
-    except BuildException as be:
-        msg = "Could not scan app {0} due to BuildException: {1}".format(app['id'], be)
-        return (None, msg)
     except VCSException as vcse:
         msg = "VCS error while scanning app {0}: {1}".format(app['id'], vcse)
         return (None, msg)

diff --git a/fdroidserver/common.py b/fdroidserver/common.py
index a26716ae1989d5cad9c5f0461a6eee5218c1571c..1b6631b39ffbcf103519f4bc32ea7a5bf749bf9b 100644 (file)
--- a/fdroidserver/common.py
+++ b/fdroidserver/common.py
@@ -1021,7 +1021,7 @@ def getsrclib(spec, srclib_dir, srclibpaths=[], subdir=None,
             name, subdir = name.split('/', 1)
 
     if name not in metadata.srclibs:
-        raise BuildException('srclib ' + name + ' not found.')
+        raise VCSException('srclib ' + name + ' not found.')
 
     srclib = metadata.srclibs[name]
 
@@ -1058,7 +1058,7 @@ def getsrclib(spec, srclib_dir, srclibpaths=[], subdir=None,
                     s_tuple = t
                     break
             if s_tuple is None:
-                raise BuildException('Missing recursive srclib %s for %s' % (
+                raise VCSException('Missing recursive srclib %s for %s' % (
                     lib, name))
             place_srclib(libdir, n, s_tuple[2])
             n += 1