Skip to content

fix(startup): init script / rm output option#4035

Merged
jamtur01 merged 1 commit into
vectordotdev:masterfrom
emuesuenu:fix/initscript
Sep 23, 2020
Merged

fix(startup): init script / rm output option#4035
jamtur01 merged 1 commit into
vectordotdev:masterfrom
emuesuenu:fix/initscript

Conversation

@emuesuenu
Copy link
Copy Markdown
Contributor

function daemon does not have an output option, so delete it.

I couldn't find the init.d processor that has the output option, so I just want to delete it.
https://github.com/fedora-sysv/initscripts/blob/master/etc/rc.d/init.d/functions#L239-L272

@emuesuenu emuesuenu changed the title fix(<Linux>): init script / rm output option fix(Linux): init script / rm output option Sep 21, 2020
Signed-off-by: emuesuenu <emuesuenu@hotmail.com>
@emuesuenu emuesuenu changed the title fix(Linux): init script / rm output option fix(startup): init script / rm output option Sep 21, 2020
Copy link
Copy Markdown
Contributor

@jamtur01 jamtur01 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jamtur01 jamtur01 merged commit f020524 into vectordotdev:master Sep 23, 2020
mengesb pushed a commit to jacobbraaten/vector that referenced this pull request Dec 9, 2020
Signed-off-by: emuesuenu <emuesuenu@hotmail.com>
Signed-off-by: Brian Menges <brian.menges@anaplan.com>
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.

2 participants