-
Notifications
You must be signed in to change notification settings - Fork 26
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
msi: fix slow start issue on Windows #631
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 <[email protected]>
7e873b8
to
ca4bea6
Compare
checking only end_with?("/lib/win32/windows/functions.rb") is not enough:
|
CI has passed ! |
Thanks so much! |
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.