Skip to content

New package: opensubdiv#15310

Merged
adamjstewart merged 18 commits intospack:developfrom
qwertos:qwertos/opensubdiv
Mar 5, 2020
Merged

New package: opensubdiv#15310
adamjstewart merged 18 commits intospack:developfrom
qwertos:qwertos/opensubdiv

Conversation

@qwertos
Copy link
Copy Markdown
Contributor

@qwertos qwertos commented Mar 3, 2020

No description provided.

url = "https://github.com/PixarAnimationStudios/OpenSubdiv/archive/v3_4_0.tar.gz"
git = "https://github.com/PixarAnimationStudios/OpenSubdiv"

version('dev', branch='dev')
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 should call this version develop if you want it to sort properly

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed


variant('tbb', default=False, description='Builds with Intel TBB support')
variant('openmp', default=False, description='Builds with OpenMP support')
variant('doc', default=False, description='Builds documentation. Requires python@@2.6')
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.

Why the double @?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@ needs to be escaped with @@ else ==> Error: Incomplete color format: '@ in [email protected] for command spack info opensubdiv

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.

Hmm, that's probably a bug. Want to open up a separate issue for this? In the meantime, you can just change it to "Python 2", or not mention the dependency at all.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

fixed

@adamjstewart adamjstewart merged commit 9940392 into spack:develop Mar 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants