diff options
| author |   Matt Turner <mattst88@gentoo.org> | 2022-11-07 20:01:54 -0500 |
|---|---|---|
| committer | Matt Turner <mattst88@gentoo.org> | 2022-11-08 18:48:51 -0500 |
| commit | e4a9c7da92382416ba4a25b2bc04a47820f116c1 (patch) | |
| tree | 9de140912f0019e7e5ad053286a4d5dc31682539 /eclass | |
| parent | media-fonts/x11fonts-jmk: Install files with 0644 permissions (diff) | |
| download | gentoo-e4a9c7da92382416ba4a25b2bc04a47820f116c1.tar.gz gentoo-e4a9c7da92382416ba4a25b2bc04a47820f116c1.tar.bz2 gentoo-e4a9c7da92382416ba4a25b2bc04a47820f116c1.zip | |
font.eclass: Remove racy pkg_postinst code
Noticed on ChromeOS when installing a large number of font packages in
parallel:
/usr/share/fonts/noto/NotoSerifThai-Regular.ttf#new' from 0004 (------r--) to 2440 (r--r-S---)
* ERROR: media-fonts/ipaex-004.01-r1::chromiumos failed (postinst phase):
* failed to fix font files perms
The "#new" filename is the hint. Portage uses "#new" suffixes when
copying files to the system, and then renames them to their final
filenames.
This code was executing while another font was in the process of being
copied to the system. Font packages should just ensure that they install
files with correct permissions to begin with, and all except
media-fonts/x11fonts-jmk already use 0644 permissions.
media-fonts/x11fonts-jmk used 0444 (which was probably fine) until the
previous commit which changes its installed files to 0644.
Bug: https://bugs.gentoo.org/187774
Signed-off-by: Matt Turner <mattst88@gentoo.org>
Diffstat (limited to 'eclass')
| -rw-r--r-- | eclass/font.eclass | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/eclass/font.eclass b/eclass/font.eclass index 4970c959f7cd..0196755ce3e1 100644 --- a/eclass/font.eclass +++ b/eclass/font.eclass @@ -186,12 +186,6 @@ font_src_install() { # @DESCRIPTION: # Updates fontcache if !prefix and media-libs/fontconfig installed _update_fontcache() { - if [[ -d "${EROOT}"/usr/share/fonts ]] ; then - # unreadable font files = fontconfig segfaults - find "${EROOT}"/usr/share/fonts/ -type f '!' -perm 0644 \ - -exec chmod -v 0644 2>/dev/null {} + || die "failed to fix font files perms" - fi - if [[ -z ${ROOT} ]] ; then if has_version media-libs/fontconfig ; then ebegin "Updating global fontcache" |