alibuild 1.17.11__tar.gz → 1.17.13__tar.gz

{alibuild-1.17.11 → alibuild-1.17.13}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: alibuild
-Version: 1.17.11
+Version: 1.17.13
 Summary: ALICE Build Tool
 Home-page: https://alisw.github.io/alibuild
 Author: Giulio Eulisse

{alibuild-1.17.11 → alibuild-1.17.13}/alfaBuild

@@ -15,7 +15,7 @@ from alibuild_helpers.init import doInit
 from alibuild_helpers.clean import doClean
 from alibuild_helpers.doctor import doDoctor
 from alibuild_helpers.deps import doDeps
-from alibuild_helpers.log import info, warning, debug, logger, error
+from alibuild_helpers.log import info, debug, logger, error
 from alibuild_helpers.utilities import detectArch
 from alibuild_helpers.build import doBuild
 
@@ -40,7 +40,6 @@ def doMain(args, parser):
     "LC_MONETARY": "C",
     "LC_NUMERIC": "C",
     "LC_TIME": "C",
-    "LC_ALL": "C",
     "GREP_OPTIONS": "",
     "BASH_ENV": "",
     "ALIBUILD_ARCHITECTURE": args.architecture
@@ -58,7 +57,7 @@ def doMain(args, parser):
       error(e.message)
       exit(1)
 
-  if args.action == "version":
+  if args.action == "version" or args.action == None:
     print("aliBuild version: {version} ({arch})".format(
       version=__version__ or "unknown", arch=args.architecture or "unknown"))
     sys.exit(0)
@@ -116,10 +115,7 @@ if __name__ == "__main__":
     if profiler:
       print("profiler started")
       import cProfile, pstats
-      try:
-        from StringIO import StringIO
-      except ImportError:
-        from io import StringIO
+      from io import StringIO
       pr = cProfile.Profile()
       pr.enable()
       def profiler():

{alibuild-1.17.11 → alibuild-1.17.13}/aliBuild

@@ -15,7 +15,7 @@ from alibuild_helpers.init import doInit
 from alibuild_helpers.clean import doClean
 from alibuild_helpers.doctor import doDoctor
 from alibuild_helpers.deps import doDeps
-from alibuild_helpers.log import info, warning, debug, logger, error
+from alibuild_helpers.log import info, debug, logger, error
 from alibuild_helpers.utilities import detectArch
 from alibuild_helpers.build import doBuild
 
@@ -40,7 +40,6 @@ def doMain(args, parser):
     "LC_MONETARY": "C",
     "LC_NUMERIC": "C",
     "LC_TIME": "C",
-    "LC_ALL": "C",
     "GREP_OPTIONS": "",
     "BASH_ENV": "",
     "ALIBUILD_ARCHITECTURE": args.architecture
@@ -58,7 +57,7 @@ def doMain(args, parser):
       error(e.message)
       exit(1)
 
-  if args.action == "version":
+  if args.action == "version" or args.action == None:
     print("aliBuild version: {version} ({arch})".format(
       version=__version__ or "unknown", arch=args.architecture or "unknown"))
     sys.exit(0)
@@ -116,10 +115,7 @@ if __name__ == "__main__":
     if profiler:
       print("profiler started")
       import cProfile, pstats
-      try:
-        from StringIO import StringIO
-      except ImportError:
-        from io import StringIO
+      from io import StringIO
       pr = cProfile.Profile()
       pr.enable()
       def profiler():

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: alibuild
-Version: 1.17.11
+Version: 1.17.13
 Summary: ALICE Build Tool
 Home-page: https://alisw.github.io/alibuild
 Author: Giulio Eulisse

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild.egg-info/SOURCES.txt

@@ -3,7 +3,6 @@
 .pylintrc
 ANALYTICS.md
 DESIGN.md
-Jenkinsfile
 LICENSE.md
 MANIFEST.in
 PACKAGING.md
