FS#45928 - [fontconfig] [gsfonts] adding new URW names to fontconfig

Attached to Project: Arch Linux
Opened by Vorbote (vorbote) - Saturday, 08 August 2015, 21:24 GMT
Last edited by Gaetan Bisson (vesath) - Thursday, 13 August 2015, 06:28 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Jan de Groot (JGC)
Andreas Radke (AndyRTR)
Gaetan Bisson (vesath)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

The new new URW fonts commissioned by Artifex that Arch uses now for Ghostscript and display cannot be use presently as a replacement for the legacy LaserWriter 35 (Times, Helvetica, Courier, Symbol, etc.) because the fontconfig mappings do not include the new names assigned to the fonts. In a short email exchange with Gaetan Bisson earlier today, I committed to work out a solution.

fontconfig's master contains the updated mappings (see for example [1]). so I propose using this temporary copy of the original upstream patch until a new release of fontconfig includes it.


[1] http://cgit.freedesktop.org/fontconfig/tree/conf.d/30-metric-aliases.conf
This task depends upon

Closed by  Gaetan Bisson (vesath)
Thursday, 13 August 2015, 06:28 GMT
Reason for closing:  Implemented
Additional comments about closing:  fontconfig-2.11.1-2 in [testing]
Comment by Vorbote (vorbote) - Saturday, 08 August 2015, 21:28 GMT
Missed adding a full subject. I'm blushing here.

I suggest, "adding new URW names to fontconfig".
Comment by Gaetan Bisson (vesath) - Sunday, 09 August 2015, 12:10 GMT
That sounds great to me. I'm happy to apply this patch to our fontconfig package unless Jan or Andreas have any objections.
Comment by Vorbote (vorbote) - Monday, 10 August 2015, 02:05 GMT
Hi, it goes to show that one should not get carried away with enthusiasm so easily. :-) The upstream patch doesn't apply cleanly in the most important config file, 30-metric-aliases.conf, so I have prepared a set of hand-made patches that I've tested to death. :-)

Comment by Gaetan Bisson (vesath) - Tuesday, 11 August 2015, 14:32 GMT
Hum. Your patch to 30-metric-aliases.conf applies cleanly, but the last two hunks of the one for 45-latin.conf fails. Also, it seems your patch to 60-latin.conf
is alreay applied. I'm using the conf.d dir from http://www.fontconfig.org/release/fontconfig-2.11.1.tar.bz2 which our package uses. Could you check this?

Also, it would help if what we'd need to patch was just an upstream commit. Would that one be enough? http://cgit.freedesktop.org/fontconfig/commit/conf.d/30-metric-aliases.conf?id=b732bf057f4b3ec3bac539803005e9c42d056b2a If not, could you tell us what other commits are needed. And if further changes are needed, could you push them upstream? Cheers.
Comment by Vorbote (vorbote) - Wednesday, 12 August 2015, 01:12 GMT
Sorry about the bad patches, I somehow mangled my working copies. All can be done with upstream patches, btw.

I missed the *previous* patch to http://cgit.freedesktop.org/fontconfig/commit/conf.d/30-metric-aliases.conf?id=b732bf057f4b3ec3bac539803005e9c42d056b2a because its description is rather opaque. Without it, the main patch doesn't apply cleanly creating a reject that drops the Times mapping. First there's the need to apply http://cgit.freedesktop.org/fontconfig/commit/conf.d/30-metric-aliases.conf?id=e7121de237a1873c3241a5b8451e7d00a3d41524

I'm attaching both upstream patches numbered in the proper order.

Cheers.
Comment by Gaetan Bisson (vesath) - Wednesday, 12 August 2015, 13:04 GMT
I've pushed fontconfig-2.11.1-2 to [testing]. Please let me know if anything is wrong with it. Cheers!
Comment by patrick (potomac) - Wednesday, 12 August 2015, 13:29 GMT
Gaetan : I don't find your fontconfig-2.11.1-2, I get a "404 error" if I click the link "download for mirror here" :

https://www.archlinux.org/packages/testing/x86_64/fontconfig/

and the PKGBUILD has the version "2.11.1-1", it seems that the version 2.11.1-2 is not online
Comment by Gaetan Bisson (vesath) - Wednesday, 12 August 2015, 14:28 GMT
It takes a few minutes to propagate. It's certainly there now. :)
Comment by patrick (potomac) - Wednesday, 12 August 2015, 15:08 GMT
yes I can download now your testing version,

I did some tests and I notice with your testing version that I can't use my personnal workaround for the bug  FS#45938  :

- with fontconfig-2.11.1-1 and gsfonts 20150122-1 --> all is Ok ( good font rendering in phpbb forums )

- with fontconfig-2.11.1-2 and gsfonts 20150122-1 --> bad font rendering in phpbb forums

- with fontconfig-2.11.1-2 and gsfonts 20150808-1 --> bad font rendering in phpbb forums

it seems that these 2 patches have an impact on fonts rendering even with old versions of gsfonts, this could be a problem for people who have the  FS#45938  bug
Comment by Vorbote (vorbote) - Wednesday, 12 August 2015, 17:40 GMT
The new configuration files are working great, thanks Gaetan. Font substitution is working as expected, finally. I'll barge in  FS#45938  ...

Loading...