entrypoint: avoid polluting stdout (#17125)

yaauie · logstashmachine · commit bbdbb1619e9f · 2025-02-20T20:56:46.000Z
routes output from setup-related functions to stderr, so that stdout can include only the output of the actual program. (cherry picked from commit 91258c3)
diff --git a/bin/logstash.bat b/bin/logstash.bat
@@ -6,7 +6,7 @@ set params='%*'
 if "%1" == "-V" goto version
 if "%1" == "--version" goto version
 
-call "%~dp0setup.bat" || exit /b 1
+1>&2 (call "%~dp0setup.bat") || exit /b 1
 if errorlevel 1 (
 	if not defined nopauseonerror (
 		pause
diff --git a/bin/logstash.lib.sh b/bin/logstash.lib.sh
@@ -186,8 +186,8 @@ setup_vendored_jruby() {
 }
 
 setup() {
-  setup_java
-  setup_vendored_jruby
+  >&2 setup_java
+  >&2 setup_vendored_jruby
 }
 
 ruby_exec() {

Original file line number	Diff line number	Diff line change
`@@ -186,8 +186,8 @@ setup_vendored_jruby() {`
`186`	`186`	`}`
`187`	`187`
`188`	`188`	`setup() {`
`189`		`- setup_java`
`190`		`- setup_vendored_jruby`
	`189`	`+ >&2 setup_java`
	`190`	`+ >&2 setup_vendored_jruby`
`191`	`191`	`}`
`192`	`192`
`193`	`193`	`ruby_exec() {`