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

Remove blank line at the start from init file (Debian) #276

Merged
merged 1 commit into from Oct 18, 2017

Conversation

Projects
None yet
3 participants
@DDGo
Contributor

DDGo commented Nov 29, 2016

There was a blank line at the start of the file before #! /bin/sh, which declares which interpeter it has to use. With this blank line the init script will not work and will throw error "exit status 1", meaning the deamon doesn't start.

The change was introduced in version 4.22.

This simple edit to the file will fix that.

Remove blank line at the start from init file
There was a blank line at the start of the file before #! /bin/sh, which declares which interpeter it has to use. With this blank line the init script will not work and will throw error "exit status 1", meaning the deamon doesn't start.

This simple edit to the file will fix that
@paskal

This comment has been minimized.

Show comment
Hide comment
@paskal

paskal Jan 13, 2017

@dnobori, could you please merge this one? That's a real problem for all Ubuntu \ Debian users and we need to fix it before introducing package to Debian repo.

paskal commented Jan 13, 2017

@dnobori, could you please merge this one? That's a real problem for all Ubuntu \ Debian users and we need to fix it before introducing package to Debian repo.

@DDGo

This comment has been minimized.

Show comment
Hide comment
@DDGo

DDGo Feb 2, 2017

Contributor

In case it might be still needed: I choose option 1

Contributor

DDGo commented Feb 2, 2017

In case it might be still needed: I choose option 1

@paskal

This comment has been minimized.

Show comment
Hide comment
@paskal

paskal Feb 11, 2017

It's preventing newly installed vpnserver service from start, and package doesn't count as installed properly. Very trivial change, please merge.

paskal commented Feb 11, 2017

It's preventing newly installed vpnserver service from start, and package doesn't count as installed properly. Very trivial change, please merge.

@dnobori

This comment has been minimized.

Show comment
Hide comment
@dnobori

dnobori Oct 18, 2017

Member

Thank you so much for your contribution to enrich the SoftEther VPN source code.

Your patch has been merged on the main source-tree of SoftEther VPN.

As a token of our gratitude, your GitHub username will be added on the AUTHORS.TXT file and on the header of the related source file.
(It will be done within a few days.)
https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT

Thanks again for your contribution.

Member

dnobori commented Oct 18, 2017

Thank you so much for your contribution to enrich the SoftEther VPN source code.

Your patch has been merged on the main source-tree of SoftEther VPN.

As a token of our gratitude, your GitHub username will be added on the AUTHORS.TXT file and on the header of the related source file.
(It will be done within a few days.)
https://github.com/SoftEtherVPN/SoftEtherVPN/blob/master/AUTHORS.TXT

Thanks again for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment