Bash: BUG FIX: _p4m_lcg_port() would try to output debug messages to …

…file descriptor 3 (not 2 as in stderr). This was a left over from debugging via Bats. It would also only look at ${PORT4ME_DEBUG}, when it should have been ${PORT4ME_DEBUG:-false}
HenrikBengtsson · Jan 16, 2024 · 01d7af2 · 01d7af2
1 parent 1cf3c80
commit 01d7af2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/bash/NEWS.md b/bash/NEWS.md
@@ -1,3 +1,10 @@
+# Version (development version)
+
+## Bug Fixes
+
+ * ...
+
+
 # Version 0.7.0 [2024-01-15]
 
 ## Significant Changes

diff --git a/bash/incl/port4me.bash b/bash/incl/port4me.bash
@@ -38,7 +38,7 @@
 #' Requirements:
 #' * Bash (>= 4)
 #'
-#' Version: 0.7.0
+#' Version: 0.7.0-9001
 #' Copyright: Henrik Bengtsson (2022-2024)
 #' License: MIT
 #' Source code: https://github.com/HenrikBengtsson/port4me
@@ -293,11 +293,11 @@ _p4m_lcg_port() {
         has_subset=true
     fi
 
-    if ${PORT4ME_DEBUG}; then
+    if ${PORT4ME_DEBUG:-false}; then
         {
             echo "(min,max): ($min,$max)"
             echo "subset: [n=${#subset[@]}]"
-        } >&3
+        }
     fi
 
     ## Sample values in [0,m-2] (sic!), but reject until in [min,max],