Skip to content

Fix conditions for feature MARS2GRIB #212

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

Merged
danovaro merged 1 commit into from
May 13, 2026
Merged

Fix conditions for feature MARS2GRIB #212

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
14 changes: 8 additions & 6 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,13 @@ if(NOT METKIT_CONFIGS_BRANCH)
set(METKIT_CONFIGS_BRANCH chk)
endif()

Comment thread
tweska marked this conversation as resolved.
# eckit

set( PERSISTENT_NAMESPACE "eckit" CACHE INTERNAL "" ) # needed for generating .b files for persistent support

ecbuild_find_package( NAME eckit VERSION 1.32 REQUIRED )


ecbuild_add_option( FEATURE BUILD_TOOLS
DEFAULT ON
DESCRIPTION "Build the command line tools" )
Expand Down Expand Up @@ -64,8 +71,7 @@ ecbuild_add_option( FEATURE ODB

ecbuild_add_option( FEATURE MARS2GRIB
DEFAULT ON
# CONDITION HAVE_GRIB AND HAVE_ECKIT_GEO ==> revert as soon as we disambiguate the eckit geo support from the one in eccodes
CONDITION HAVE_GRIB
CONDITION HAVE_GRIB AND eckit_HAVE_ECKIT_GEO
DESCRIPTION "Build the MARS2GRIB encoder" )

# Python bindings for mars2grib encoder
Expand All @@ -88,11 +94,7 @@ ecbuild_add_option( FEATURE METKIT_CONFIG
ecbuild_add_option( FEATURE FAIL_ON_CCSDS
DESCRIPTION "Fail on CCSDS"
DEFAULT OFF )
# eckit

set( PERSISTENT_NAMESPACE "eckit" CACHE INTERNAL "" ) # needed for generating .b files for persistent support

ecbuild_find_package( NAME eckit VERSION 1.32 REQUIRED )

############################################################################################
# sources
Expand Down
Loading