From 7d9f2bf4ed89823b4f6c515b7f4e2994bf58c3a9 Mon Sep 17 00:00:00 2001 From: "Adam J. Stewart" Date: Thu, 3 Sep 2020 17:31:00 -0500 Subject: depends_on cannot handle ^ sigil (#18220) * depends_on cannot handle ^ sigil * cardioid+mfem+cuda requires hypre+cuda * Document this limitation * Move warning message to Known Issues docs * Better handling of parmetis dep --- lib/spack/docs/known_issues.rst | 29 ++++++++++++++++++++++++++++- 1 file changed, 28 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/spack/docs/known_issues.rst b/lib/spack/docs/known_issues.rst index 531b3bcc5a..accec2422b 100644 --- a/lib/spack/docs/known_issues.rst +++ b/lib/spack/docs/known_issues.rst @@ -14,7 +14,7 @@ problems if you encounter them. Variants are not properly forwarded to dependencies --------------------------------------------------- -**Status:** Expected to be fixed in the next release +**Status:** Expected to be fixed by Spack's new concretizer Sometimes, a variant of a package can also affect how its dependencies are built. For example, in order to build MPI support for a package, it may @@ -48,3 +48,30 @@ A workaround is to explicitly activate the variants of dependencies as well: See https://github.com/spack/spack/issues/267 and https://github.com/spack/spack/issues/2546 for further details. + +----------------------------------------------- +depends_on cannot handle recursive dependencies +----------------------------------------------- + +**Status:** Not yet a work in progress + +Although ``depends_on`` can handle any aspect of Spack's spec syntax, +it currently cannot handle recursive dependencies. If the ``^`` sigil +appears in a ``depends_on`` statement, the concretizer will hang. +For example, something like: + +.. code-block:: python + + depends_on('mfem+cuda ^hypre+cuda', when='+cuda') + + +should be rewritten as: + +.. code-block:: python + + depends_on('mfem+cuda', when='+cuda') + depends_on('hypre+cuda', when='+cuda') + + +See https://github.com/spack/spack/issues/17660 and +https://github.com/spack/spack/issues/11160 for more details. -- cgit v1.2.3-60-g2f50