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

Add conditional to cover systemd in Ubuntu 15.04+ #112

Merged
merged 1 commit into from Feb 24, 2016

Conversation

Projects
None yet
3 participants
@elijah
Contributor

elijah commented Feb 17, 2016

This should make the ssh-hardening cookbook work properly on ubuntu 15.04 and later. Currently it freaks out if you try to run it with Vivid or Wily, which have moved to the use of systemd rather than Upstart.

@chris-rock

This comment has been minimized.

Show comment
Hide comment
@chris-rock

chris-rock Feb 24, 2016

Member

Thanks @elijah that addition is awesome!

Member

chris-rock commented Feb 24, 2016

Thanks @elijah that addition is awesome!

chris-rock added a commit that referenced this pull request Feb 24, 2016

Merge pull request #112 from elijah/patch-1
Add conditional to cover systemd in Ubuntu 15.04+

@chris-rock chris-rock merged commit 442f0e4 into dev-sec:master Feb 24, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@elijah

This comment has been minimized.

Show comment
Hide comment
@elijah

elijah May 26, 2016

Contributor

This change is no longer in HEAD and needs to be. Please fix.

Contributor

elijah commented on 7454b7b May 26, 2016

This change is no longer in HEAD and needs to be. Please fix.

This comment has been minimized.

Show comment
Hide comment
@shortdudey123

shortdudey123 Sep 30, 2016

Contributor

@elijah looks like Systemd has been preferred over Upstart for debain based systems for almost 2 yrs
Changed in 12.1.0.rc.0

Contributor

shortdudey123 replied Sep 30, 2016

@elijah looks like Systemd has been preferred over Upstart for debain based systems for almost 2 yrs
Changed in 12.1.0.rc.0

This comment has been minimized.

Show comment
Hide comment
@elijah

elijah Sep 30, 2016

Contributor
Contributor

elijah replied Sep 30, 2016

@elijah

This comment has been minimized.

Show comment
Hide comment
@elijah

elijah May 26, 2016

Contributor

Needs to be re-opened.

Contributor

elijah commented May 26, 2016

Needs to be re-opened.

@elijah

This comment has been minimized.

Show comment
Hide comment
@elijah

elijah May 26, 2016

Contributor

Eh, I was wrong. It IS there. :) 👍

Contributor

elijah commented May 26, 2016

Eh, I was wrong. It IS there. :) 👍

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