clearlinux / clr-installer

Clear Linux* OS Installer
GNU General Public License v3.0
92 stars 42 forks source link

Let systemd check kernel command line options #781

Closed bwarden closed 2 years ago

bwarden commented 2 years ago

Cleanup for clr-installer-provision.service. Trying to avoid extra embedded bash calls -- use systemd's internal capabilties instead of relying on bashisms.

Fixes Issue: #

Changes proposed in this pull request:

bryteise commented 2 years ago

Seems fairly reasonable to me.

ahkok commented 2 years ago

It looks fine and a lot cleaner. I'm not sure if we want to Restart=on-failure since the default value for that is no and it will reboot the system immediately? I would definitely experiment with making clr-installer fail and see whether at a minimum there is some usable output before the system reboots.

But aside that this is a good cleanup.

bwarden commented 2 years ago

Also, the console blocking failures I was seeing are possibly because I was on ttyS1, which isn't listed in Conflicts. Worth further testing in a controlled environment.