[Fontconfig] Re: fontconfig: FcConfigAppFontAddDir () doesn't work right

[Mike FABIAN]

Patrick Lam <plam at MIT.EDU> さんは書きました:

> Mike FABIAN wrote:
>> See 
>> 
>> http://bugzilla.novell.com/show_bug.cgi?id=148693
>
> I'm kind of concerned about this patch potentially bringing back the
> issues we had a while back with 'fc-cache .' not doing the right thing,
> so I can't commit it until I check that issue out.  In particular, if we
> don't normalize before we add a directory, it seems like fc-cache .
> might do the wrong thing.

In the CVS checkout from today some hunks of Takashi's patch are included,
but not all. Without the missing hunks it still doesn't work,
my test program from 

http://bugzilla.novell.com/show_bug.cgi?id=148693

comment #3 still runs into an endless loop with the test-case
from comment #4 of that bug report.

The remaining hunks of Takashi's patch are attached.

Actually the very last hunk:

diff -ru fontconfig-2.3.93.20060210.orig/src/fcdir.c
fontconfig-2.3.93.20060210/src/fcdir.c
--- fontconfig-2.3.93.20060210.orig/src/fcdir.c 2006-02-10 13:06:08.000000000
+0100
+++ fontconfig-2.3.93.20060210/src/fcdir.c      2006-02-10 16:35:30.000000000
+0100
@@ -133,6 +133,8 @@
        d_can = FcConfigNormalizeFontDir (config, dir);
     if (d_can)
        dir = d_can;
+    else
+       FcConfigAddFontDir (config, dir);

     if (!force)
     {

is not necessary, it works without this hunk.

But this hunk seems logical to me, probably it is better to include
this as well.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: fc-app-dir-fix-new-new.diff
Type: text/x-patch
Size: 1994 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/fontconfig/attachments/20060210/216b2af8/fc-app-dir-fix-new-new.bin
-------------- next part --------------

-- 
Mike FABIAN   <mfabian at suse.de>   http://www.suse.de/~mfabian
睡眠不足はいい仕事の敵だ。