From 98e06183e3d949b76822df897211db962f92e93d Mon Sep 17 00:00:00 2001
From: gampnico <45390064+gampnico@users.noreply.github.com>
Date: Fri, 29 Nov 2024 10:27:22 +0100
Subject: [PATCH] fix: add COSIPY_DIR environment variable for COSIPY
 configuration hook

Refs: cryotools/cosipy#86
---
 README.md                       | 14 +++++++++-----
 src/patch_wrfxcspy/README.md    | 14 +++++++++-----
 src/patch_wrfxcspy/build_wrf.sh |  3 ++-
 3 files changed, 20 insertions(+), 11 deletions(-)

diff --git a/README.md b/README.md
index c6be4ce..9034639 100755
--- a/README.md
+++ b/README.md
@@ -26,16 +26,20 @@ module load perl/5.36 perl-path-tiny perl-time-piece
 bash patch_wrfxcspy.sh --install-all -i ./foo/bar/
 ```
 
-2. Install WRF's dependencies and edit ``build_wrf.sh`` to point to the right paths for COSIPY_API, NETCDF, HDF5, and DIR. Make sure WRF's dependencies are correctly installed.
+2. Install WRF's dependencies and edit ``build_wrf.sh`` or ``build_wrf_full.sh`` to point to the right paths for COSIPY_API, COSIPY_DIR, NETCDF, HDF5, and DIR. Alternatively, you can load your own build file, and export COSIPY_API and COSIPY_DIR to your LD_LIBRARY_PATH.
 
-3. Load environment variables, configure and patch WRF:
+3. Load environment variables:
 ```console
-bash patch_wrfxcspy -e -c -p -i ./foo/bar/WRF/
+bash patch_wrfxcspy -e
+source build_wrf.sh  # if the previous command fails
 ```
 
-Depending on your system and access permissions, you may need to run ``build_wrf.sh`` separately.
+4. Configure and patch WRF:
+```console
+bash patch_wrfxcspy -c -p -i ./foo/bar/WRF/
+```
 
-4. Build WRF: ``./compile em_real >& log.compile``
+5. Build WRF: ``./compile em_real >& log.compile``
 
 **Arguments:**
 
diff --git a/src/patch_wrfxcspy/README.md b/src/patch_wrfxcspy/README.md
index c6be4ce..9034639 100755
--- a/src/patch_wrfxcspy/README.md
+++ b/src/patch_wrfxcspy/README.md
@@ -26,16 +26,20 @@ module load perl/5.36 perl-path-tiny perl-time-piece
 bash patch_wrfxcspy.sh --install-all -i ./foo/bar/
 ```
 
-2. Install WRF's dependencies and edit ``build_wrf.sh`` to point to the right paths for COSIPY_API, NETCDF, HDF5, and DIR. Make sure WRF's dependencies are correctly installed.
+2. Install WRF's dependencies and edit ``build_wrf.sh`` or ``build_wrf_full.sh`` to point to the right paths for COSIPY_API, COSIPY_DIR, NETCDF, HDF5, and DIR. Alternatively, you can load your own build file, and export COSIPY_API and COSIPY_DIR to your LD_LIBRARY_PATH.
 
-3. Load environment variables, configure and patch WRF:
+3. Load environment variables:
 ```console
-bash patch_wrfxcspy -e -c -p -i ./foo/bar/WRF/
+bash patch_wrfxcspy -e
+source build_wrf.sh  # if the previous command fails
 ```
 
-Depending on your system and access permissions, you may need to run ``build_wrf.sh`` separately.
+4. Configure and patch WRF:
+```console
+bash patch_wrfxcspy -c -p -i ./foo/bar/WRF/
+```
 
-4. Build WRF: ``./compile em_real >& log.compile``
+5. Build WRF: ``./compile em_real >& log.compile``
 
 **Arguments:**
 
diff --git a/src/patch_wrfxcspy/build_wrf.sh b/src/patch_wrfxcspy/build_wrf.sh
index ee8d496..c52d9bd 100755
--- a/src/patch_wrfxcspy/build_wrf.sh
+++ b/src/patch_wrfxcspy/build_wrf.sh
@@ -13,7 +13,8 @@ module load user-spack
 module load time
 
 # Coupler
-export COSIPY_API="${USER_PATH}/WRF/COUPLER"
+export COSIPY_API="${USER_PATH}/path/to/WRF/COUPLER"
+export COSIPY_DIR="${USER_PATH}/path/to/WRF/cosipy"
 export LD_LIBRARY_PATH=${LD_LIBRARY_PATH}:${COSIPY_API}
 export LIBRARY_PATH=${LIBRARY_PATH}:${COSIPY_API}
 export PATH=${PATH}:${COSIPY_API}