Fix: Remove serialbox as permanent dep of ICON (#1031)

* only make serialbox a dep when needed * GitHub Action: Apply Pep8-formatting --------- Co-authored-by: github-actions <[email protected]>
C2SM · Nov 19, 2024 · 666babb · 666babb
1 parent a239856
commit 666babb
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/repos/c2sm/packages/icon/package.py b/repos/c2sm/packages/icon/package.py
@@ -226,10 +226,12 @@ class Icon(AutotoolsPackage, CudaPackage):
     depends_on('libxml2', when='+art')
 
     for x in serialization_values:
-        depends_on('serialbox+fortran', when='serialization={0}'.format(x))
+
         # WORKAROUND: A build and link dependency should imply that the same compiler is used. This enforces it.
-        depends_on('serialbox %nvhpc', when='%nvhpc')
-        depends_on('serialbox %gcc', when='%gcc')
+        depends_on('serialbox+fortran %nvhpc',
+                   when='serialization={0} %nvhpc'.format(x))
+        depends_on('serialbox+fortran %gcc',
+                   when='serialization={0} %gcc'.format(x))
 
     depends_on('eccodes +fortran', when='+emvorado')
     depends_on('eccodes', when='+grib2')