diff options
author | Martin Aumüller <aumuell@reserv.at> | 2023-08-02 06:21:47 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-08-01 21:21:47 -0700 |
commit | 5d8482598b9210507901dfefe8e7250d0b1f231b (patch) | |
tree | b937561a554b6b1f42ef97b4526d8bfd583d4b96 | |
parent | f079e7fc34f05516e943c03369ad92b256593c68 (diff) | |
download | spack-5d8482598b9210507901dfefe8e7250d0b1f231b.tar.gz spack-5d8482598b9210507901dfefe8e7250d0b1f231b.tar.bz2 spack-5d8482598b9210507901dfefe8e7250d0b1f231b.tar.xz spack-5d8482598b9210507901dfefe8e7250d0b1f231b.zip |
dcmtk: new version and pic configuration (#39161)
* dcmtk: checksum 3.6.7
compiles on macos
* dcmtk: support pic configuration
* [@spackbot] updating style on behalf of aumuell
* dcmtk: use define_from_variant for shorter code
* dcmtk: refine conflict
it appears that dcmtk < 3.6.7 only fails on macos/aarch64:
/Library/Developer/CommandLineTools/usr/lib/clang/14.0.3/include/xmmintrin.h:14:2:
error: "This header is only meant to be used on x86 and x64 architecture"
---------
Co-authored-by: aumuell <aumuell@users.noreply.github.com>
-rw-r--r-- | var/spack/repos/builtin/packages/dcmtk/package.py | 23 |
1 files changed, 15 insertions, 8 deletions
diff --git a/var/spack/repos/builtin/packages/dcmtk/package.py b/var/spack/repos/builtin/packages/dcmtk/package.py index 64efc612c9..96c1870cdd 100644 --- a/var/spack/repos/builtin/packages/dcmtk/package.py +++ b/var/spack/repos/builtin/packages/dcmtk/package.py @@ -13,6 +13,7 @@ class Dcmtk(CMakePackage): homepage = "https://dicom.offis.de" url = "https://github.com/DCMTK/dcmtk/archive/DCMTK-3.6.3.tar.gz" + version("3.6.7", sha256="17705dcdb2047d1266bb4e92dbf4aa6d4967819e8e3e94f39b7df697661b4860") version("3.6.6", sha256="117097da6d50ddbad0e48bb1e6cdc61468e82ba1d32001dd8e2366b445133a8c") version("3.6.5", sha256="37dad355d5513b4de4a86b5b7b0c3e9ec059860d88781b80916bba2a04e6d5b8") version("3.6.4", sha256="e4b1de804a3fef38fe8cb9edd00262c3cbbd114b305511c14479dd888a9337d2") @@ -41,9 +42,12 @@ class Dcmtk(CMakePackage): variant("iconv", default=True, description="Charset conversion support (iconv)") depends_on("iconv", type=("build", "link")) + variant("pic", default=False, description="Produce position-independent code") variant("cxx11", default=False, description="Enable c++11 features") variant("stl", default=True, description="Use native STL implementation") + conflicts("platform=darwin target=aarch64:", when="@:3.6.6") + def patch(self): # Backport 3.6.4 if self.spec.satisfies("@:3.6.3 %fj"): @@ -55,12 +59,15 @@ class Dcmtk(CMakePackage): ) def cmake_args(self): - args = ["-DDCMTK_WITH_OPENSSL={0}".format("ON" if "+ssl" in self.spec else "OFF")] - args += ["-DDCMTK_WITH_ZLIB={0}".format("ON" if "+zlib" in self.spec else "OFF")] - args += ["-DDCMTK_WITH_TIFF={0}".format("ON" if "+tiff" in self.spec else "OFF")] - args += ["-DDCMTK_WITH_PNG={0}".format("ON" if "+png" in self.spec else "OFF")] - args += ["-DDCMTK_WITH_XML={0}".format("ON" if "+xml" in self.spec else "OFF")] - args += ["-DDCMTK_WITH_ICONV={0}".format("ON" if "+iconv" in self.spec else "OFF")] - args += ["-DDCMTK_ENABLE_CXX11={0}".format("ON" if "+cxx11" in self.spec else "OFF")] - args += ["-DDCMTK_ENABLE_STL={0}".format("ON" if "+stl" in self.spec else "OFF")] + args = [ + self.define_from_variant("DCMTK_WITH_OPENSSL", "ssl"), + self.define_from_variant("DCMTK_WITH_ZLIB", "zlib"), + self.define_from_variant("DCMTK_WITH_TIFF", "tiff"), + self.define_from_variant("DCMTK_WITH_PNG", "png"), + self.define_from_variant("DCMTK_WITH_XML", "xml"), + self.define_from_variant("DCMTK_WITH_ICONV", "iconv"), + self.define_from_variant("DCMTK_ENABLE_CXX11", "cxx11"), + self.define_from_variant("DCMTK_ENABLE_STL", "stl"), + self.define_from_variant("CMAKE_POSITION_INDEPENDENT_CODE", "pic"), + ] return args |