<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Jul 7, 2015 at 2:30 PM, Lawrence D'Oliveiro <span dir="ltr"><<a href="mailto:ldo@geek-central.gen.nz" target="_blank">ldo@geek-central.gen.nz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">On Mon, 6 Jul 2015 14:24:54 -0700, Bryce Harrington wrote:<br>
<br>
> On Thu, Jul 02, 2015 at 11:52:15AM +0200, Andrea Canciani wrote:<br>
><br>
>> On Thu, Jul 2, 2015 at 12:05 AM, Lawrence D'Oliveiro <<br>
>> <a href="mailto:ldo@geek-central.gen.nz">ldo@geek-central.gen.nz</a>> wrote:<br>
>><br>
>>> On Wed,  1 Jul 2015 12:00:49 +0200, Andrea Canciani wrote:<br>
>>><br>
>>>> This change should not affect Linux environments, assuming that<br>
>>>> the default font is already set to "DejaVu Sans".<br>
>>><br>
>>> On Linux environments, I would expect to see Fontconfig in use.<br>
>>> This can take a generic name like “Sans Serif” and map it to<br>
>>> whatever font the user desires.<br>
>><br>
>> Yes, but (for the test suite) if this happens you are likely already<br>
>> getting failures when you test cairo, because unless the<br>
>> user-chosen font glyphs have the same shape as the DejaVu Sans<br>
>> ones, the output of the test cases will not match the reference.<br>
>> Please notice that most tests already set the font to<br>
>> CAIRO_TEST_FONT_FAMILY " ...".<br>
><br>
</span><span class="">> Right, for the test suite, there is no user to have desires.  :-)<br>
> Goal is to just make sure results match up to the references.<br>
<br>
</span>However, even a specific-sounding name like “DejaVu Sans” can be<br>
remapped by Fontconfig. If a test suite can be led astray by vagaries<br>
of user configuration, then isn’t that a bug in the test suite?<br></blockquote><div><br></div><div>From that point of view, I believe we expect users to actively try and get a "working" configuration to pass the test suite (and provide some directions in test/README).<br></div><div>Just like we are not preventing a user from changing its font configuration, we cannot really prevent him/her from replacing pixman, fontconfig, freetype and the whole X server with other libraries/programs which misbehave in arbitrary ways (possibly preserving the original API, but completely changing the behaviour).<br></div><div>I do not think that making the testsuite "completely" robust (for example by including all of the cairo dependencies as submodules and using them in a more controlled way) is a good target. Instead I agree that for some resources (like fonts) we should try harder to bundle them: currently we do not even check if the font subsystem knows about DejaVu, so it is very easy to end up running the test suite in an environment which is missing some fonts (this is especially true on Windows and MacOSX, I believe).<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>
If you want to depend on a specific font in the test suite, then the<br>
font needs to be included in the test suite.<br></blockquote><div><br></div><div>That's a long standing issue (unrelated to this change), which is even mentioned in the test/README file:<br><br>"We are constantly working to reduce the number of variables that need<br>to be tweaked to get a clean run, (for example, by bundling fonts with<br>the test suite itself), and also working to more carefully document<br>the software configuration used to generate the reference images."<br><br></div><div>IIRC there are some licensing issues that might prevent us from bundling the DejaVu font with the test suite.<br></div><div>We might eventually decide to bundle another font (for example we use the public domain font test/6x13.pcf), or at least to provide an automated way to fetch the appropriate files from a trusted source (Debian packages?). This seems to be material for an independent discussion, though.<br><br></div><div>Andrea<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class=""><div class="h5">--<br>
cairo mailing list<br>
<a href="mailto:cairo@cairographics.org">cairo@cairographics.org</a><br>
<a href="http://lists.cairographics.org/mailman/listinfo/cairo" rel="noreferrer" target="_blank">http://lists.cairographics.org/mailman/listinfo/cairo</a></div></div></blockquote></div><br></div></div>