@@ -63,6 +62,7 @@ docs/docs/reference.md
 docs/docs/troubleshooting.md
 docs/docs/user.md
 docs/docs/stylesheets/extra.css
+templates/alibuild_to_please.jnj
 tests/test_analytics.py
 tests/test_args.py
 tests/test_build.py

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/_version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '1.17.11'
-__version_tuple__ = version_tuple = (1, 17, 11)
+__version__ = version = '1.17.13'
+__version_tuple__ = version_tuple = (1, 17, 13)

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/analytics.py

@@ -22,10 +22,7 @@ def askForAnalytics():
   banner("In order to improve user experience, aliBuild would like to gather "
          "analytics about your builds.\nYou can find all the details at:\n\n"
          "  https://github.com/alisw/alibuild/blob/master/ANALYTICS.md\n")
-  # raw_input and input are different between python 2 and 3
-  try: _input = raw_input
-  except NameError: _input = input
-  a = _input("Is that ok for you [YES/no]? ")
+  a = input("Is that ok for you [YES/no]? ")
   if a.strip() and a.strip().lower().startswith("n"):
     debug("User requsted disabling analytics.")
     return disable_analytics()

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/args.py

@@ -7,10 +7,7 @@ import re
 import os
 import shlex
 
-try:
-  import commands
-except ImportError:
-  import subprocess as commands
+import subprocess as commands
 from os.path import abspath, dirname, basename
 import sys
 
@@ -110,6 +107,8 @@ def doParseArgs():
                                   "in multiple packages. The comment will only be stored if "
                                   "PACKAGE is compiled or downloaded during this run; if it "
                                   "already exists, this does not happen."))
+  build_parser.add_argument("--only-deps", dest="onlyDeps", default=False, action="store_true",
+                            help="Only build dependencies, not the main package (e.g. for caching)")
 
   build_docker = build_parser.add_argument_group(title="Build inside a container", description="""\
   Builds can be done inside a Docker container, to make it easier to get a

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/build.py

@@ -4,8 +4,8 @@ from alibuild_helpers import __version__
 from alibuild_helpers.analytics import report_event
 from alibuild_helpers.log import debug, info, banner, warning
 from alibuild_helpers.log import dieOnError
-from alibuild_helpers.cmd import execute, DockerRunner, BASH, install_wrapper_script
-from alibuild_helpers.utilities import prunePaths, symlink, call_ignoring_oserrors, topological_sort
+from alibuild_helpers.cmd import execute, DockerRunner, BASH, install_wrapper_script, getstatusoutput
+from alibuild_helpers.utilities import prunePaths, symlink, call_ignoring_oserrors, topological_sort, detectArch
 from alibuild_helpers.utilities import resolve_store_path
 from alibuild_helpers.utilities import parseDefaults, readDefaults
 from alibuild_helpers.utilities import getPackageList, asList
@@ -17,19 +17,12 @@ from alibuild_helpers.git import Git, git
 from alibuild_helpers.sl import Sapling
 from alibuild_helpers.scm import SCMError
 from alibuild_helpers.sync import remote_from_url
-import yaml
 from alibuild_helpers.workarea import logged_scm, updateReferenceRepoSpec, checkout_sources
 from alibuild_helpers.log import ProgressPrint, log_current_package
 from glob import glob
 from textwrap import dedent
-try:
-  from collections import OrderedDict
-except ImportError:
-  from ordereddict import OrderedDict
-try:
-  from shlex import quote  # Python 3.3+
-except ImportError:
-  from pipes import quote  # Python 2.7
+from collections import OrderedDict
+from shlex import quote
 
 import concurrent.futures
 import importlib
@@ -193,10 +186,7 @@ def storeHashes(package, specs, considerRelocation):
       hasher(data)
 
   for key in ("env", "append_path", "prepend_path"):
-    if sys.version_info[0] < 3 and key in spec and isinstance(spec[key], OrderedDict):
-      # Python 2: use YAML dict order to prevent changing hashes
-      h_all(str(yaml.safe_load(yamlDump(spec[key]))))
-    elif key not in spec:
+    if key not in spec:
       h_all("none")
     else:
       # spec["env"] is of type OrderedDict[str, str].
@@ -486,7 +476,7 @@ def doBuild(args, parser):
     checkedOutCommitName = scm.checkedOutCommitName(directory=args.configDir)
   except SCMError:
     dieOnError(True, "Cannot find SCM directory in %s." % args.configDir)
-  os.environ["ALIBUILD_ALIDIST_HASH"] = checkedOutCommitName
+  os.environ["ALIBUILD_ALIDIST_HASH"] = checkedOutCommitName # type: ignore
 
   debug("Building for architecture %s", args.architecture)
   debug("Number of parallel builds: %d", args.jobs)
@@ -550,9 +540,17 @@ def doBuild(args, parser):
     del develCandidates, develCandidatesUpper, develPkgsUpper
 
   if buildOrder:
-    banner("Packages will be built in the following order:\n - %s",
-           "\n - ".join(x+" (development package)" if x in develPkgs else "%s@%s" % (x, specs[x]["tag"])
-                        for x in buildOrder if x != "defaults-release"))
+    if args.onlyDeps: 
+      builtPackages = buildOrder[:-1]
+    else:
+      builtPackages = buildOrder
+    if len(builtPackages) > 1:
+      banner("Packages will be built in the following order:\n - %s",
+             "\n - ".join(x+" (development package)" if x in develPkgs else "%s@%s" % (x, specs[x]["tag"])
+                          for x in builtPackages if x != "defaults-release"))
+    else:
+      banner("No dependencies of package %s to build.", buildOrder[-1])
+
 
   if develPkgs:
     banner("You have packages in development mode (%s).\n"
@@ -703,6 +701,12 @@ def doBuild(args, parser):
     deps=",".join(buildOrder[:-1]),
   ), args.architecture)
 
+  # If we are building only the dependencies, the last package in
+  # the build order can be considered done.
+  if args.onlyDeps and len(buildOrder) > 1:
+    mainPackage = buildOrder.pop()
+    warning("Not rebuilding %s because --only-deps option provided.", mainPackage)
+
   while buildOrder:
     p = buildOrder[0]
     spec = specs[p]
@@ -1108,6 +1112,29 @@ def doBuild(args, parser):
       """)
       buildErrMsg += "".join("\n  ( cd %s && git pull --rebase )" % dp for dp in updatablePkgs)
 
+    # Gather build info for the error message
+    try:
+      safe_args = {
+        "pkgname", "defaults", "architecture", "forceUnknownArch",
+        "develPrefix", "jobs", "noSystem", "noDevel", "forceTracked", "plugin",
+        "disable", "annotate", "onlyDeps", "docker"
+          }
+      args_str = " ".join(f"--{k}={v}" for k, v in vars(args).items() if v and k in safe_args)
+      detected_arch = detectArch()
+      buildErrMsg += dedent(f"""
+      Build info:
+      OS: {detected_arch}
+      Using aliBuild from alibuild@{__version__ or "unknown"} recipes in alidist@{os.environ["ALIBUILD_ALIDIST_HASH"][:10]}
+      Build arguments: {args_str}
+      """)
+
+      if detected_arch.startswith("osx"):
+        buildErrMsg += f'XCode version: {getstatusoutput("xcodebuild -version")[1]}'
+
+    except Exception as exc:
+      warning("Failed to gather build info", exc_info=exc)
+
+
     dieOnError(err, buildErrMsg.strip())
 
     # We need to create 2 sets of links, once with the full requires,
@@ -1122,14 +1149,19 @@ def doBuild(args, parser):
     if not spec["revision"].startswith("local"):
       syncHelper.upload_symlinks_and_tarball(spec)
 
