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 771510 - seed: fix make distcheck
seed: fix make distcheck
Status: RESOLVED FIXED
Product: seed
Classification: Bindings
Component: libseed
git master
Other All
: Normal normal
: ---
Assigned To: seed-maint
seed-maint
Depends on:
Blocks:
 
 
Reported: 2016-09-16 05:00 UTC by Mathieu Duponchelle
Modified: 2016-09-16 12:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
make check: mark known-to-fail tests with XFAIL_TESTS. (2.10 KB, patch)
2016-09-16 05:01 UTC, Mathieu Duponchelle
committed Details | Review
jsextensions: sanitize and make make distcheck pass (2.70 KB, patch)
2016-09-16 05:01 UTC, Mathieu Duponchelle
committed Details | Review
doc: fix make distcheck (2.07 KB, patch)
2016-09-16 05:01 UTC, Mathieu Duponchelle
committed Details | Review
Fix tests when builddir != srcdir (4.47 KB, patch)
2016-09-16 05:01 UTC, Mathieu Duponchelle
none Details | Review
Delete old .gitignore (31.97 KB, patch)
2016-09-16 05:01 UTC, Mathieu Duponchelle
committed Details | Review
Uninstall documentation correctly (1.84 KB, patch)
2016-09-16 05:01 UTC, Mathieu Duponchelle
committed Details | Review
doc/modules: gtk-doc must be run in the html subfolder. (1.67 KB, patch)
2016-09-16 05:01 UTC, Mathieu Duponchelle
committed Details | Review
Fix tests when builddir != srcdir (4.43 KB, patch)
2016-09-16 05:25 UTC, Mathieu Duponchelle
committed Details | Review

Description Mathieu Duponchelle 2016-09-16 05:00:59 UTC
This library is currently in a pretty pitiful state, I'm only doing
basic sanitization here.

Two things I have not addressed, because they will certainly imply
way more work:

Actually fixing the tests; running make check failed in the first of
the test subdirectories, once I marked a few tests as known to fail
other tests appeared, and unsurprisingly did not pass, the final result
is 11 tests failing out of 60, at least we're now aware of these failures :)

seed currently requires being installed both for the tests to fail as
expected and for make fistcheck to pass (these two issues are linked).
This should be fixed at some point, but my priority was to have make distcheck
pass, which is the case with this set of patches.
Comment 1 Mathieu Duponchelle 2016-09-16 05:01:02 UTC
Created attachment 335667 [details] [review]
make check: mark known-to-fail tests with XFAIL_TESTS.

Without this, make check stopped recursing up at the first
subdirectory failure, in our case in tests/javascript/gtypes,
leading to:

* The incorrect impression that the test suite consisted of
  only 12 tests.

* Making it impossible for make distcheck to pass.

* First-time users wasting time figuring out what in their
  environment was responsible for these failures, as one
  usually expects make check to pass.

See
https://www.gnu.org/software/automake/manual/html_node/Scripts_002dbased-Testsuites.html
Comment 2 Mathieu Duponchelle 2016-09-16 05:01:07 UTC
Created attachment 335668 [details] [review]
jsextensions: sanitize and make make distcheck pass

.. in that directory, there are other errors in other places.
Comment 3 Mathieu Duponchelle 2016-09-16 05:01:12 UTC
Created attachment 335669 [details] [review]
doc: fix make distcheck
Comment 4 Mathieu Duponchelle 2016-09-16 05:01:16 UTC
Created attachment 335670 [details] [review]
Fix tests when builddir != srcdir
Comment 5 Mathieu Duponchelle 2016-09-16 05:01:21 UTC
Created attachment 335671 [details] [review]
Delete old .gitignore

And use git.mk instead
Comment 6 Mathieu Duponchelle 2016-09-16 05:01:26 UTC
Created attachment 335672 [details] [review]
Uninstall documentation correctly
Comment 7 Mathieu Duponchelle 2016-09-16 05:01:30 UTC
Created attachment 335673 [details] [review]
doc/modules: gtk-doc must be run in the html subfolder.

gtkdoc-mkhtml will output html.stamp in the parent folder, and
it never got cleaned afterwards.

Not sure why doc/modules does not use the gtk-doc makefile
fragment, but I'm not going for the best solution, simply the
one that does not break distcheck by leaving build artefacts in
random places.
Comment 8 Mathieu Duponchelle 2016-09-16 05:25:12 UTC
Created attachment 335674 [details] [review]
Fix tests when builddir != srcdir
Comment 9 Mathieu Duponchelle 2016-09-16 12:57:03 UTC
Attachment 335667 [details] pushed as 6e115d2 - make check: mark known-to-fail tests with XFAIL_TESTS.
Attachment 335668 [details] pushed as 7105800 - jsextensions: sanitize and make make distcheck pass
Attachment 335669 [details] pushed as 7daa813 - doc: fix make distcheck
Attachment 335671 [details] pushed as 6651630 - Delete old .gitignore
Attachment 335672 [details] pushed as ec7aaf7 - Uninstall documentation correctly
Attachment 335673 [details] pushed as 7a70f39 - doc/modules: gtk-doc must be run in the html subfolder.
Attachment 335674 [details] pushed as 369fda3 - Fix tests when builddir != srcdir