diff options
| author | Glenn Johnson <glenn-johnson@uiowa.edu> | 2021-06-08 04:08:10 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-06-08 03:08:10 -0600 |
| commit | 418db4e9106371976d0ac3d95b4b4bf5adf3ffc8 (patch) | |
| tree | 0be718ac3d92a06fb0f831580cfbc488a295d044 | |
| parent | f231ae97f4bd976815f3b395572ea940784e8b48 (diff) | |
| download | spack-418db4e9106371976d0ac3d95b4b4bf5adf3ffc8.tar.gz spack-418db4e9106371976d0ac3d95b4b4bf5adf3ffc8.tar.bz2 spack-418db4e9106371976d0ac3d95b4b4bf5adf3ffc8.tar.xz spack-418db4e9106371976d0ac3d95b4b4bf5adf3ffc8.zip | |
gatk: make r a variant (#24189)
According to the docs, r is needed for plotting, but plotting is
untested. In addition, the specific version requirement of java for gatk
could lead to multiple installations of r being triggered in an
environment. That might cause people to have to be deliberate about
java in a deployment. All in all, it seems that r is better as a
variant for gatk.
| -rw-r--r-- | var/spack/repos/builtin/packages/gatk/package.py | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/var/spack/repos/builtin/packages/gatk/package.py b/var/spack/repos/builtin/packages/gatk/package.py index f1a0511b70..f83ead3d2d 100644 --- a/var/spack/repos/builtin/packages/gatk/package.py +++ b/var/spack/repos/builtin/packages/gatk/package.py @@ -89,9 +89,20 @@ class Gatk(Package): url="https://storage.googleapis.com/gatk-software/package-archive/gatk/GenomeAnalysisTK-3.8-0-ge9d806836.tar.bz2" ) + # Make r a variant. According to the gatk docs it is not essential and not + # tested. + # https://github.com/broadinstitute/gatk#R + # Using R to generate plots + # Certain GATK tools may optionally generate plots using the R installation + # provided within the conda environment. If you are uninterested in plotting, + # R is still required by several of the unit tests. Plotting is currently + # untested and should be viewed as a convenience rather than a primary + # output. + variant('r', default=False, description='Use R for plotting') + depends_on("java@8", type="run") depends_on("python@2.6:2.8,3.6:", type="run", when="@4.0:") - depends_on("r@3.2:", type="run", when="@4.0:") + depends_on("r@3.2:", type="run", when="@4.0: +r") def install(self, spec, prefix): mkdirp(prefix.bin) |