-  banner("Build of %s successfully completed on `%s'.\n"
-         "Your software installation is at:"
-         "\n\n  %s\n\n"
-         "You can use this package by loading the environment:"
-         "\n\n  alienv enter %s/latest-%s",
-         mainPackage, socket.gethostname(),
-         abspath(join(args.workDir, args.architecture)),
-         mainPackage, mainBuildFamily)
+  if not args.onlyDeps:
+      banner("Build of %s successfully completed on `%s'.\n"
+             "Your software installation is at:"
+             "\n\n  %s\n\n"
+             "You can use this package by loading the environment:"
+             "\n\n  alienv enter %s/latest-%s",
+             mainPackage, socket.gethostname(),
+             abspath(join(args.workDir, args.architecture)),
+             mainPackage, mainBuildFamily)
+  else:
+      banner("Successfully built dependencies for package %s on `%s'.\n",
+             mainPackage, socket.gethostname()
+            )
   for spec in specs.values():
     if spec["is_devel_pkg"]:
       banner("Build directory for devel package %s:\n%s/BUILD/%s-latest%s/%s",

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/cmd.py

@@ -1,68 +1,37 @@
 import os
 import os.path
-import sys
 import time
 from subprocess import Popen, PIPE, STDOUT
 from textwrap import dedent
-try:
-  from subprocess import TimeoutExpired
-except ImportError:
-  class TimeoutExpired(Exception):
-    """Stub exception for Python 2."""
-try:
-  from shlex import quote  # Python 3.3+
-except ImportError:
-  from pipes import quote  # Python 2.7
+from subprocess import TimeoutExpired
+from shlex import quote
 
 from alibuild_helpers.log import debug, warning, dieOnError
 
-# Keep the linter happy
-if sys.version_info[0] >= 3:
-  basestring = str
-  unicode = None
-
-
-def is_string(s):
-  if sys.version_info[0] >= 3:
-    return isinstance(s, str)
-  return isinstance(s, basestring)
-
-
 def decode_with_fallback(data):
   """Try to decode DATA as utf-8; if that doesn't work, fall back to latin-1.
 
   This combination should cover every possible byte string, as latin-1 covers
   every possible single byte.
   """
-  if sys.version_info[0] >= 3:
-    if isinstance(data, bytes):
-      try:
-        return data.decode("utf-8")
-      except UnicodeDecodeError:
-        return data.decode("latin-1")
-    else:
-      return str(data)
-  elif isinstance(data, str):
-    return unicode(data, "utf-8")  # utf-8 is a safe assumption
-  elif not isinstance(data, unicode):
-    return unicode(str(data))
-  return data
+  if isinstance(data, bytes):
+    try:
+      return data.decode("utf-8")
+    except UnicodeDecodeError:
+      return data.decode("latin-1")
+  else:
+    return str(data)
 
 
 def getoutput(command, timeout=None):
   """Run command, check it succeeded, and return its stdout as a string."""
-  proc = Popen(command, shell=is_string(command), stdout=PIPE, stderr=PIPE)
+  proc = Popen(command, shell=isinstance(command, str), stdout=PIPE, stderr=PIPE)
   try:
     stdout, stderr = proc.communicate(timeout=timeout)
   except TimeoutExpired:
     warning("Process %r timed out; terminated", command)
     proc.terminate()
     stdout, stderr = proc.communicate()
-  except TypeError:
-    # On Python 2, we don't have the timeout= parameter. However, "regular"
-    # users shouldn't be running under Python 2 any more, so just don't timeout
-    # there and let the admins handle it manually.
-    stdout, stderr = proc.communicate()
   dieOnError(proc.returncode, "Command %s failed with code %d: %s" %
              (command, proc.returncode, decode_with_fallback(stderr)))
   return decode_with_fallback(stdout)
@@ -70,18 +39,13 @@ def getoutput(command, timeout=None):
 
 def getstatusoutput(command, timeout=None):
   """Run command and return its return code and output (stdout and stderr)."""
-  proc = Popen(command, shell=is_string(command), stdout=PIPE, stderr=STDOUT)
+  proc = Popen(command, shell=isinstance(command, str), stdout=PIPE, stderr=STDOUT)
   try:
     merged_output, _ = proc.communicate(timeout=timeout)
   except TimeoutExpired:
     warning("Process %r timed out; terminated", command)
     proc.terminate()
     merged_output, _ = proc.communicate()
-  except TypeError:
-    # On Python 2, we don't have the timeout= parameter. However, "regular"
-    # users shouldn't be running under Python 2 any more, so just don't timeout
-    # there and let the admins handle it manually.
-    merged_output, _ = proc.communicate()
   merged_output = decode_with_fallback(merged_output)
   # Strip a single trailing newline, if one exists, to match the behaviour of
   # subprocess.getstatusoutput.
@@ -91,7 +55,7 @@ def getstatusoutput(command, timeout=None):
 
 
 def execute(command, printer=debug, timeout=None):
-  popen = Popen(command, shell=is_string(command), stdout=PIPE, stderr=STDOUT)
+  popen = Popen(command, shell=isinstance(command, str), stdout=PIPE, stderr=STDOUT)
   start_time = time.time()
   for line in iter(popen.stdout.readline, b""):
     printer("%s", decode_with_fallback(line).strip("\n"))

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/git.py

@@ -46,8 +46,8 @@ class Git(SCM):
   def listRefsCmd(self, repository):
     return ["ls-remote", "--heads", "--tags", repository]
 
-  def cloneReferenceCmd(self, source, referenceRepo, usePartialClone):
-    cmd = ["clone", "--bare", source, referenceRepo]
+  def cloneReferenceCmd(self, spec, referenceRepo, usePartialClone):
+    cmd = ["clone", "--bare", spec, referenceRepo]
     if usePartialClone:
       cmd.extend(clone_speedup_options())
     return cmd
@@ -64,11 +64,11 @@ class Git(SCM):
       cmd.extend(clone_speedup_options())
     return cmd
 
-  def checkoutCmd(self, ref):
-    return ["checkout", "-f", ref]
+  def checkoutCmd(self, tag):
+    return ["checkout", "-f", tag]
 
-  def fetchCmd(self, source, *refs):
-    return ["fetch", "-f"] + clone_speedup_options() + [source, *refs]
+  def fetchCmd(self, remote, *refs):
+    return ["fetch", "-f"] + clone_speedup_options() + [remote, *refs]
 
   def setWriteUrlCmd(self, url):
     return ["remote", "set-url", "--push", "origin", url]

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/scm.py

@@ -21,7 +21,7 @@ class SCM(object):
     raise NotImplementedError
   def cloneReferenceCmd(self, spec, referenceRepo, usePartialClone):
     raise NotImplementedError
-  def cloneSourceCmd(self, spec, referenceRepo, usePartialClone):
+  def cloneSourceCmd(self, source, destination, referenceRepo, usePartialClone):
     raise NotImplementedError
   def setWriteUrlCmd(self, url):
     raise NotImplementedError

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/sync.py

@@ -373,7 +373,7 @@ class CVMFSRemoteSync:
     ))
 
   def upload_symlinks_and_tarball(self, spec):
