fix: Change flags in buildconfig

Reverting the introduced change from ca01956.
Also moved the flags to ON instead using 1

Author: tbkr

python/fdb5lib/buildconfig

@@ -11,12 +11,10 @@
 # TODO we duplicate information -- pyproject.toml's `name` and `packages` are derivable from $NAME and must stay consistent
 
 NAME="fdb"
-# TODO -DENABLE_GRIB=1 causes build to fail, no idea why, no idea if desired
-#
 # NOTE zarr interface is dependent on python 3.11+ -- but we dont activate the venv at the time this is sourced, that's why we rely on PYVERSION. Remove this whole part around October 2026 when 3.10 goes EoL
 # if [ "True" = "$(python -c 'import sys; print(sys.version_info[0:2] >= (3, 11))')" ] ; then
 if [ "3.10" != "$PYVERSION" ] ; then
-    ZARR_IFACE="-DENABLE_PYTHON_ZARR_INTERFACE=1"
+    ZARR_IFACE="-DENABLE_PYTHON_ZARR_INTERFACE=ON"
 else
     >&2 echo "Not enabling zarr interface because python version is $PYVERSION"
     ZARR_IFACE=""
@@ -27,12 +25,12 @@ fi
 # this whole part around October 2026 when 3.10 goes EoL
 # if [ "True" = "$(python -c 'import sys; print(sys.version_info[0:2] >= (3, 11))')" ] ; then
 if [ "3.10" != "$PYVERSION" ] ; then
-    PYFDB_IFACE="-DENABLE_PYTHON_FDB_INTERFACE=1"
+    PYFDB_IFACE="-DENABLE_PYTHON_FDB_INTERFACE=ON"
 else
     >&2 echo "Not enabling pyfdb interface because python version is $PYVERSION"
     PYFDB_IFACE=""
 fi
 
-CMAKE_PARAMS="-Deckit_ROOT=/tmp/fdb/prereqs/eckitlib -Dmetkit_ROOT=/tmp/fdb/prereqs/metkitlib -Deccodes_ROOT=/tmp/fdb/prereqs/eccodeslib -DENABLE_GRIB=0 $ZARR_IFACE $PYFDB_IFACE -DENABLE_FDB_BUILD_TOOLS=ON"
+CMAKE_PARAMS="-Deckit_ROOT=/tmp/fdb/prereqs/eckitlib -Dmetkit_ROOT=/tmp/fdb/prereqs/metkitlib -Deccodes_ROOT=/tmp/fdb/prereqs/eccodeslib $ZARR_IFACE $PYFDB_IFACE -DENABLE_FDB_BUILD_TOOLS=ON"
 PYPROJECT_DIR="python/fdb5lib"
 DEPENDENCIES='["eckitlib", "eccodeslib", "metkitlib"]'