[Bug 766319] New: fontconfig: hinting configuration in /etc/fonts/conf.d/*.conf is ignored
https://bugzilla.novell.com/show_bug.cgi?id=766319 https://bugzilla.novell.com/show_bug.cgi?id=766319#c0 Summary: fontconfig: hinting configuration in /etc/fonts/conf.d/*.conf is ignored Classification: openSUSE Product: openSUSE 12.2 Version: Beta 1 Platform: i586 OS/Version: openSUSE 12.2 Status: NEW Severity: Major Priority: P5 - None Component: Other AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: ftake@geeko.jp QAContact: qa-bugs@suse.de Found By: --- Blocker: --- User-Agent: Opera/9.80 (Macintosh; Intel Mac OS X 10.7.3; U; ja) Presto/2.10.229 Version/11.64 IPA fonts, which is Japanese default fonts, cannot be well rendored by freetype when hinging is enabled, thereby hinting have been disabled for those fonts by /etc/fonts/conf.d/21-ipahinting.conf. However, on openSUSE 12.2 beta, hinting configuration described in that file is not applied. I tried the latest fontconfig from the M17N repository on 12.1. This problem was demonstrated but running $ SuSEconfig --module fonts fix this problem. On 12.2, this problem cannot be resolved in this way. Since hinting setting is changed to hintfull from 12.2, the result of rendering is worse than 12.1 with hinting enabled. Reproducible: Always Steps to Reproduce: 1. Install openSUSE 12.2 beta in Japanese 2. run SUSEgreeter -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c
kk zhang
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c1
Takashi Iwai
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c2
Petr Gajdos
This is a regression by the update of /etc/fonts/suse-hinting.conf, which is read from conf.d/50-suse-pre-user.conf. It overrides the font hintstyle setups prior 50-*.
As a temporary solution, I changed the priority of ipa-*.conf from 21 to 51. Submitted to FACTORY via SRID 124604 and 124605.
Takashi, I think we could do permanent solution :-). According to /etc/fonts/conf.d/README, font rendering options should really have priority 20-29. So I would link suse-hinting.conf into conf.d/20-suse-hinting.conf. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c3
Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c4
--- Comment #4 from Bernhard Wiedemann
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c5
Takashi Iwai
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c6
--- Comment #6 from Fuminobu Takeyama
According to /etc/fonts/conf.d/README, font rendering options should really have priority 20-29. So I would link suse-hinting.conf into conf.d/20-suse-hinting.conf.
Is "10 through 19 system rendering defaults" also acceptable? How about 1?-default-hinting.conf: for default value 20-font-specific-hinting.conf: for some fonts like MS* 2?-*-hinting.conf In suse-hinting.conf, autohint is enabled for MS Japanese fonts at first but then autohint is disable for CJK. It's strange. I think we need review of font configuration. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c7
--- Comment #7 from Takashi Iwai
I'm not expoert of fontconfig but I think configuration of autohint and hintstyle can be written in no order.
Not really. When you look at suse-hinting.conf, it sets autohinter at first for TrueType fonts: <match target="font"> <test name="fontformat"> <string>TrueType</string> </test> <edit name="autohint"> <bool>false</bool> </edit> </match> then clearing autohinter for some known fonts <match target="font"> <test name="fontformat"> <string>TrueType</string> </test> <test name="family"> <string>Verdana</string> <string>Liberation Sans</string> <string>MS Gothic</string> ..... </test> <edit name="autohint"> <bool>true</bool> </edit> </match> and so on. Then at the last of suse-hinting.conf, it sets hintstyle *depending* on the autohint value. <match target="font"> <test name="autohint"> <bool>false</bool> </test> <edit name="hintstyle"> <const>hintfull</const> </edit> </match> <match target="font"> <test name="autohint"> <bool>true</bool> </test> <edit name="hintstyle"> <const>hintslight</const> </edit> </match> Because of this dependency, there is some order in the config lines. Now, imagine you put a new font and you want to disable autohinter. You can achieve it overriding both autohint and hintstyle after suse-hinting.conf, too. But, if the last part setting the hintstyle were split, you can put your own config setting only autohint=false before the hintstyle setup, then hintstyle will be automatically adjusted. (The case of IPA fonts doesn't match with this, though, because we want to disable it completely.) That is, we'd have four parts: - default autohint setup; we may or may not split font-specific default to another file - opt-in autohint setup by other packages - default hintstyle setup - other autohint/hintstyle overrides (e.g. ipa-*.conf)
According to /etc/fonts/conf.d/README, font rendering options should really have priority 20-29. So I would link suse-hinting.conf into conf.d/20-suse-hinting.conf.
Is "10 through 19 system rendering defaults" also acceptable?
Yes, the suse-hinting.conf should be put in 10-19 region, indeed. Then the font-specific overrides can be put in 20-29.
How about 1?-default-hinting.conf: for default value 20-font-specific-hinting.conf: for some fonts like MS* 2?-*-hinting.conf
This won't work because of the dependency of hintstyle on autohint, if you want to change only autohint but keep hintstyle as system default. Maybe like this: 10-default-autohint.conf 1[1-6]-$font-autohint.conf 17-default-hintstyle.conf 2?-$font-hinting.conf
In suse-hinting.conf, autohint is enabled for MS Japanese fonts at first but then autohint is disable for CJK. It's strange. I think we need review of font configuration.
The block of CJK test is commented out, so it's not evaluated there. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c8
--- Comment #8 from Takashi Iwai
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c9
--- Comment #9 from Takashi Iwai
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c10
--- Comment #10 from Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c11
--- Comment #11 from Bernhard Wiedemann
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c12
Petr Gajdos
This is an autogenerated message for OBS integration: This bug (766319) was mentioned in https://build.opensuse.org/request/show/124917 Factory / fonts-config https://build.opensuse.org/request/show/124918 Factory / fontconfig
Closing as fixed. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c13
Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c14
Petr Gajdos
fc-match -v IPAGothic | grep "hint" hintstyle: 0(i)(w) hinting: FcTrue(s) autohint: FcFalse(w)
M17N:fonts:
fc-match -v IPAGothic | grep "hint" hintstyle: 1(i)(w) hinting: FcTrue(s) autohint: FcTrue(w)
Which one is actually correct? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c15
--- Comment #15 from Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c16
Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c17
--- Comment #17 from Fuminobu Takeyama
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c18
--- Comment #18 from Fuminobu Takeyama
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c19
Takashi Iwai
Debian where many Japanese developers are living uses: :hintstyle=3:hinting=True:autohint=False # In this setting, we know "_" is not displayed.
Both Fedora and Debian use not IPA(P)Gothic but VL fonts as default fonts for Japanese.
On Ubuntu: fc-match "TakaoPGothic" hinting hintstyle autohint :hintstyle=1:hinting=True:autohint=False
Takao fonts is basically same as old IPA fonts. I guess Takao fonts does not have hinting information. So actually autohinting is used.
Since they use their special FreeType2, rendered glyphs are extremely beautiful.
It seems that we have to turn off hinting for *Mincho fonts.
For *Gothic fonts, depending on font size and a character.
Although I said "Let's enable autohinter", it might be better to be conservative until next FreeType (the infinality patch is merged) is released.
Takeyama-san, do you have already the fixed package ready for submission? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c20
--- Comment #20 from Fuminobu Takeyama
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c21
--- Comment #21 from Petr Gajdos
Is BCI *really* disabled under autohint=True?
Yes, as far as I know. autohint=true means 'use autohinter' and autohint=false means 'use native hinting instructions (via BCI)'. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c22
--- Comment #22 from Petr Gajdos
Not yet. # And I cannot find Petr's fontconfig.
home:pgajdos:maintenance:fontconfig/fontconfig.openSUSE_12.2_Update as I said yet in comment 16. Nevertheless I would recommend you to branch package yourself, if you want other change than described in comment 16 (which would preserve 12.1 behaviour). I cannot do anything here, because I don't understand from your previous comments which setting would be the best. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c23
--- Comment #23 from Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c24
--- Comment #24 from Fuminobu Takeyama
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c25
--- Comment #25 from Petr Gajdos
Maybe, to increase probability that fix will be accepted into 12.2, maybe we could only change ipa font packages. ipa-fonts, for example could hold the *.conf file and other packages would Require ipa-fonts (well, not correct, but soft option). Takashi, what do you think?
Ok, the plan is (12.2 only!), to add to each ipa font package (ipa-fonts, ipa-ex-fonts, ipa-uigothic-fonts and ipa-bolditalic-fonts) own unique conf file. For the record: ipa-fonts: IPAPGothic, IPAGothic, IPAPMincho, IPAMincho ipa-uigothic-fonts: dtto ipa-ex-fonts: IPAexGothic, IPAexMincho ipa-bolditalic-fonts: IPAGothic Bold, IPAGothic Italic, IPAGothic BoldItalic, IPAPGothic Bold, IPAPGothic Italic, IPAPGothic BoldItalic -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c26
--- Comment #26 from Petr Gajdos
Ok, the plan is (12.2 only!), to add to each ipa font package (ipa-fonts, ipa-ex-fonts, ipa-uigothic-fonts and ipa-bolditalic-fonts) own unique conf file.
For the record: ipa-fonts: IPAPGothic, IPAGothic, IPAPMincho, IPAMincho ipa-uigothic-fonts: dtto
Err: ipa-uigothic-fonts package contains IPAUIGothic family
ipa-ex-fonts: IPAexGothic, IPAexMincho ipa-bolditalic-fonts: IPAGothic Bold, IPAGothic Italic, IPAGothic BoldItalic, IPAPGothic Bold, IPAPGothic Italic, IPAPGothic BoldItalic
Working on this right now. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c27
--- Comment #27 from Takashi Iwai
Maybe, to increase probability that fix will be accepted into 12.2, maybe we could only change ipa font packages. ipa-fonts, for example could hold the *.conf file and other packages would Require ipa-fonts (well, not correct, but soft option). Takashi, what do you think?
Hmm, this was how we did before 12.2. I was so happy to get rid of these own configs finally... IMO, better to push the FACTORY fontconfig (and fonts-config) to 12.2, too. Divergence at this moment doesn't help for maintenance. And the configuration included in 12.2-RC1 is clearly buggy. It just doesn't appear without CJK or such. Of course, we can change the default hinting for IPA fonts at the same time, if hintnone is really preferred. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c28
Petr Gajdos
(In reply to comment #23)
Maybe, to increase probability that fix will be accepted into 12.2, maybe we could only change ipa font packages. ipa-fonts, for example could hold the *.conf file and other packages would Require ipa-fonts (well, not correct, but soft option). Takashi, what do you think?
Hmm, this was how we did before 12.2. I was so happy to get rid of these own configs finally...
Takashi, I like it too, that would of course remain in factory for sure.
IMO, better to push the FACTORY fontconfig (and fonts-config) to 12.2, too. Divergence at this moment doesn't help for maintenance. And the configuration included in 12.2-RC1 is clearly buggy. It just doesn't appear without CJK or such.
Of course, we can change the default hinting for IPA fonts at the same time, if hintnone is really preferred.
http://lists.opensuse.org/opensuse-factory/2012-07/msg00153.html Maybe you want to talk to coolo, good luck :-). -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c29
Takashi Iwai
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c30
--- Comment #30 from Stephan Kulow
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c31
Petr Gajdos
I find the changes done too intrusive to change just now. But I already said on some list (and I can't find the post right now) that I'm ok with a maintenance update to verify the fix. But that wasn't done from what I know.
You mean http://lists.opensuse.org/opensuse-factory/2012-07/msg00209.html ? That sounds to me like "do maintenance update with least impact". That's what I am trying to do with changing only ipa-*font packages. So, to be sure, you are for updating 12.2/font{,s-}config to newest factory version? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c32
--- Comment #32 from Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c33
--- Comment #33 from Stephan Kulow
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c34
Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c35
Takashi Iwai
Btw, Takashi, just curious, what how is present 12.2's configuration broken (except ipa-*fonts)?
The problem is the rule to enable BCI and autohinter. In FACTORY, it's split to 16-suse-hintstyle.conf while it's in a single file on 12.2. As mentioned in comment 7, there is no chance to put the proper setups between the definition and the autohinter/BCI enablement rule because all are set in a single file. This broke the setups done by fonts-config script (which means all CJK fonts), for example, in addition to the original bug (ipa fonts hinting). Yes, we can provide override of hintstyle and autohinter as a quick fix-up. But then this has to be provided for *all* CJK fonts. And, still the setups user changed via sysconfig/fontconfig file are ignored. This regression can't be fixed by that alone, but requires the split of config files as we have on FACTORY. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c36
Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c37
--- Comment #37 from Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c38
--- Comment #38 from Petr Gajdos
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c39
--- Comment #39 from Takashi Iwai
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c40
--- Comment #40 from Petr Gajdos
Thanks Petr. I tested your packages (fontconfig and fonts-config) now and they seem working fine. The config for IPA fonts is still old (i.e. hinting active), but I guess it's left so intentionally.
That's what I am asking in comment 36: hintstyle: 1(i)(w) hinting: FcTrue(s) autohint: FcTrue(w) is provided for IPA fonts since your ------------------------------------------------------------------- Tue Jun 12 10:41:54 CEST 2012 - tiwai@suse.de [....] - Set autohint=true for IPA* fonts as well as MS fonts; hintslight will be chosen automatically (bnc#766319) [....] , right? I am confused now what setting is the right one.
Speaking ipa-fonts configs: between 12.1 and 12.2, we had already a package ipa-fonts-config.rpm. This was equivalent what you have in ipa-fonts.rpm.
You mean in home:pgajdos:maintenance:? I hope these ipa-* packages are obsoleted by update of fontconfig and fonts-config itself (which I hadn't taken into account before).
Or, if we want to fix in fontconfig itself, it's fine, too.
Definitely. Please, what setting should be used for IPA* families? If I am not clear please tell me. Plan was: update all ipa-* packages only with own configuration. Plan is: update fontconfig and fonts-config packages only, to factory version plus setting for IPA fonts, which is: -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c41
--- Comment #41 from Takashi Iwai
(In reply to comment #39)
Thanks Petr. I tested your packages (fontconfig and fonts-config) now and they seem working fine. The config for IPA fonts is still old (i.e. hinting active), but I guess it's left so intentionally.
That's what I am asking in comment 36:
hintstyle: 1(i)(w) hinting: FcTrue(s) autohint: FcTrue(w)
is provided for IPA fonts since your
------------------------------------------------------------------- Tue Jun 12 10:41:54 CEST 2012 - tiwai@suse.de
[....] - Set autohint=true for IPA* fonts as well as MS fonts; hintslight will be chosen automatically (bnc#766319) [....]
, right? I am confused now what setting is the right one.
Right. As Takeyama-san reported, the current setting doesn't seem working well for all IPA fonts, so I guess your proposal to set hintstyle=0 should be OK.
Speaking ipa-fonts configs: between 12.1 and 12.2, we had already a package ipa-fonts-config.rpm. This was equivalent what you have in ipa-fonts.rpm.
You mean in home:pgajdos:maintenance:? I hope these ipa-* packages are obsoleted by update of fontconfig and fonts-config itself (which I hadn't taken into account before).
Or, if we want to fix in fontconfig itself, it's fine, too.
Definitely. Please, what setting should be used for IPA* families?
If I am not clear please tell me. Plan was: update all ipa-* packages only with own configuration. Plan is: update fontconfig and fonts-config packages only, to factory version plus setting for IPA fonts, which is:
Missing comment? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c42
--- Comment #42 from Petr Gajdos
Right.
As Takeyama-san reported, the current setting doesn't seem working well for all
I see, thanks. I didn't catch that perfectly until now.
to factory version plus setting for IPA fonts, which is:
Missing comment?
No, just second appeal to clarify :-). Takashi, due some security bugs in php and graphic libraries I don't have time for studying your previous changes much; 1. To remove IPA* families from 11-suse-hinting.conf and 2. to add following snippet at the end of 16-suse-hintstyle.conf <match target="font"> <test name="family" compare="contains"> <string>IPA</string> </test> <edit name="hintstyle"> <const>hintnone</const> </edit> </match> is correct and should do the job? [offtopic, don't know what's the difference between hinting=false and hinting=true,hintstyle=hintnone] -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c43
--- Comment #43 from Takashi Iwai
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c44
--- Comment #44 from Petr Gajdos
The tag <string>IPA</string> won't match, unfortunately. You need to specify all variants, <string>IPAMincho</string>, <string>IPAPMincho</string>, etc.
Note there is operator "contains". Of course, if you think it is not reilable, I can change that. Nevertheless it worked for me well when I was testing it.
Also, I guess we don't have to remove IPA* entries in 11-suse-hinting.conf. Instead, just put <edit name="autohint"> <bool>false</bool> </edit> in 16-suse-hintstyle.conf together with a comment that this whole block is a temporary workaround and should be removed later.
We don't have to, nevertheless why to set up autohint on for IPA* entries and then setting it back off is at least confusing for reader. Especially, when comment in 11-suse-hinting.conf explicitely lists IPA fonts as fonts which should not be rendered with autohinter. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c45
--- Comment #45 from Takashi Iwai
(In reply to comment #43)
The tag <string>IPA</string> won't match, unfortunately. You need to specify all variants, <string>IPAMincho</string>, <string>IPAPMincho</string>, etc.
Note there is operator "contains". Of course, if you think it is not reilable, I can change that. Nevertheless it worked for me well when I was testing it.
Ah I overlooked it. Yes, it should work. But maybe safer to list all these, as three letters might match with others :)
Also, I guess we don't have to remove IPA* entries in 11-suse-hinting.conf. Instead, just put <edit name="autohint"> <bool>false</bool> </edit> in 16-suse-hintstyle.conf together with a comment that this whole block is a temporary workaround and should be removed later.
We don't have to, nevertheless why to set up autohint on for IPA* entries and then setting it back off is at least confusing for reader. Especially, when comment in 11-suse-hinting.conf explicitely lists IPA fonts as fonts which should not be rendered with autohinter.
The comment in 11-suse-hinting.conf is obsoleted. The comment should be removed, at least. Actually I don't mind too much about addition/deletion. Maybe it's better to avoid confusion right now... -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c46
--- Comment #46 from Petr Gajdos
Note there is operator "contains". Of course, if you think it is not
Ah I overlooked it. Yes, it should work. But maybe safer to list all these, as three letters might match with others :)
Yeah, and more over, "contains" operator is not documented as far as I see, so it is not supported and might be more buggy than expected :-).
Actually I don't mind too much about addition/deletion. Maybe it's better to avoid confusion right now...
Following patch is sent to home:pgajdos:maintenance:fontconfig. Please test as soon as it is built. I will submit maintenance update tomorrow morning. Index: 11-suse-hinting.conf =================================================================== --- 11-suse-hinting.conf (revision a9740df189a7fff5dcf91afa9525f618) +++ 11-suse-hinting.conf (working copy) @@ -149,13 +149,6 @@ <string>HGPKyokashotai</string> <string>HGSSoeiKakupoptai</string> <string>HGSeikaishotaiPRO</string> - <string>IPAexMincho</string> - <string>IPAexGothic</string> - <string>IPAMincho</string> - <string>IPAGothic</string> - <string>IPAPMincho</string> - <string>IPAPGothic</string> - <string>IPAUIGothic</string> <string>TLKyokashotai</string> <string>TLMincho</string> <string>TLPMincho</string> Index: 16-suse-hintstyle.conf =================================================================== --- 16-suse-hintstyle.conf (revision a9740df189a7fff5dcf91afa9525f618) +++ 16-suse-hintstyle.conf (working copy) @@ -119,4 +119,28 @@ </edit> </match> + <!-- IPA* fonts: setting hintstyle=hintnone for now, IPA font + setting should be improved in the future [bnc#766319] --> + + <match target="font"> + <test name="family"> + <string>IPAexGothic</string> + <string>IPAexMincho</string> + <string>IPAGothic BoldItalic</string> + <string>IPAGothic Bold</string> + <string>IPAGothic Italic</string> + <string>IPAGothic</string> + <string>IPAMincho</string> + <string>IPAPGothic BoldItalic</string> + <string>IPAPGothic Bold</string> + <string>IPAPGothic Italic</string> + <string>IPAPGothic</string> + <string>IPAPMincho</string> + <string>IPAUIGothic</string> + </test> + <edit name="hintstyle"> + <const>hintnone</const> + </edit> + </match> + -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c47
--- Comment #47 from Fuminobu Takeyama
[offtopic, don't know what's the difference between hinting=false and hinting=true,hintstyle=hintnone] # As far as I tested after comment #20, this attribute seems to be ignored.
# This is also off topic and less serious than this, # $ fc-match "IPAゴシック" hintstyle # still returns 3 although "IPAゴシック" and "IPAGothic" refer the same font. # As far as I know, Firefox and Opera uses this Japanese name. # I guess this problem is also in 12.1. (not tried yet) # # The spec of fontconfig is really unclear. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c48
--- Comment #48 from Petr Gajdos
I installed your package in clean 12.2 RC1. Now, IPA Fonts work as 12.1. Thanks a lot!
Thanks a lot also to you, both for reporting and testing, and you are welcome to improve IPA fontconfig setting for 12.3! Just keep in touch with us.
[offtopic, don't know what's the difference between hinting=false and hinting=true,hintstyle=hintnone] # As far as I tested after comment #20, this attribute seems to be ignored.
Sorry, I feel this sentence ambiguous. Which attribute you find ignored and when?
# This is also off topic and less serious than this, # $ fc-match "IPAゴシック" hintstyle # still returns 3 although "IPAゴシック" and "IPAGothic" refer the same font. # As far as I know, Firefox and Opera uses this Japanese name. # I guess this problem is also in 12.1. (not tried yet)
Fortunately "fullname" element is always english no matter on request's language, so following change should do the job. <match target="font"> - <test name="family"> + <test name="fullname"> <string>IPAexGothic</string> <string>IPAexMincho</string> <string>IPAGothic BoldItalic</string> I have sent maintenance requests for fontconfig and fonts-config (mr#, mr#) yet, but I still would appreciate yet another feedback from you.
# The spec of fontconfig is really unclear.
You mean fontconfig.spec or fontconfig specification :-)? -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c49
Petr Gajdos
I have sent maintenance requests for fontconfig and fonts-config (mr#, mr#)
mr#129056 mr#129057 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=766319
https://bugzilla.novell.com/show_bug.cgi?id=766319#c50
--- Comment #50 from Bernhard Wiedemann
participants (1)
-
bugzilla_noreply@novell.com