add build recipe for tempestremap library #4174
Conversation
jakebolewski
force-pushed
the
add-tempestremap
branch
from
34a2331 to
05d028a
Compare
| platforms = [ | ||
| Platform("x86_64", "linux"), | ||
| Platform("aarch64", "linux"; libc="glibc"), | ||
| Platform("x86_64", "macos"), |
There was a problem hiding this comment.
Can we also support Platform("aarch64", "macos")?
There was a problem hiding this comment.
Does NetCDF_jll work on aarch64? if it does then yes
There was a problem hiding this comment.
Doesn't look like it? https://github.com/JuliaBinaryWrappers/NetCDF_jll.jl#platforms
There was a problem hiding this comment.
there is no end-date to a long time I guess.... #567
jakebolewski
force-pushed
the
add-tempestremap
branch
from
05d028a to
0710d81
Compare
|
Generally following the outline of the NetCDF_jll recipe made this work for linking the runtime shared lib dependencies, good to know given the prevalence of dependent NetCDF software for this project in the future. |
jakebolewski
force-pushed
the
add-tempestremap
branch
from
0710d81 to
9fa0235
Compare
|
libblastrampoline worked out of the box, while OpenBLAS required me to re-write the autotools blas/lapack file which is why I choose it. Given the model is going to support 1.7+ this seems a reasonable starting point which can be later relaxed. |
| Dependency("HDF5_jll"), | ||
| # The following is adapted from NetCDF_jll | ||
| Dependency("LibCURL_jll"), | ||
| # The following libraries are dependencies of LibCURL_jll which is now a |
There was a problem hiding this comment.
Are you sure this is still needed? I thought this had been fixed, but I'm not 100% sure
There was a problem hiding this comment.
I can try again locally without the libcurl deps, but adding these fixed all the issues I had.
TempestRemap is a conservative, consistent and monotone remapping package for arbitrary grid geometry.
https://github.com/ClimateGlobalChange/tempestremap