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.

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

added check if init file is a symlink to upstart #16

Merged
merged 1 commit into from
Feb 25, 2016
Merged

added check if init file is a symlink to upstart #16

merged 1 commit into from
Feb 25, 2016

Conversation

hggh
Copy link
Contributor

@hggh hggh commented Feb 18, 2016

on Ubuntu 14.04 with needrestart 2.6 without the patch:

Scanning processes...
WARNING: /etc/init.d/cron has no LSB tags!===
Odd number of elements in hash assignment at /usr/sbin/needrestart line
556, line 119.
Use of uninitialized value in list assignment at /usr/sbin/needrestart
line 556, line 119.
Scanning candidates...
Scanning linux images...
Running kernel seems to be up-to-date.
Services to be restarted:

ls -al /etc/init.d/cron
lrwxrwxrwx 1 root root 21 Dec 15 2014 /etc/init.d/cron -> /lib/init/upstart-job

the new code will check if the init script is a symlink and the symlink
points to /lib/init/upstart-job.

on Ubuntu 14.04 with needrestart 2.6 without the patch:

Scanning processes...
WARNING: /etc/init.d/cron has no LSB tags!===
Odd number of elements in hash assignment at /usr/sbin/needrestart line
556, <HLSB> line 119.
Use of uninitialized value in list assignment at /usr/sbin/needrestart
line 556, <HLSB> line 119.
Scanning candidates...
Scanning linux images...
Running kernel seems to be up-to-date.
Services to be restarted:

ls -al /etc/init.d/cron
lrwxrwxrwx 1 root root 21 Dec 15  2014 /etc/init.d/cron -> /lib/init/upstart-job

the new code will check if the init script is a symlink and the symlink
points to /lib/init/upstart-job.
liske added a commit that referenced this pull request Feb 25, 2016
added check if init file is a symlink to upstart
@liske liske merged commit 42090fb into liske:master Feb 25, 2016
@liske liske self-assigned this Feb 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants