external/more_fonts/ExternalPackage_noto_sans_arabic.mk |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit cdd393a8a519236987e7fbcf69321ebec5121f26
Author:     Michael Stahl <michael.st...@allotropia.de>
AuthorDate: Fri Aug 2 12:48:05 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Thu Aug 22 10:44:24 2024 +0200

    more_fonts: ship "hinted" instead of "full" for "Noto Sans Arabic"
    
    There is a really annoying problem with "Noto Sans Arabic" on Fedora:
    
    The fontconfig file /etc/fonts/conf.d/56-google-noto-sans-arabic-vf.conf
    in the google-noto-sans-arabic-vf-fonts rpm is sorted before
    /etc/fonts/conf.d/56-google-noto-sans-vf.conf and sets it as default
    without any "lang" restriction:
    
      <alias>
        <family>Noto Sans Arabic</family>
        <default>
          <family>sans-serif</family>
        </default>
      </alias>
    
    This is not an issue with the google-noto-sans-arabic-vf-fonts rpm's
    /usr/share/fonts/google-noto-vf/NotoSansArabic[wght].ttf because it
    contains only appropriate languages:
    
        lang: ar|az-ir|fa|ks|ku-ir|ps-af|ps-pk|ug|ur|ku-iq|ota|pa-pk|sd|lah(s)
        fontversion: 131858(i)(s)
    
    However the instdir/share/fonts/truetype/NotoSansArabic-Regular.ttf
    contains extra languages:
    
        lang: 
aa|ar|ay|az-ir|bi|br|bs|ch|co|cs|cy|da|de|en|es|et|eu|fa|fi|fj|fo|fr|fur|fy|gd|gl|gv|ho|hr|hu|ia|id|ie|io|is|it|ks|ku-ir|lb|lt|lv|mg|mh|mt|nb|nds|nl|nn|no|nr|nso|ny|oc|om|pl|ps-af|ps-pk|pt|rm|ro|sk|sl|sma|smj|smn|so|sq|ss|st|sv|sw|tk|tl|tn|tr|ts|ug|ur|uz|vo|vot|wa|wen|wo|xh|yap|zu|an|crh|csb|fil|hsb|ht|jv|kj|ku-iq|ku-tr|kwm|lg|li|ms|ng|ota|pa-pk|pap-an|pap-aw|rn|rw|sc|sd|sg|sn|su|za|lah(s)
        fontversion: 131727(i)(s)
    
    Now the combination of the rpm config file and the LO bundled ttf causes
    fontconfig to prefer "Noto Sans Arabic" over "Noto Sans" for latin
    script, which is a problem because the fonts have different ascent,
    which causes test CppunitTest_sw_uiwriter8 testTdf157129 to fail in a
    rather non-obvious manner.
    
    Instead of the "full" build of the font, bundle the "hinted" build,
    which contains only appropriate languages.
    
    There is also an inconclusive Fedora bugreport about similar but
    different problem:
    https://bugzilla.redhat.com/show_bug.cgi?id=2262410
    
    Change-Id: I6632ef454191fbb3ebec7c2daa83cc0e6acd829c
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171401
    Tested-by: Jenkins
    Reviewed-by: Michael Stahl <michael.st...@allotropia.de>
    (cherry picked from commit e6282cff7857999b031bb847a6ca8d996a090db1)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/171766
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/external/more_fonts/ExternalPackage_noto_sans_arabic.mk 
b/external/more_fonts/ExternalPackage_noto_sans_arabic.mk
index 88a5b2e6d444..75e9005c1bc1 100644
--- a/external/more_fonts/ExternalPackage_noto_sans_arabic.mk
+++ b/external/more_fonts/ExternalPackage_noto_sans_arabic.mk
@@ -10,8 +10,8 @@
 $(eval $(call 
gb_ExternalPackage_ExternalPackage,fonts_noto_sans_arabic,font_noto_sans_arabic))
 
 $(eval $(call 
gb_ExternalPackage_add_unpacked_files,fonts_noto_sans_arabic,$(LIBO_SHARE_FOLDER)/fonts/truetype,\
-       NotoSansArabic/full/ttf/NotoSansArabic-Bold.ttf \
-       NotoSansArabic/full/ttf/NotoSansArabic-Regular.ttf \
+       NotoSansArabic/hinted/ttf/NotoSansArabic-Bold.ttf \
+       NotoSansArabic/hinted/ttf/NotoSansArabic-Regular.ttf \
 ))
 
 # vim: set noet sw=4 ts=4:

Reply via email to