<div dir="ltr">On Sun, Feb 8, 2015 at 11:17 AM, Ryan Schmidt <span dir="ltr"><<a href="mailto:cairo-2015@ryandesign.com" target="_blank">cairo-2015@ryandesign.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class=""><br>
On Feb 5, 2015, at 10:42 AM, Andrea Canciani wrote:<br>
<br>
>> 2) the following patch fixes runtime warnings about CGFontGetGlyphPath deprecation under OS X 10.10 (Yosemite):<br>
>>    (discussion at <a href="http://comments.gmane.org/gmane.comp.lib.cairo/24875" target="_blank">http://comments.gmane.org/gmane.comp.lib.cairo/24875</a>)<br>
><br>
> I pushed the patch available in the associated bugreport to master:<br>
> commit 70cc8f250b5669e757b4f044571ba0f71e3dea9e<br>
> Author: Andrea Canciani <<a href="mailto:ranma42@gmail.com">ranma42@gmail.com</a>><br>
> Date:   Tue Dec 9 16:13:00 2014 +0100<br>
><br>
>     quartz: Remove call to obsolete CGFontGetGlyphPath<br>
><br>
>     CGFontGetGlyphPath was not public and is not available anymore on<br>
>     modern OSX/iOS systems. The same functionality is available through<br>
>     the CoreText API since OSX 10.5.<br>
><br>
>     Based on a patch by Simon Cozens.<br>
><br>
>     Fixes <a href="https://bugs.freedesktop.org/show_bug.cgi?id=84324" target="_blank">https://bugs.freedesktop.org/show_bug.cgi?id=84324</a><br>
<br>
</span>With this change, cairo now cannot be built on OS X 10.4 with the --enable-quartz-font flag, whereas this worked before:<br></blockquote><div><br></div><div>This is expected, in fact it is one of the issues I had pointed out on the original patch:<br><a href="http://permalink.gmane.org/gmane.comp.lib.cairo/24905">http://permalink.gmane.org/gmane.comp.lib.cairo/24905</a><br><br></div><div>(I also sketched an alternative patch that should/might also work on 10.4)<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
:info:build cairo-quartz-font.c: In function '_cairo_quartz_init_glyph_path':<br>
:info:build cairo-quartz-font.c:548: error: 'CTFontRef' undeclared (first use in this function)<br>
:info:build cairo-quartz-font.c:548: error: (Each undeclared identifier is reported only once<br>
:info:build cairo-quartz-font.c:548: error: for each function it appears in.)<br>
:info:build cairo-quartz-font.c:548: error: expected ';' before 'ctFont'<br>
:info:build cairo-quartz-font.c:563: error: 'ctFont' undeclared (first use in this function)<br>
:info:build cairo-quartz-font.c:563: error: implicit declaration of function 'CTFontCreateWithGraphicsFont'<br>
:info:build cairo-quartz-font.c:564: error: implicit declaration of function 'CTFontCreatePathForGlyph'<br>
:info:build cairo-quartz-font.c:564: warning: assignment makes pointer from integer without a cast<br>
:info:build make[3]: *** [cairo-quartz-font.lo] Error 1<br>
<br>
<br>
Would it be possible to keep the previous code for OS X 10.4 and use the new code on OS X 10.5 and newer?<br></blockquote><div><br></div><div>It is possible to do that, but testing on 10.4 is becoming pretty hard (I have no such environment and I'm afraid that Mozilla is not testing anymore on such an old OS). In the bugreport the agreement seemed to be that 10.4 is too old to be supported. Its removal would also allow for a significant cleanup, as 10.4 is missing blend operators.<br><br></div><div>I am unsure what the best course is... Chris, Bryce, what are your opinions on this?<br><br></div><div>Andrea<br></div></div></div></div>