Skip to content

Update for environment-modules.#2279

Merged
tgamblin merged 1 commit intospack:developfrom
skosukhin:env_mod_pr
Nov 8, 2016
Merged

Update for environment-modules.#2279
tgamblin merged 1 commit intospack:developfrom
skosukhin:env_mod_pr

Conversation

@skosukhin
Copy link
Copy Markdown
Member

There is a chance that tclConfig.sh that comes with the tcl package is saved not in the 'lib' directory but in the 'lib64' directory. The environment-modules package takes that into account now.

Added 'lib64' directory as a possible option where tclConfig.sh file can be found.
@davydden
Copy link
Copy Markdown
Member

davydden commented Nov 8, 2016

funny enough, we just discussed it with @christianbaensch who came across the same issue on Suse, although we though about much simple solution using if os.path.isdir(join_path(tcl_spec.prefix, 'lib')) to choose between the two directories.

@davydden davydden added bug Something isn't working new-package ready labels Nov 8, 2016
@skosukhin
Copy link
Copy Markdown
Member Author

@davydden in my case the directory 'lib' also exists (if I understand your solution correctly) but the script is not there.

@tgamblin tgamblin merged commit 4457c6a into spack:develop Nov 8, 2016
@skosukhin skosukhin deleted the env_mod_pr branch November 8, 2016 16:02
becker33 pushed a commit that referenced this pull request Nov 22, 2016
Added 'lib64' directory as a possible option where tclConfig.sh file can be found.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working new-package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants