search

projg2 / eclean-kernel

Installed kernel cleanup tool
GNU General Public License v2.0
32 stars 11 forks source link

Use samefile() instead of regex matching when checking bootloader kernel paths #43

Closed bstaletic closed 1 year ago

bstaletic commented 1 year ago

This allows eclean-kernel to correctly identify the kernels used by the bootloader config in the following scenario:

  1. LILO says image=/boot/vmlinuz
  2. /boot/vmlinuz is a symlink to /boot/vmlinuz-X.Y.Z-suffix, which is the actual kernel.

Before this commit, eclean-kernel -ap woulc complain that

Note: strangely named used kernel: /boot/vmlinuz

and then list vmlinuz-X.Y.Z-suffix as one of the kernels to be removed.

Closes #42

mgorny commented 1 year ago

Do you want to fix the QA issues reported by CI or should I do it while merging?

bstaletic commented 1 year ago

Do you want to fix the QA issues reported by CI or should I do it while merging?

Sorry about that! Should all be fixed now.

mgorny commented 1 year ago

I'll release 2.99.6 shortly.

bstaletic commented 1 year ago

Thanks! You do not need to make me the reason for a hasty release. I can use =eclean-kernel-9999 for a while.