fluent / fluent-package-builder

td-agent (Fluentd) Building and Packaging System
Apache License 2.0
21 stars 23 forks source link

msi: refactor exectuion bat files #647

Closed daipom closed 1 week ago

daipom commented 2 weeks ago

fluentd.bat

Before: FLUENT_PACKAGE_VERSION was passed to Fluentd process. After: FLUENT_PACKAGE_VERSION is not passed to Fluentd process.

fluent-gem.bat

Previously, it changed the environment variables unintentionally.

Before

> ruby -e "ENV['PATH'].split(';').select{|v| v.include?('fluent')}.map{|v| p v}"
"C:\\opt\\fluent\\"
"C:\\opt\\fluent\\bin"

> fluent-gem --version
3.4.19

> ruby -e "ENV['PATH'].split(';').select{|v| v.include?('fluent')}.map{|v| p v}"
"C:\\opt\\fluent\\bin"
"C:\\opt\\fluent\\"
"C:\\opt\\fluent\\bin"

After

> ruby -e "ENV['PATH'].split(';').select{|v| v.include?('fluent')}.map{|v| p v}"
"C:\\opt\\fluent\\"
"C:\\opt\\fluent\\bin"

> fluent-gem --version
3.4.19

> ruby -e "ENV['PATH'].split(';').select{|v| v.include?('fluent')}.map{|v| p v}"
"C:\\opt\\fluent\\"
"C:\\opt\\fluent\\bin"