From a0420cc73fe611baaa139646da088e57bb7e2024 Mon Sep 17 00:00:00 2001
From: Achim Gsell <achim.gsell@psi.ch>
Date: Wed, 28 Apr 2021 10:53:07 +0200
Subject: [PATCH] modulecmd.bash.in: save_env() requires an argument now

PMODULES_ENV will be compiles and exported if save_env() is called
with $1 != 'no'
---
 Pmodules/modulecmd.bash.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/Pmodules/modulecmd.bash.in b/Pmodules/modulecmd.bash.in
index 000bf01..420cd30 100644
--- a/Pmodules/modulecmd.bash.in
+++ b/Pmodules/modulecmd.bash.in
@@ -102,7 +102,7 @@ export_env() {
 declare g_env_must_be_saved='no'
 
 save_env() {
-        [[ ${g_env_must_be_saved} == 'no' ]] && return 0
+        [[ "$1" == 'no' ]] && return 0
         local vars=( GroupDepths UsedReleases UseFlags UsedGroups )
         vars+=( PMODULES_DEFAULT_GROUPS PMODULES_DEFINED_RELEASES )
         vars+=( PMODULES_DEFAULT_RELEASES )
@@ -115,7 +115,7 @@ save_env() {
 	unset UsedGroups
 }
 
-trap 'save_env ' EXIT
+trap 'save_env ${g_env_must_be_saved}' EXIT
 
 get_overlay_of_moduledir() {
 	local "$1"