Skip to content

Commit

Permalink
REmove exec from bash script.
Browse files Browse the repository at this point in the history
Fixes #99
  • Loading branch information
jsuereth committed Jan 23, 2014
1 parent d7982dc commit 4e63c75
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,8 @@ execRunner () {
echo ""
}

exec "$@"
# We Don't use "exec" here for our pids to be accurate.
"$@"
}
addJava () {
dlog "[addJava] arg = '$1'"
Expand Down

3 comments on commit 4e63c75

@muuki88
Copy link
Contributor

Choose a reason for hiding this comment

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

I will test it this weekend. Conclusion would be that upstart and systemv using the same script?

@kardapoltsev
Copy link
Member

Choose a reason for hiding this comment

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

I'll change /etc/init.d template and test it tomorrow.

@jsuereth
Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, I think we can get them using the same script, which would be ideal.

Thanks guys!

Please sign in to comment.