FS#76654 - [mkinitcpio] new sd-vconsole patch cannot deal with "regular" font files

Attached to Project: Arch Linux
Opened by Gunnar Bretthauer (Taijian) - Thursday, 24 November 2022, 11:45 GMT
Last edited by Toolybird (Toolybird) - Wednesday, 07 December 2022, 21:41 GMT
Task Type Bug Report
Category Packages: Core
Status Closed
Assigned To Giancarlo Razzolini (grazzolini)
Morten Linderud (Foxboron)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Commit a6ba032963c6f7da02ee2b8e4753ea8d0469787c changed the way the sd-vconsole hook checks for $FONT files to add to the initramfs. Since then, there is an unfortunate interaction with plymouth-git. plymouth-git declares (and uses) $FONT in its install script, setting $FONT to the full file path to the console font (typically ending in .ttf|.otf or somesuch).
sd-vconsole will now find this $FONT variable, try to resolve it, fail, and then throw an error message - likely because the new regex is faulty.

Additional info:
* v33
* ERROR: sd-vconsole: requested font not found: '/usr/share/fonts/noto/NotoSans-Regular.ttf'
* I'd love to have created one, but the account creation page for Archlinux SSO seems to be buggy - I tried to 2FA confirm with 3 different apps, but it wouldn't let me... So, no, no bug report from me

Steps to reproduce:
Install plymouth-git. Run mkinitcpio. See error message.
This task depends upon

Closed by  Toolybird (Toolybird)
Wednesday, 07 December 2022, 21:41 GMT
Reason for closing:  Fixed
Additional comments about closing:  mkinitcpio 34-1
Comment by nl6720 (nl6720) - Thursday, 24 November 2022, 12:13 GMT
https://aur.archlinux.org/cgit/aur.git/tree/plymouth.initcpio_install?h=plymouth-git and https://aur.archlinux.org/cgit/aur.git/tree/sd-plymouth.initcpio_install?h=plymouth-git should declare the FONT variable as local.

But I guess, so should the consolefont, keymap and sd-vconsole hooks.
Comment by Jelle van der Waa (jelly) - Thursday, 24 November 2022, 12:16 GMT
@taijian What 2FA apps did you try?
Comment by nl6720 (nl6720) - Thursday, 24 November 2022, 12:37 GMT
I got a potential fix, please test: https://gitlab.archlinux.org/archlinux/mkinitcpio/mkinitcpio/-/merge_requests/149
Comment by Gunnar Bretthauer (Taijian) - Friday, 25 November 2022, 09:46 GMT
@jelly: On my phone I tried KeepassDX and Aegis and after that I also tried KeepassXC on my laptop. Neither of those worked.
Comment by Gunnar Bretthauer (Taijian) - Friday, 25 November 2022, 09:48 GMT
@nl6720: OK, I went ahead and pushed an update to plymouth-git to declare all used variables as local; error message went away. In my defence, my package was kinda just doing what the repo packages where also doing ;)
Comment by nl6720 (nl6720) - Sunday, 27 November 2022, 13:46 GMT
Fixed in https://gitlab.archlinux.org/archlinux/mkinitcpio/mkinitcpio/-/commit/50108c2b5b40d9b231e6d4f61f14b8187f0e62ed pending mkinitcpio v34 release.

Loading...