-    dieOnError(self.writeStore, "CVMFS backend does not support uploading directly")
+    dieOnError(True, "CVMFS backend does not support uploading directly")
 
 class S3RemoteSync:
   """Sync package build directory from and to S3 using s3cmd.

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/utilities.py

@@ -10,14 +10,8 @@ import re
 import platform
 
 from datetime import datetime
-try:
-  from collections import OrderedDict
-except ImportError:
-  from ordereddict import OrderedDict
-try:
-  from shlex import quote  # Python 3.3+
-except ImportError:
-  from pipes import quote  # Python 2.7
+from collections import OrderedDict
+from shlex import quote
 
 from alibuild_helpers.cmd import getoutput
 from alibuild_helpers.git import git
@@ -430,6 +424,10 @@ def getPackageList(packages, specs, configDir, preferSystem, noSystem,
     filename = taps.get(pkg_filename, "%s/%s.sh" % (configDir, pkg_filename))
     err, spec, recipe = parseRecipe(getRecipeReader(filename, configDir))
     dieOnError(err, err)
+    # Unless there was an error, both spec and recipe should be valid.
+    # otherwise the error should have been caught above.
+    assert(spec is not None)
+    assert(recipe is not None)
     dieOnError(spec["package"].lower() != pkg_filename,
                "%s.sh has different package field: %s" % (p, spec["package"]))
 
@@ -490,12 +488,12 @@ def getPackageList(packages, specs, configDir, preferSystem, noSystem,
         else:
           # The check printed the name of a replacement; use it.
           key = match.group("key").strip()
-          try:
-            replacement = spec["prefer_system_replacement_specs"][key]
-          except KeyError:
-            dieOnError(True, "Could not find named replacement spec for "
-                       "%s: %s" % (spec["package"], key))
-          else:
+          replacement = None
+          for replacement_matcher in spec["prefer_system_replacement_specs"]:
+            if re.match(replacement_matcher, key):
+              replacement = spec["prefer_system_replacement_specs"][replacement_matcher]
+              break
+          if replacement:
             # We must keep the package name the same, since it is used to
             # specify dependencies.
             replacement["package"] = spec["package"]
@@ -503,6 +501,8 @@ def getPackageList(packages, specs, configDir, preferSystem, noSystem,
             # influence the package's hash, so allow the user to override it.
             replacement.setdefault("version", requested_version)
             spec = replacement
+            # Allows generalising the version based on the actual key provided
+            spec["version"] = spec["version"].replace("%(key)s", key)
             recipe = replacement.get("recipe", "")
             # If there's an explicitly-specified recipe, we're still building
             # the package. If not, aliBuild will still "build" it, but it's
@@ -512,6 +512,9 @@ def getPackageList(packages, specs, configDir, preferSystem, noSystem,
               ownPackages.add(spec["package"])
             else:
               systemPackages.add(spec["package"])
+          else:
+            warning(f"Could not find named replacement spec for {spec['package']}: {key}, "
+                    "falling back to building the package ourselves.")
 
     dieOnError(("system_requirement" in spec) and recipe.strip("\n\t "),
                "System requirements %s cannot have a recipe" % spec["package"])

{alibuild-1.17.11 → alibuild-1.17.13}/alibuild_helpers/workarea.py

@@ -4,10 +4,7 @@ import os
 import os.path
 import shutil
 import tempfile
-try:
-  from collections import OrderedDict
-except ImportError:
-  from ordereddict import OrderedDict
+from collections import OrderedDict
 
 from alibuild_helpers.log import dieOnError, debug, error
 from alibuild_helpers.utilities import call_ignoring_oserrors, symlink, short_commit_hash

{alibuild-1.17.11 → alibuild-1.17.13}/docs/mkdocs.yml

@@ -30,6 +30,7 @@ markdown_extensions:
       permalink: "#"
 
 plugins:
+  - search
   - redirects:
       redirect_maps:
         'o2-tutorial.html': 'https://alice-doc.github.io/alice-analysis-tutorial/building/'

{alibuild-1.17.11 → alibuild-1.17.13}/setup.py

@@ -76,8 +76,6 @@ setup(
     # aliBuild was built from.
     use_scm_version={'write_to': 'alibuild_helpers/_version.py'},
     setup_requires=[
-        # The 6.* series removed support for Python 2.7.
-        'setuptools_scm<6.0.0' if sys.version_info < (3, 0) else
         # The 7.* series removed support for Python 3.6.
         'setuptools_scm<7.0.0' if sys.version_info < (3, 7) else
         'setuptools_scm'

alibuild-1.17.13/templates/alibuild_to_please.jnj

@@ -0,0 +1,63 @@
+# A alibuild template to convert the alidist recipes to
+# the please build system. While not fully functional, it serves
+# as explanatory example on how one can use templates to
+# extract informations from alidist, without having to instrument
+# alibuild itself.
+#
+# Use with:
+#
+# cat alibuild_to_please.jnj | python3 alibuild/aliBuild build lz4 --defaults o2 --debug --no-system --plugin templating --no-system --no-local alibuild-recipe-tools > opendata/BUILD.plz
+#
+{% for name, spec in specs.items() %}
+
+{% if spec.source is defined %}
+# {{ spec.source is defined  }}
+new_http_archive(
+  name = "download_{{name}}",
+  urls = ["{{spec.source}}/archive/refs/tags/{{spec.tag}}.tar.gz"],
+  )
+{% endif %}
+
+genrule(
+    name = "build_{{name}}",
+    output_dirs = ["_out"],
+    outs = [
+{% if spec.tools is defined %}
+{% for tool in spec.tools%}
+      "_out/{{tool}}",
+{% endfor %}
+{% endif %}
+{% if spec.tools is defined %}
+{% for lib in spec.libs%}
+      "_out/{{lib}}",
+{% endfor %}
+{% endif %}
+    ],
+{% if spec.source is defined %}
+    srcs = [":download_{{name}}"],
+{% endif %}
+    deps = [
+{% for build_require_name in spec.build_requires %}
+      ":build_{{build_require_name}}",{% endfor %}
+    ],
+    cmd = """
+      PKGNAME="{{name}}"
+      PKGVERSION="{{spec.version}}"
+      PKGREVISION="{{spec.revision}}"
+      PKGHASH="{{spec.package_hash}}"
+{% if spec.source is defined %}
+      SOURCEDIR="$PWD/$SRCS/*"
+{% endif %}
+      INSTALLROOT="$PWD/_out"
+      BUILDDIR="$PWD/_build"
+      JOBS="10"
+      mkdir -p $BUILDDIR
+      pushd $BUILDDIR
+      set -e
+      {{ spec.recipe | replace('\$', '\\\\\\\\$')}}
+      mkdir -p $INSTALLROOT
+      touch $INSTALLROOT/.please-did-build
+      popd
+    """,
+  )
+{% endfor %}

{alibuild-1.17.11 → alibuild-1.17.13}/tests/test_args.py

@@ -1,11 +1,7 @@
 from __future__ import print_function
 # Assuming you are using the mock library to ... mock things
-try:
-    from unittest import mock
-    from unittest.mock import patch, call  # In Python 3, mock is built-in
-except ImportError:
-    import mock
-    from mock import patch, call  # Python 2
+from unittest import mock
+from unittest.mock import patch, call
 
 import alibuild_helpers.args
 from alibuild_helpers.args import doParseArgs, matchValidArch, finaliseArgs, DEFAULT_WORK_DIR, DEFAULT_CHDIR, ARCHITECTURE_TABLE
@@ -16,12 +12,8 @@ import os.path
 import unittest
 import shlex
 
-if (sys.version_info[0] >= 3):
-  BUILD_MISSING_PKG_ERROR = "the following arguments are required: PACKAGE"
-  ANALYTICS_MISSING_STATE_ERROR = "the following arguments are required: state"
-else:
-  BUILD_MISSING_PKG_ERROR = "too few arguments"
-  ANALYTICS_MISSING_STATE_ERROR = "too few arguments"
+BUILD_MISSING_PKG_ERROR = "the following arguments are required: PACKAGE"
+ANALYTICS_MISSING_STATE_ERROR = "the following arguments are required: state"
 
 # A few errors we should handle, together with the expected result
 ARCHITECTURE_ERROR = [call(u"Unknown / unsupported architecture: foo.\n\n{table}Alternatively, you can use the `--force-unknown-architecture' option.".format(table=ARCHITECTURE_TABLE))]