@chuckatkins Please review.

Doesn't this require a cuda dependency?

This is done by CudaPackage isn't it?

Also, can we merge the python and python3 variants?

This is what we tried first but we had a hard time handling the conditions involved - for instance conditionally using different matplotlib depending on python 2 or 3 or using python3 only for newer paraview and using python2 for older ones. We kept running in spack bugs when trying this.

#11485

Can you suggest a package that does something similar?

This is done by CudaPackage isn't it?

Ah, you're right, completely missed that.

This is what we tried first but we had a hard time handling the conditions involved - for instance conditionally using different matplotlib depending on python 2 or 3 or using python3 only for newer paraview and using python2 for older ones. We kept running in spack bugs when trying this.

I understand. This sounds like a bug in the concretizer that @tgamblin is working on fixing. Let's leave things alone for now and return once a smarter concretizer exists.