From eb0868b5e657c55846ac5b7b7f5cba7b3a75e245 Mon Sep 17 00:00:00 2001
From: Achim Gsell <achim.gsell@psi.ch>
Date: Mon, 4 Aug 2025 17:16:59 +0200
Subject: [PATCH] code cleanup

---
 Pmodules/libmodules.tcl | 4 ----
 Pmodules/libpbuild.bash | 3 ++-
 Pmodules/modbuild.in    | 1 -
 3 files changed, 2 insertions(+), 6 deletions(-)

diff --git a/Pmodules/libmodules.tcl b/Pmodules/libmodules.tcl
index d772444..f19c671 100644
--- a/Pmodules/libmodules.tcl
+++ b/Pmodules/libmodules.tcl
@@ -1,7 +1,4 @@
 #
-# :TODO:
-# switch/swap
-# unload modules if parent removed
 #
 
 if {[info exists env(PMODULES_DEBUG)] && $env(PMODULES_DEBUG)} {
@@ -67,7 +64,6 @@ proc _pmodules_parse_pmodules_env { } {
 	debug "return"
 }
 
-debug "module-addgroup"
 proc module-addgroup { group } {
 	global env
 	global name
diff --git a/Pmodules/libpbuild.bash b/Pmodules/libpbuild.bash
index 24790ba..f2d675d 100644
--- a/Pmodules/libpbuild.bash
+++ b/Pmodules/libpbuild.bash
@@ -931,7 +931,8 @@ _build_module() {
 			fi
 
 			std::info "Loading module: ${m}"
-			eval "$( "${modulecmd}" bash load "${m}" )"
+			local output="$("${modulecmd}" bash load "${m}")";
+			eval "${output}"
 			if ! bm::is_loaded "$m"; then
 				"${modulecmd}" bash list
 				std::die 5 \
diff --git a/Pmodules/modbuild.in b/Pmodules/modbuild.in
index daddff5..434caa6 100644
--- a/Pmodules/modbuild.in
+++ b/Pmodules/modbuild.in
@@ -1217,7 +1217,6 @@ build_modules_yaml_v1(){
 		pbuild.add_patch_files "${args[@]}"
 	}
 
-
 	is_variant_to_build(){
 		local -- name="$1"
 		local -- version="$2"