Skip to content

Variants and fixes to thrift package#513

Merged
tgamblin merged 2 commits intospack:developfrom
alfredo-gimenez:develop
Mar 10, 2016
Merged

Variants and fixes to thrift package#513
tgamblin merged 2 commits intospack:developfrom
alfredo-gimenez:develop

Conversation

@alfredo-gimenez
Copy link
Copy Markdown
Contributor

No description provided.


# Variant dependencies
extends('python', when='+python')
depends_on('python', when='+python')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can get rid of this line. According to the Spack documentation:

You can call either depends_on or extends on any one package, but not both. For example you cannot both depends_on('python') and extends(python) in the same package. extends implies depends_on.

tgamblin added a commit that referenced this pull request Mar 10, 2016
Variants and fixes to thrift package
@tgamblin tgamblin merged commit ad66254 into spack:develop Mar 10, 2016
matz-e added a commit to matz-e/spack that referenced this pull request Apr 27, 2020
Co-Authored-By: Matthias Wolf <[email protected]>
Comment on lines +46 to +50
options.append('--with-java=%s' % ('yes' if '+java' in spec else 'no'))
options.append('--with-go=%s' % ('yes' if '+go' in spec else 'no'))
options.append('--with-lua=%s' % ('yes' if '+lua' in spec else 'no'))
options.append('--with-php=%s' % ('yes' if '+php' in spec else 'no'))
options.append('--with-qt4=%s' % ('yes' if '+qt4' in spec else 'no'))
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tgamblin @alfredo-gimenez I don't see variants for these options. Am I missing something here?

climbfuji added a commit to climbfuji/spack that referenced this pull request Feb 4, 2025
release/1.9.0: Bug fix in wgrib2: apply '-Wno-error=implicit-function-declaration' for LLVM clang
gartung pushed a commit to gartung/spack that referenced this pull request Apr 25, 2025
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.

5 participants