-
Notifications
You must be signed in to change notification settings - Fork 29
msi: fix slow start issue on Windows #631
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
28c4287 to
c2059bf
Compare
|
forked version was fetched. |
|
Thanks for this! |
c2059bf to
c71f898
Compare
fix-slow-start-with-ruby may be removed by default (when that PR was merged), so it is better to create a specific branch. |
c71f898 to
49263d4
Compare
|
Added fluent-package branch and refer it from Gemfile. |
49263d4 to
c95b121
Compare
Thanks!! |
c95b121 to
fa99bc3
Compare
|
When processing :ruby_gems task, it raises |
|
bundler v2.3.27 contains fix for regression. check it. |
|
with install_gem_from_local_repo
It try to install win32-service on linux. |
|
got it. It seems that the following context does not work. |
fa99bc3 to
dabb717
Compare
|
Hmm, not reproducible failure yet about minimum Gemfile. |
|
win32-service gem was built but not installed from local repo. |
dabb717 to
e7d822d
Compare
|
NOTE: sudo apt install colorized-logs and use |
14d465b to
7e873b8
Compare
Until chef/win32-service#85 is merged, use forked version of win32-service. This fix should be applied to fluent-package not to block starting fluentdwinsvc service on Windows. See fluent#618 Closes: fluent#630 NOTE: even though just putting the following line does not install forked version of win32-service, so install it explicitly as same as fluentd gem. gem "win32-service", github: "fluent-plugins-nursery/win32-service", branch: "fluent-package", platforms: [:mingw, :x64_mingw] Signed-off-by: Kentaro Hayashi <hayashi@clear-code.com>
7e873b8 to
ca4bea6
Compare
|
checking only end_with?("/lib/win32/windows/functions.rb") is not enough: |
|
CI has passed ! |
|
Thanks so much! |
ashie
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Although it would be better to generalize the tasks to be applicable to other gems, I don't always require it in this pull request. Because we shouldn't use forked version of gems in usual.
Until chef/win32-service#85 is merged,
use forked version of win32-service.
This fix should be applied to fluent-package not to block starting
fluentdwinsvc service on Windows.
See #618
Closes: #630
NOTE: even though just putting the following line
does not install forked version of win32-service, so
install it explicitly as same as fluentd gem.