Skip to content
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

Update windows install steps for 4.0.0 #190

Merged
merged 1 commit into from
Jul 13, 2020

Conversation

kenhys
Copy link
Contributor

@kenhys kenhys commented Jul 9, 2020

It omits manual registration step as Windows service.

@kenhys kenhys force-pushed the already-registered-as-windows-service branch from 1cfd95e to ef0f66d Compare July 9, 2020 00:47
@repeatedly
Copy link
Member

td-agent 3 is widely used. Could you separate section for td-agent 3 and td-agent 4?

@kenhys
Copy link
Contributor Author

kenhys commented Jul 10, 2020

Thanks, I'll fix it.

It omits manual registration step as Windows service.

Signed-off-by: Kentaro Hayashi <[email protected]>
@kenhys kenhys force-pushed the already-registered-as-windows-service branch from ef0f66d to 91723fd Compare July 10, 2020 03:30
@kenhys
Copy link
Contributor Author

kenhys commented Jul 10, 2020

I've fixed to split into two sections.

image

@cosmo0920
Copy link
Contributor

td-agent 4 is workable with MSYS2 build system to build C extension included gems such as winevt_c and certstore_c.
Adding MSYS2 description is future task?

@ashie
Copy link
Member

ashie commented Jul 10, 2020

td-agent 4 is workable with MSYS2 build system to build C extension included gems such as winevt_c and certstore_c.
Adding MSYS2 description is future task?

FYI: td-agent 4.0.0 has a bug for it: fluent/fluent-package-builder#129

@cosmo0920
Copy link
Contributor

Oops. Adding MSYS2 description should be future task.... It should be added after 4.0.1 is released.

@repeatedly repeatedly merged commit 48577ec into fluent:1.0 Jul 13, 2020
@repeatedly
Copy link
Member

Thx!

@kenhys kenhys deleted the already-registered-as-windows-service branch March 12, 2021 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants