#1303 PR closed: Fix recovery for Arch Linux EFI boot

Labels: won't fix / can't fix / obsolete

ProBackup-nl opened issue at 2017-04-14 22:05:

Alternatively look not only look for a TARGET_DEVICE/boot/efi mount, but also for TARGET_DEVICE/boot (as long as directory TARGET_DEVICE/boot/efi exists).

Without this fix the recovery will error out after:
Patching file 'etc/fstab'
with an error like:
Empty string passed to get_device_name

Because in Arch Linux the ESP is mounted at /boot (not /boot/efi)

ProBackup-nl commented at 2017-04-21 07:58:

There is not need for case-insensitive check in this script. Here I don't want to change the check for the directory. Here checking of the real mount point is happening: /boot/efi or /boot in the list of mount points. Consider merging without changes.

jsmeix commented at 2017-04-21 10:00:

The comment in the code tells different:

# check if $TARGET_FS_ROOT/boot/efi directory exists
# case-insensitive because Arch Linux uses /boot/EFI
[[ -d "$TARGET_FS_ROOT/boot/efi" ]]

so that either the comment or the actual code must be fixed.

ProBackup-nl commented at 2017-05-10 20:36:

Comment line #1: general explain why this check is there
Comment line #2: in more detail explain why the check needs to be done case insensitive: otherwise it will break Arch.

Apparently the comments need to be rewritten, but I don't know how to express myself clearer in this case.

gdha commented at 2018-01-25 15:14:

The question is - is the PR still relevant after all these months?

jsmeix commented at 2018-01-25 15:58:

What I meant in my
is that I do not see how the actual code

[[ -d "$TARGET_FS_ROOT/boot/efi" ]]

inplements a case-insensitive test.
I.e. the actual code seems to contradict the comment.

gdha commented at 2019-01-09 07:40:

@jsmeix @gozora Is this PR still relevant as the code around UEFI has been updated multiple time over the last year?

jsmeix commented at 2019-01-09 08:12:

I close it as "won't fix" because in the foreseeable future I will have
no time to fix this pull request so that its changes look o.k. to me.

I am not a Arch Linux user but I assume Arch Linux EFI boot works
meanwhile because I am not aware of an open issue in this area.

[Export of Github issue for rear/rear.]