From 165e6bfe5fb3ce327315c698a3a275deda9e6d35 Mon Sep 17 00:00:00 2001 From: Mark Grondona Date: Wed, 1 Nov 2017 08:53:37 -0700 Subject: flux: update flux-core to v0.8.0 (#5914) * dockbook: remove update to dependent environment Remove override of setup_dependent_environment in both docbook-xml and docbook-xsl, which seems to generate a warning from spack about "Suspicious requests to set or unset 'XML_CATLOG_FILES' found." * flux: update version/dependencies to 0.8.0 Update flux-core to v0.8.0. Clean up dependencies and remove unneeded extra invocation of ./autogen.sh when building from @master. * flux: modify environment for Lua tests Modify Spack environment to add `./?.lua` to LUA_PATH so that fluxometer.lua can be found by tests during `make check` run from `spack install --tests all flux` * flux: fix broken flux version when on master When building @master with spack, the flux version test from `make check` fails because spack is using a shallow clone. Add a setup section for @master that calls git pull --depth=50 --tags to pull down tags and (hopefully) enough information for git-describe to work properly. --- .../repos/builtin/packages/docbook-xml/package.py | 4 ---- .../repos/builtin/packages/docbook-xsl/package.py | 4 ---- var/spack/repos/builtin/packages/flux/package.py | 25 ++++++++++++++++------ 3 files changed, 19 insertions(+), 14 deletions(-) diff --git a/var/spack/repos/builtin/packages/docbook-xml/package.py b/var/spack/repos/builtin/packages/docbook-xml/package.py index a3d4b0b1b0..c6bb232d78 100644 --- a/var/spack/repos/builtin/packages/docbook-xml/package.py +++ b/var/spack/repos/builtin/packages/docbook-xml/package.py @@ -42,10 +42,6 @@ class DocbookXml(Package): else: install(src, dst) - def setup_dependent_environment(self, spack_env, run_env, dependent_spec): - catalog = os.path.join(self.spec.prefix, 'catalog.xml') - spack_env.set('XML_CATALOG_FILES', catalog, separator=' ') - def setup_environment(self, spack_env, run_env): catalog = os.path.join(self.spec.prefix, 'catalog.xml') run_env.set('XML_CATALOG_FILES', catalog, separator=' ') diff --git a/var/spack/repos/builtin/packages/docbook-xsl/package.py b/var/spack/repos/builtin/packages/docbook-xsl/package.py index b0d5df5268..f005ad5ff5 100644 --- a/var/spack/repos/builtin/packages/docbook-xsl/package.py +++ b/var/spack/repos/builtin/packages/docbook-xsl/package.py @@ -44,10 +44,6 @@ class DocbookXsl(Package): else: install(src, dst) - def setup_dependent_environment(self, spack_env, run_env, dependent_spec): - catalog = os.path.join(self.spec.prefix, 'catalog.xml') - spack_env.set('XML_CATALOG_FILES', catalog, separator=' ') - def setup_environment(self, spack_env, run_env): catalog = os.path.join(self.spec.prefix, 'catalog.xml') run_env.set('XML_CATALOG_FILES', catalog, separator=' ') diff --git a/var/spack/repos/builtin/packages/flux/package.py b/var/spack/repos/builtin/packages/flux/package.py index 39956fc10c..ace8609b05 100644 --- a/var/spack/repos/builtin/packages/flux/package.py +++ b/var/spack/repos/builtin/packages/flux/package.py @@ -30,38 +30,51 @@ class Flux(AutotoolsPackage): """ A next-generation resource manager (pre-alpha) """ homepage = "https://github.com/flux-framework/flux-core" - url = "https://github.com/flux-framework/flux-core/releases/download/v0.6.0/flux-core-0.6.0.tar.gz" + url = "https://github.com/flux-framework/flux-core/releases/download/v0.8.0/flux-core-0.8.0.tar.gz" - version('0.6.0', md5='d44a0f719744771d168edd205bd8e74e') + version('0.8.0', md5='9ee12974a8b2ab9a30533f69826f3bec') version('master', branch='master', git='https://github.com/flux-framework/flux-core') variant('docs', default=True, description='Build flux manpages') - # Also needs autotools, but should use the system version if available depends_on("zeromq@4.0.4:") depends_on("czmq@2.2:") depends_on("hwloc") depends_on("lua@5.1:5.1.99") + depends_on("lua-luaposix") depends_on("munge") - depends_on("json-c") - depends_on("libxslt") + depends_on("libuuid") depends_on("python") depends_on("py-cffi", type=('build', 'run')) depends_on("jansson") depends_on("asciidoc", type='build', when="+docs") + # Need autotools when building on master: depends_on("autoconf", type='build', when='@master') depends_on("automake", type='build', when='@master') depends_on("libtool", type='build', when='@master') + def setup(): + pass + + @when('@master') + def setup(self): + # Allow git-describe to get last tag so flux-version works: + git = which('git') + git('pull', '--depth=50', '--tags') + def autoreconf(self, spec, prefix): + self.setup() if os.path.exists('autogen.sh'): # Bootstrap with autotools bash = which('bash') bash('./autogen.sh') - bash('./autogen.sh') # yes, twice, intentionally + + def setup_environment(self, spack_env, run_env): + # Ensure ./fluxometer.lua can be found during flux's make check + spack_env.append_path('LUA_PATH', './?.lua', separator=';') def configure_args(self): return ['--disable-docs'] if '+docs' not in self.spec else [] -- cgit v1.2.3-70-g09d2