Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enable plugin install #172

Merged
merged 12 commits into from
Mar 29, 2023
Merged

Enable plugin install #172

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
3e19792
Enable plugin install
Mar 27, 2023
97d5db1
Bump build number
Mar 27, 2023
e9d6927
Store artifacts for debugging
Mar 27, 2023
f79531c
Set HDF5_PLUGIN_PATH
Mar 27, 2023
ebfd226
MNT: Re-rendered with conda-build 3.23.3, conda-smithy 3.23.1, and co…
Mar 27, 2023
0a19f0e
Fix quoting problem
Mar 27, 2023
f5db371
Explicitly set plugin install dir
Mar 27, 2023
4266a27
Add blosc
Mar 27, 2023
bceeea1
Use clang-cpp on osx
Mar 29, 2023
b34ae05
Activate plugin installation on Windows
Mar 29, 2023
c78591a
Remove artifact storage
Mar 29, 2023
c2bec50
MNT: Re-rendered with conda-build 3.23.3, conda-smithy 3.23.1, and co…
Mar 29, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion recipe/bld.bat
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ rem manually specify hdf5 paths to work-around https://github.com/Unidata/netcdf
cmake -LAH -G "NMake Makefiles" ^
-DCMAKE_INSTALL_PREFIX="%LIBRARY_PREFIX%" ^
-DBUILD_SHARED_LIBS=ON ^
-DPLUGIN_INSTALL_DIR=YES ^
-DENABLE_TESTS=ON ^
-DENABLE_HDF4=ON ^
-DCMAKE_PREFIX_PATH="%LIBRARY_PREFIX%" ^
Expand Down Expand Up @@ -42,4 +43,3 @@ goto end_tests
:vc9_tests
ctest -VV
:end_tests

6 changes: 6 additions & 0 deletions recipe/build.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,13 +47,17 @@ if [[ ${HOST} =~ .*darwin.* ]]; then
# (symbol in a section other than those above according to man nm), instead though
# or to fix ld64 so that it checks for symbols being used in this section).
export LDFLAGS=$(echo "${LDFLAGS}" | sed "s/-Wl,-dead_strip_dylibs//g")
export HDF5_PLUGIN_PATH=$(echo "H5_DEFAULT_PLUGINDIR" | clang-cpp -P -include $PREFIX/include/H5pubconf.h - | tr -d '"')
else
export HDF5_PLUGIN_PATH=$(echo "H5_DEFAULT_PLUGINDIR" | $CPP -P -include $PREFIX/include/H5pubconf.h - | tr -d '"')
fi

# 2022/04/25
# DAP Remote tests are causing spurious failures at the momment
# https://github.com/Unidata/netcdf-c/issues/2188#issuecomment-1015927961
# -DENABLE_DAP_REMOTE_TESTS=OFF


mkdir build-shared
cd build-shared
# Build shared.
Expand All @@ -65,6 +69,8 @@ cmake ${CMAKE_ARGS} \
-DENABLE_HDF4=ON \
-DENABLE_NETCDF_4=ON \
-DBUILD_SHARED_LIBS=ON \
-DENABLE_PLUGIN_INSTALL=ON \
-DPLUGIN_INSTALL_DIR=${HDF5_PLUGIN_PATH} \
-DENABLE_TESTS=ON \
-DBUILD_UTILITIES=ON \
-DENABLE_DOXYGEN=OFF \
Expand Down
3 changes: 2 additions & 1 deletion recipe/meta.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{% set version = "4.9.2" %}
{% set build = 2 %}
{% set build = 3 %}

# recipe-lint fails if mpi is undefined
{% set mpi = mpi or 'nompi' %}
Expand Down Expand Up @@ -78,6 +78,7 @@ requirements:
- libzip
- zstd
- libxml2
- blosc
run:
- {{ mpi }} # [mpi != 'nompi']
- curl
Expand Down