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

Initialize Configuration::InitRunDir for Windows and writing the PID file #6679

Merged
merged 1 commit into from Oct 15, 2018

Conversation

dnsmichi
Copy link
Contributor

Even if this some Unix specific stuff, we rely on writing this file
in our generic application code. Therefore its path needs to be specified.

This got broken with 9fbc406 which renamed RunDir to InitRunDir, but
didn't re-add it to the Windows #ifdefs.

@dnsmichi dnsmichi added Plugins area/windows Windows agent and plugins and removed Plugins labels Oct 12, 2018
…file

Even if this some Unix specific stuff, we rely on writing this file
in our generic application code. Therefore its path needs to be specified.

This got broken with 9fbc406 which renamed RunDir to InitRunDir, but
didn't re-add it to the Windows #ifdefs.

fixes #6677
@dnsmichi dnsmichi force-pushed the bugfix/initialize-initrundir-constant-on-windows branch from c7ec99e to 1c7fddb Compare October 15, 2018 07:25
@dnsmichi dnsmichi changed the title WIP: Initialize Configuration::InitRunDir for Windows and writing the PID file Initialize Configuration::InitRunDir for Windows and writing the PID file Oct 15, 2018
@dnsmichi dnsmichi added the bug Something isn't working label Oct 15, 2018
@dnsmichi dnsmichi added this to the 2.10.1 milestone Oct 15, 2018
@dnsmichi dnsmichi merged commit db322eb into master Oct 15, 2018
@dnsmichi dnsmichi deleted the bugfix/initialize-initrundir-constant-on-windows branch October 15, 2018 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/windows Windows agent and plugins bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant