Open jonasjucker opened 2 months ago
PR Preview Action v1.4.7
:---:
:rocket: Deployed preview to https://C2SM.github.io/spack-c2sm/pr-preview/pr-992/
on branch gh-pages
at 2024-08-22 08:00 UTC
launch jenkins all
Interesting! Can you upstream it to https://github.com/spack/spack too?
Test | |
---|---|
:green_circle: | summary |
Interesting! Can you upstream it to https://github.com/spack/spack too?
I was thinking about that. Looking at the package in spack it seems that only configure-option are included in the recipe.
Since there is no --enable-nvtx
as configure-option I would not clutter the recipe.
But it's okay to clutter ours?
I think so, since we will use it and also know how to use it :-)
Also this is alligned with our goal to build a hierarchy of icon-recipes we would go from generic (spack recipe) nwp (spack-c2sm recipe) ...
It's okay to clutter software that MeteoSwiss' operations rely on, even though we don't think it's good enough for the public icon community? I don't understand this logic.
Ok then you don't
This variant enables to use nvtx ranges for profiling in a clean way.