After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 781340 - Tests / "make check" fail on Mageia v6 x86-64
Tests / "make check" fail on Mageia v6 x86-64
Status: RESOLVED FIXED
Product: GEGL
Classification: Other
Component: core
git master
Other Linux
: Normal normal
: ---
Assigned To: Default Gegl Component Owner
Default Gegl Component Owner
Depends on:
Blocks:
 
 
Reported: 2017-04-15 12:10 UTC by Shlomi Fish
Modified: 2017-06-27 13:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
A patch to fix some of the problems. (1.31 KB, patch)
2017-04-15 12:10 UTC, Shlomi Fish
none Details | Review
Output of "make check" on my system. (39.11 KB, text/plain)
2017-04-15 12:17 UTC, Shlomi Fish
  Details
New patch against the working copy (1.31 KB, patch)
2017-05-04 17:22 UTC, Shlomi Fish
none Details | Review
make check output on mageia v6 x86-64. (39.26 KB, text/plain)
2017-05-04 17:23 UTC, Shlomi Fish
  Details
Extra-new patch against the working copy - same failure (499 bytes, patch)
2017-05-07 16:24 UTC, Shlomi Fish
none Details | Review

Description Shlomi Fish 2017-04-15 12:10:45 UTC
Created attachment 349887 [details] [review]
A patch to fix some of the problems.

make check fails here on Mageia Linux v6 x86-64. I'll soon attach a patch I used to fix some of them and the output of make check.
Comment 1 Shlomi Fish 2017-04-15 12:17:24 UTC
Created attachment 349888 [details]
Output of "make check" on my system.
Comment 2 Øyvind Kolås (pippin) 2017-05-04 16:07:51 UTC
The test is useful also without the added text which relies on freetype as well as installed fonts for making the same output. The change to gegl-imgcmp might still be desirable though.

commit e1049601b997c76c5e75b5eb3a07c6402abc6f82
Author: Øyvind Kolås <pippin@gimp.org>
Date:   Thu May 4 18:04:08 2017 +0200

    tests: remove text from gegl test
    
    This should fix bug #781340
Comment 3 Shlomi Fish 2017-05-04 16:51:09 UTC
(In reply to Øyvind Kolås from comment #2)
> The test is useful also without the added text which relies on freetype as
> well as installed fonts for making the same output. The change to
> gegl-imgcmp might still be desirable though.
> 
> commit e1049601b997c76c5e75b5eb3a07c6402abc6f82
> Author: Øyvind Kolås <pippin@gimp.org>
> Date:   Thu May 4 18:04:08 2017 +0200
> 
>     tests: remove text from gegl test
>     
>     This should fix bug #781340

where can I find this commit? It's not here - https://git.gnome.org/browse/GEGL/commit/?id=e1049601b997c76c5e75b5eb3a07c6402abc6f82 . Thanks!
Comment 4 Øyvind Kolås (pippin) 2017-05-04 16:58:34 UTC
It is there now - I posted the comment without knowing that the git push had completed successfully.
Comment 5 Shlomi Fish 2017-05-04 17:21:44 UTC
(In reply to Øyvind Kolås from comment #4)
> It is there now - I posted the comment without knowing that the git push had
> completed successfully.

Thanks!

Unfortunately, tests are still failing. I'll attach the new diff and make check output soon.
Comment 6 Shlomi Fish 2017-05-04 17:22:32 UTC
Created attachment 351066 [details] [review]
New patch against the working copy
Comment 7 Shlomi Fish 2017-05-04 17:23:37 UTC
Created attachment 351067 [details]
make check output on mageia v6 x86-64.
Comment 8 Øyvind Kolås (pippin) 2017-05-04 18:18:01 UTC
Comment on attachment 351066 [details] [review]
New patch against the working copy

The part referring to fonts and special casing gegl.xml should not be needed any more - there is not font-rendering going on in the output of this test https://git.gnome.org/browse/gegl/plain/tests/compositions/reference/gegl.png anymore.
Comment 9 Shlomi Fish 2017-05-07 16:24:41 UTC
Created attachment 351311 [details] [review]
Extra-new patch against the working copy - same failure

This is an extra-new patch against the working copy with Oyvind's comments incorporated. I am still getting the same failure.
Comment 10 Øyvind Kolås (pippin) 2017-05-07 17:08:57 UTC
It is a different failure from the one reported in this bug though, failing for the color-reduction.xml test not gegl.xml.

I am starting to wonder if the default make check/distcheck of GEGL should be more relaxed about passing tests comparing pixel by pixel expected output, or maybe add an ability to configure the build to skip compositions tests. The even more hairline triggering tests in docs are not breaking the build anymore, even if they still are reporting.
Comment 11 Shlomi Fish 2017-05-07 17:52:02 UTC
(In reply to Øyvind Kolås from comment #10)
> It is a different failure from the one reported in this bug though, failing
> for the color-reduction.xml test not gegl.xml.
> 

Thanks for the analysis.

> I am starting to wonder if the default make check/distcheck of GEGL should
> be more relaxed about passing tests comparing pixel by pixel expected
> output, or maybe add an ability to configure the build to skip compositions
> tests. The even more hairline triggering tests in docs are not breaking the
> build anymore, even if they still are reporting.
Comment 12 Shlomi Fish 2017-06-27 12:39:15 UTC
The color-reduction.xml tests is still failing on mageia v6. What should we do?
Comment 13 Øyvind Kolås (pippin) 2017-06-27 12:43:23 UTC
I'd suggest closing this bug and opening a bug about the "color-reduction test failing". Since the title of this bug is misleading - making it hard to find, and the issue of the original report and first comments are about a different issue.
Comment 14 Øyvind Kolås (pippin) 2017-06-27 13:05:10 UTC
Original issue is resolved, closing this and awaiting a new bug report with cleaner comment history.