Page MenuHomeFreeBSD

reboot: Allow this to be installed as nextboot
ClosedPublic

Authored by imp on Feb 11 2024, 2:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 21, 1:24 AM
Unknown Object (File)
Mon, Oct 21, 1:24 AM
Unknown Object (File)
Mon, Oct 21, 1:24 AM
Unknown Object (File)
Mon, Oct 21, 1:24 AM
Unknown Object (File)
Mon, Oct 21, 1:24 AM
Unknown Object (File)
Mon, Oct 21, 1:24 AM
Unknown Object (File)
Fri, Oct 18, 7:32 PM
Unknown Object (File)
Fri, Oct 18, 8:24 AM
Subscribers

Details

Summary

Allow nextboot to be a symlink link to reboot. It does everything reboot
does, except doesn't actually setup the sytem to reboot and reboot. Also,
don't accept the reboot args related to rebooting when in nextboot mode.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp requested review of this revision.Feb 11 2024, 2:27 AM
imp created this revision.
kib added inline comments.
sbin/reboot/reboot.c
298
This revision is now accepted and ready to land.Feb 12 2024, 2:15 AM
sbin/reboot/reboot.c
197

Instead of having to repeat this many times, what about having a per-prog getopt string?

268
sbin/reboot/reboot.c
268

Pre-existing style though, which i try not to gratuitously not change...

fold in ed's suggestion and add {} around the while loop to reflect updated
practices since this code was written.

This revision now requires review to proceed.Feb 12 2024, 6:33 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 12 2024, 6:54 PM
This revision was automatically updated to reflect the committed changes.