https://bugzilla.novell.com/show_bug.cgi?id=766319 https://bugzilla.novell.com/show_bug.cgi?id=766319#c1 Takashi Iwai changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED CC| |tiwai@suse.com Resolution| |FIXED --- Comment #1 from Takashi Iwai 2012-06-11 18:08:50 UTC --- 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. -- 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 changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |NEEDINFO InfoProvider| |tiwai@suse.com --- Comment #3 from Petr Gajdos 2012-06-11 18:17:46 UTC --- What do you think? -- 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#c5 Takashi Iwai changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |REOPENED InfoProvider|tiwai@suse.com | --- Comment #5 from Takashi Iwai 2012-06-11 19:28:15 UTC --- Yeah, this would be better. Though, looking at what suse-hints.conf does, I think it's better to split it up to two parts. There is a part setting autohint as default, then there is a part setting hintstyle depending on the autohint value. If any font-specific config is needed, it can be put between them, if these were split and the latter is set later, e.g. 25-default-hintstyle.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#c7 --- Comment #7 from Takashi Iwai 2012-06-12 05:50:45 UTC --- (In reply to comment #6)

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#c9 --- Comment #9 from Takashi Iwai 2012-06-12 11:14:10 UTC --- OK, I found a problem. fonts-config program modifies these config files, so we can't rename without fixing fonts-config part. I updated fonts-config package to have own config files instead of modifying other package's files. Updated again in OBS home:tiwai:branches:M17N/fontconfig and fonts-config repos. -- 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#c11 --- Comment #11 from Bernhard Wiedemann 2012-06-14 12:00:08 CEST --- 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 -- 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 changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P5 - None |P3 - Medium Status|RESOLVED |REOPENED Resolution|FIXED | --- Comment #13 from Petr Gajdos 2012-07-13 09:58:01 UTC --- Reopen in order to fix this for 12.2. -- 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 2012-07-13 11:14:16 UTC --- Fedora has hinting turned off entirely (hinting=false). -- 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#c17 --- Comment #17 from Fuminobu Takeyama 2012-07-13 13:45:31 UTC --- 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. -- 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#c19 Takashi Iwai changed: What |Removed |Added ---------------------------------------------------------------------------- InfoProvider|tiwai@suse.com |ftake@geeko.jp --- Comment #19 from Takashi Iwai 2012-07-16 10:39:20 UTC --- Sorry, I've been on vacation in the last week, so I couldn't follow the discussion. (In reply to comment #17)

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#c21 --- Comment #21 from Petr Gajdos 2012-07-23 10:46:51 UTC --- (In reply to comment #18)

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#c23 --- Comment #23 from Petr Gajdos 2012-07-24 08:39:17 UTC --- 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? Question remains: could someone please provide correct configuration? Or should we use 12.1 setting also for 12.2 (comment 16) to be quick and tune it for next suse release? Takeyama-san, please advise. -- 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#c25 --- Comment #25 from Petr Gajdos 2012-07-24 16:45:22 UTC --- (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?

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#c27 --- Comment #27 from Takashi Iwai 2012-07-25 09:26:31 UTC --- (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... 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#c29 Takashi Iwai changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEEDINFO InfoProvider| |coolo@suse.com --- Comment #29 from Takashi Iwai 2012-07-25 09:59:16 UTC --- Coolo, why this must be blocked for 12.2? The default font configuration files included in 12.2-rc1 are fundamentally broken, as stated in the early comments of this bug. But the bugs become visible only when some settings by fonts-config package are activated. It's the only reason why only few people in Europe have complained so far. -- 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#c31 Petr Gajdos changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |coolo@suse.com --- Comment #31 from Petr Gajdos 2012-07-25 10:45:31 UTC --- (In reply to comment #30)

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#c33 --- Comment #33 from Stephan Kulow 2012-07-25 12:57:43 CEST --- I don't know what I'm for :) I want the solution with the biggest ratio of benefit per risk. If that solution is factory versions, fine with me as long as we test whatever version through update channel *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#c35 Takashi Iwai changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED InfoProvider|tiwai@suse.com | --- Comment #35 from Takashi Iwai 2012-07-25 11:48:15 UTC --- (In reply to comment #32)

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#c37 --- Comment #37 from Petr Gajdos 2012-07-25 15:01:57 UTC --- (I am testing it right now on my workstation and it looks good. Tomorrow I will try another machine.) -- 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#c39 --- Comment #39 from Takashi Iwai 2012-07-26 09:22:19 UTC --- 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. 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. Then this was obsoleted by the update of fontconfig. You can see obsoletes/provides tags there. So, the question is, whether we still need the package for ipa-fonts for providing the own config. If yes, we can remove obsoletes/provides in fontconfig and revive ipa-fonts-config.rpm. Or, if we want to fix in fontconfig itself, it's fine, too. Or, if ipa-fonts.rpm is preferred to ipa-fonts-config.rpm, we should move obsoletes/provides tags to ipa-fonts.spec from fontconfig.spec. -- 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 2012-07-26 10:02:59 UTC --- (In reply to comment #40)

(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#c43 --- Comment #43 from Takashi Iwai 2012-07-26 11:07:36 UTC --- The tag <string>IPA</string> won't match, unfortunately. You need to specify all variants, <string>IPAMincho</string>, <string>IPAPMincho</string>, etc. 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. -- 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 2012-07-26 11:36:48 UTC --- (In reply to comment #44)

(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#c47 --- Comment #47 from Fuminobu Takeyama 2012-07-27 01:30:50 UTC --- I installed your package in clean 12.2 RC1. Now, IPA Fonts work as 12.1. Thanks a lot!

[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#c49 Petr Gajdos changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |RESOLVED InfoProvider|ftake@geeko.jp | Resolution| |FIXED --- Comment #49 from Petr Gajdos 2012-07-27 08:50:13 UTC --- (In reply to comment #48)

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.