Copilot commented on code in PR #780:
URL: https://github.com/apache/unomi/pull/780#discussion_r3469109386


##########
build.sh:
##########
@@ -510,6 +530,15 @@ while [ "$1" != "" ]; do
     shift
 done
 
+# Wire up log file output if requested
+if [ -n "$LOG_FILE" ]; then
+    if [ "$LOG_FILE_ONLY" = true ]; then
+        exec > "$LOG_FILE" 2>&1
+    else
+        exec > >(tee -a "$LOG_FILE") 2>&1
+    fi
+fi

Review Comment:
   `--log-file-only` currently has no effect unless `--log-file PATH` is also 
provided (it is silently ignored). Since this flag changes output behavior, it 
should fail fast with a clear error when used without `--log-file` to avoid 
confusing runs in CI/local scripts.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to