#1225 PR merged: 320_include_uefi_env made less case sensitive¶

Labels: enhancement, fixed / solved / done, minor bug

ProBackup-nl opened issue at 2017-03-08 19:47:¶

and also allow /boot to be completely a VFAT file system, instead of just /boot/efi (rear/rear#1223)

gozora commented at 2017-03-09 09:34:¶

Looks better now, if there are no objections i'll merge it today evening (CET).

V.

gdha commented at 2017-03-14 17:53:¶

@ProBackup-nl on an UEFI system:

$ find /boot -maxdepth 1 -iname efi -type d
/boot/efi
[gdha@antwerp rear]$ echo $?
0

and on a non-UEFI system:

# find /boot -maxdepth 1 -iname efi -type d
[root@centos7-kvm gdha]# echo $?
0

and therefore, we get now on non-UEFI systems:

ERROR: Cannot find required programs: dosfsck efibootmgr
Aborting due to an error, check /var/log/rear/rear-centos7-kvm.log for details

Should have tested it more carefully.

jsmeix commented at 2017-03-15 13:26:¶

I did
https://github.com/rear/rear/pull/1246
a proposal how I think the test for
the /boot/efi directory might be simplified.
Currently I do not understand the logic, cf
https://github.com/rear/rear/commit/6c477d4e99edeb21a1a8c7649c24b2a56de535a7#commitcomment-21329841
and therefore I ask for a careful review.

---

[Export of Github issue for rear/rear.]