FS#59165 - [ttf-fira-code] Font package should not have any dependencies

Attached to Project: Community Packages
Opened by Khaled Hosny (khaledhosny) - Thursday, 28 June 2018, 00:26 GMT
Last edited by Jiachen Yang (farseerfc) - Tuesday, 03 July 2018, 04:57 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Jiachen Yang (farseerfc)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Installing ttf-fira-code installs also fontconfig and xorg-font-utils. None of these is required for the fonts to function, and install hooks already take care of updating the font cache. Though depending on fontconfig is not a big issue, the other package is of little use on modern Linux systems.
This task depends upon

Closed by  Jiachen Yang (farseerfc)
Tuesday, 03 July 2018, 04:57 GMT
Reason for closing:  Fixed
Additional comments about closing:  Removed dependencies in 1.205-3
Comment by Jiachen Yang (farseerfc) - Thursday, 28 June 2018, 05:51 GMT
Hooks need to be present in order for them to work. During the transition from post install scripts to pacman hooks, we removed the post install scripts that update fontconfig cache and mkfontdir, and they are replaces by cooresponding pacman hooks. But we keep the dependencies so that the hooks (and the Exec scripts) are present on the system. Maybe I should replace `xorg-font-utils` by `xorg-mkfontdir`, but both dependencies are neccessory from my understanding.
Comment by Eli Schwartz (eschwartz) - Thursday, 28 June 2018, 12:59 GMT
The counter-argument is that a fontconfig cache is completely useless and not *needed* when fontconfig is not installed, so why exactly is the solution to force it to be installed? As soon as you do install it, it will create a full cache anyway.

xorg-mkfontdir should ideally be pulled in by xorg (or software which specifically needs the legacy mkfontdir index), and, hmm, I think this may already be the case via xorg-fonts-misc.

See also  FS#59164 .
Comment by Jiachen Yang (farseerfc) - Tuesday, 03 July 2018, 04:56 GMT
I also not so sure about these dependencies back when we did the fontconfig hook.
Now I decided that keeping these dependencies is indeed a bad idea.
Will remove these in 1.205-3

Loading...