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

README: consolidate systemctl commands #334

Merged
merged 1 commit into from
Sep 2, 2022
Merged

README: consolidate systemctl commands #334

merged 1 commit into from
Sep 2, 2022

Conversation

ferdnyc
Copy link
Contributor

@ferdnyc ferdnyc commented Sep 1, 2022

It's a really minor thing, but the two commands provided in the README:

$ sudo systemctl enable sysstat
$ sudo systemctl start sysstat

can be written as a single command using the systemctl enable --now flag. This PR updates the README commands accordingly.

@sysstat sysstat merged commit e8969fe into sysstat:master Sep 2, 2022
@stevekay
Copy link
Contributor

stevekay commented Sep 3, 2022

The --now functionality is not present in older versions of systemd.

See systemd changelog showing it was introduced in v220, 7 years ago.

Figured worth noting this here, in case subsequent queries come in from folk saying the step in the README isn't working on their (old and crusty!) environment.

@ferdnyc ferdnyc deleted the patch-1 branch September 3, 2022 13:45
@ferdnyc
Copy link
Contributor Author

ferdnyc commented Sep 3, 2022

@stevekay

The --now functionality is not present in older versions of systemd.

Fair point, I admit I didn't realize it was that "new". But Ubuntu bionic, the oldest still supported LTS release, shipped with systemd 237, and CentOS/RHEL didn't even adopt systemd until their release 7, when it was at... sh*t! 219. FML.

Well... the maintainers' call, of course. But in light of RHEL7 systemctl not supporting --now, I'd support reverting this so their poor users' heads don't explode. Sorry about that.

@stevekay
Copy link
Contributor

stevekay commented Sep 3, 2022

@stevekay

The --now functionality is not present in older versions of systemd.

Fair point, I admit I didn't realize it was that "new". But Ubuntu bionic, the oldest still supported LTS release, shipped with systemd 237, and CentOS/RHEL didn't even adopt systemd until their release 7, when it was at... sh*t! 219. FML.

Well... the maintainers' call, of course. But in light of RHEL7 systemctl not supporting --now, I'd support reverting this so their poor users' heads don't explode. Sorry about that.

I think the fine folk at Red Hat back ported the --now functionality into the systemd they shipped at RHEL 7 GA. So all good on that distro.

sysstat added a commit that referenced this pull request Sep 4, 2022
Display the commands that can be used when option --now is not supported
by systemd.

Signed-off-by: Sebastien GODARD <[email protected]>
@sysstat
Copy link
Owner

sysstat commented Sep 4, 2022

Commit 0726684 also adds the old syntax for those using a systemd version not supporting option --now.

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.

None yet

4 participants