Skip to content

First fix for SPACK_DEPENDENCIES problem when doing setup#10715

Merged
becker33 merged 4 commits intospack:developfrom
cedricchevalier19:bug-dependencies
Jun 26, 2020
Merged

First fix for SPACK_DEPENDENCIES problem when doing setup#10715
becker33 merged 4 commits intospack:developfrom
cedricchevalier19:bug-dependencies

Conversation

@cedricchevalier19
Copy link
Copy Markdown
Contributor

Tentative Fix for #10714.

@becker33
Copy link
Copy Markdown
Member

If you change SPACK_TRANSITIVE_INCLUDE_DIRS to SPACK_INCLUDE_DIRS everywhere you should be able to have the same effect without losing functionality, and you can still delete the function causing the problem. Spack computes in other places the same thing that function was designed to compute.

@germasch
Copy link
Copy Markdown
Contributor

So I'm not sure what the right fix is, either, but i'm hitting the same problem, ie., spack setup <pkg> not working because it'll complain about SPACK_DEPENDENCIES (and not with a very helpful error message).

==> Generating spconfig.py [camtimers@local%[email protected] build_type=RelWithDebInfo arch=linux-ubuntu18.04-haswell/prvk35b]
==> Error: 'SPACK_DEPENDENCIES'

FWIW, a workaround is to export SPACK_DEPENDENCIES="" first. But it looks like SPACK_DEPENDENCIES has been removed a long time ago except in this one place. I don't know what the correct fix is -- this one here looks like it'll fix the problem for me. I could also instead make a PR that just handles SPACK_DEPENDENCIES not being set in the environment gracefully, though that seems more like a workaround rather than a fix.

@becker33
Copy link
Copy Markdown
Member

@cedricchevalier19 @germasch I suggest replacing SPACK_INCLUDE_DIRS with SPACK_TRANSITIVE_INCLUDE_DIRS everywhere it appears in $spack/lib/spack/spack/cmd/setup.py, in addition to the changes already in this branch. If you can push that to this branch, I can review it and get it merged.

@cedricchevalier19
Copy link
Copy Markdown
Contributor Author

cedricchevalier19 commented Jun 17, 2020

Now spack setup properly exports SPACK_INCLUDE_DIRS into spconfig.py.

@becker33 becker33 merged commit 529c659 into spack:develop Jun 26, 2020
@cedricchevalier19 cedricchevalier19 deleted the bug-dependencies branch June 30, 2020 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants