summaryrefslogtreecommitdiff
path: root/var/spack/repos/builtin/packages/aspcud/package.py
diff options
context:
space:
mode:
authorAbhik Sarkar <62109745+asarkar-parsys@users.noreply.github.com>2022-03-17 16:42:07 -0700
committerGitHub <noreply@github.com>2022-03-17 17:42:07 -0600
commit0ce8b9d3983acaf2469b3b10af8bd1b72798026b (patch)
tree080b8498f9bbe8777be3279dd3a6a385b3396adb /var/spack/repos/builtin/packages/aspcud/package.py
parentf8f4aafe8134a859c751ea6a7698aa936eed48a6 (diff)
downloadspack-0ce8b9d3983acaf2469b3b10af8bd1b72798026b.tar.gz
spack-0ce8b9d3983acaf2469b3b10af8bd1b72798026b.tar.bz2
spack-0ce8b9d3983acaf2469b3b10af8bd1b72798026b.tar.xz
spack-0ce8b9d3983acaf2469b3b10af8bd1b72798026b.zip
Make boost minimal and composable (Original PR#22303) (#28623)
* Make boost composable Currently Boost enables a few components through variants by default, which means that if you want to use only what you need and no more, you have to explicitly disable these variants, leading to concretization errors whenever a second package explicitly needs those components. For instance if package A only needs `+component_a` it might depend on `boost +component_a ~component_b`. And if packge B only needs `+component_b` it might depend on `boost ~component_a +component_b`. If package C now depends on both A and B, this leads to unsatisfiable variants and hence a concretization error. However, if we default to disabling all components, package A can simply depend on `boost +component_a` and package B on `boost +component_b` and package C will concretize to depending on `boost +component_a +component_b`, and whatever you install, you get the bare minimum. * Fix style * Added composable boost dependencies for folly * fixing akantu merge issue * hpctoolkit boost dependencies already defined * Fix Styles * Fixup style once more * Adding isort fix * isort one more time * Fix for package audit issue Co-authored-by: Harmen Stoppels <harmenstoppels@gmail.com> Co-authored-by: Ryan O'Malley <rd.omalley@comcast.net>
Diffstat (limited to 'var/spack/repos/builtin/packages/aspcud/package.py')
-rw-r--r--var/spack/repos/builtin/packages/aspcud/package.py6
1 files changed, 6 insertions, 0 deletions
diff --git a/var/spack/repos/builtin/packages/aspcud/package.py b/var/spack/repos/builtin/packages/aspcud/package.py
index 920fe4d071..6422dc7771 100644
--- a/var/spack/repos/builtin/packages/aspcud/package.py
+++ b/var/spack/repos/builtin/packages/aspcud/package.py
@@ -4,6 +4,7 @@
# SPDX-License-Identifier: (Apache-2.0 OR MIT)
from spack import *
+from spack.pkg.builtin.boost import Boost
class Aspcud(CMakePackage):
@@ -22,6 +23,11 @@ class Aspcud(CMakePackage):
version('1.9.4', sha256='3645f08b079e1cc80e24cd2d7ae5172a52476d84e3ec5e6a6c0034492a6ea885')
depends_on('boost@1.74:', type=('build'), when='@1.9.5:')
+
+ # TODO: replace this with an explicit list of components of Boost,
+ # for instance depends_on('boost +filesystem')
+ # See https://github.com/spack/spack/pull/22303 for reference
+ depends_on(Boost.with_default_variants, type=('build'), when='@1.9.5:')
depends_on('cmake', type=('build'))
depends_on('re2c', type=('build'))
depends_on('clingo')