Merge pull request #164 from georust/zarr_enabled_by_default

Enable zarr support on static builds

Author: magnusuMET

.github/workflows/ci.yml

@@ -143,6 +143,10 @@ jobs:
       - name: Install Rust (${{matrix.rust}})
         uses: dtolnay/rust-toolchain@stable
         with: {toolchain: '${{matrix.rust}}'}
+      - name: Set target version for macos-13
+        run: |
+          echo "MACOSX_DEPLOYMENT_TARGET=13.7" >> $GITHUB_ENV
+        if : ${{ matrix.os == 'macos-13' }}
       - name: Build and test
         run: cargo test -vv --features netcdf/derive,static --workspace --exclude netcdf-derive
 

netcdf-src/build.rs

@@ -33,7 +33,9 @@ fn main() {
         //
         .define("NETCDF_ENABLE_PARALLEL4", "OFF") // TODO: Enable mpi support
         //
-        .define("NETCDF_ENABLE_NCZARR", "OFF") // TODO: requires a bunch of deps
+        .define("NETCDF_ENABLE_NCZARR", "ON")
+        .define("NETCDF_ENABLE_NCZARR_ZIP", "OFF") // TODO: requires forwarding flags for libz
+        .define("NETCDF_ENABLE_S3", "OFF") // TODO: requires a bunch of deps
         //
         .define("NETCDF_ENABLE_DAP", "OFF") // TODO: feature flag, requires curl
         .define("NETCDF_ENABLE_BYTERANGE", "OFF") // TODO: feature flag, requires curl