Page MenuHomeFreeBSD

Gives more fexibility to kernel installation
AbandonedPublic

Authored by salvadore on Apr 2 2019, 6:33 PM.
Tags
None
Referenced Files
F81647983: D19800.diff
Fri, Apr 19, 10:49 AM
Unknown Object (File)
Dec 20 2023, 1:25 AM
Unknown Object (File)
Dec 5 2023, 3:13 PM
Unknown Object (File)
Oct 1 2023, 11:40 AM
Unknown Object (File)
Sep 18 2023, 10:12 AM
Unknown Object (File)
Jun 30 2023, 8:59 PM
Unknown Object (File)
Jun 27 2023, 11:19 AM
Unknown Object (File)
Jun 15 2023, 2:18 PM

Details

Reviewers
cem
allanjude
Group Reviewers
Contributor Reviews (src)
Summary

Introduces a new USEKERNCONFNAME, that works as follows:

  • if USEKERNCONFNAME is undefined, then nothing is changed;
  • if USEKERNCONFNAME is defined, then install all kernels in a directory named exactly as the kernel configuration.

For example, if KERNCONF = GENERIC CURRENT STABLE, then the 3 kernels would be installed in /boot/GENERIC, /boot/CURRENT, /boot/STABLE respectively. No /boot/kernel would be created.
Then, /boot/loader.conf should be updated with the new default kernel name (by setting the "kernel" variable) or a /boot/kernel file should be created manually (by copying a valid kernel directory, by soft linking to a valid kernel directory...).

Test Plan

I tested successfully the patch. I did not test what happens if no /boot/kernel file exists and the kernel variable in /boot/loader.conf is not defined.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Rename USEKERNNAME to USEKERNCONFNAME and reverts logic: now if USEKERNCONFNAME is defined then directory will be called as kernel configurations, otherwise everything works as usual.

Fix style: before I used ifndef, but it seems that if !defined is prefered.

The patch is too old and does not not work as expected (or as I remember).

I am going to fix it anyway for personal use. If anyone is interested in it, please let me know: in that case I would reopen the review with the updated patch once it is ready.