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.

Sign up for GitHub

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

Jump to bottom

Don't fork. Choose the "simple" type #1

Open
borgogelli opened this issue Apr 4, 2017 · 2 comments
Open

Don't fork. Choose the "simple" type #1

borgogelli opened this issue Apr 4, 2017 · 2 comments

Comments

@borgogelli
Copy link

I think it should be
Type=simple
(And that's definetly the right solution for Sonar 6)

Regards

Andrea
@wichtounet
Copy link
Owner

wichtounet commented Apr 7, 2017 •
edited
Loading

Hi,

Thanks for the suggestion. Given your comment, I'll consider that you are talking about the sonar service.

I'm not sure Type=simple is correct. This means that the process started by ExecStart is the main process. This is not the case since the ExecStart= is a bash script and the script is launching a Java process (by forking). I think forking is the way to go here.

@Victor-D
Copy link

Maybe Type=oneshot ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@wichtounet @Victor-D @borgogelli