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

Cleanup Systemd files and add Travis-CI test for Systemd (Rebased) #18

Closed
wants to merge 4 commits into from
Closed

Cleanup Systemd files and add Travis-CI test for Systemd (Rebased) #18

wants to merge 4 commits into from

Conversation

evilhamsterman
Copy link

I wanted to make sure you could say yes to #12 so I rebased it on your master and consolidated the systemd clean up lines.

rm -f /lib/systemd/system/basic.target.wants/*;\
rm -f /lib/systemd/system/anaconda.target.wants/*; \
rm -f /lib/systemd/system/systemd*udev*; \
rm -f /lib/systemd/system/getty.target
Copy link

@vincent-legoll vincent-legoll May 28, 2020

Choose a reason for hiding this comment

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

Why the multiple rms ?
There are also whitespace inconsistencies ;-)

Copy link
Author

Choose a reason for hiding this comment

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

This was just a rebase of #12 where they explained those files need to be removed to fix problems with systemd in a container running in a systemd system. The biggest problem is the container causes the container to log the user of the host out when it stops. Why multiple rms instead of a single rm. No real reason I was just rebasing the pull because @geerlingguy asked the original guy to rebase but the original guy wasn't responding. Same goes for the whitespace inconsistency.

@evilhamsterman
Copy link
Author

Since this is way out of date and 18.04 is nearly EOL just closing this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants