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 767629 - Remove geocode-enum-types.xml to fix out-of-source gtk-doc build
Remove geocode-enum-types.xml to fix out-of-source gtk-doc build
Status: RESOLVED FIXED
Product: geocode-glib
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: geocode-glib maintainer(s)
geocode-glib maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2016-06-14 03:06 UTC by Ting-Wei Lan
Modified: 2016-09-27 17:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build: Fix gtk-doc build when builddir != srcdir (859 bytes, patch)
2016-06-14 03:07 UTC, Ting-Wei Lan
committed Details | Review

Description Ting-Wei Lan 2016-06-14 03:06:09 UTC
docs/geocode-glib-docs.xml lists geocode-enum-types.xml, but geocode-enum-types.xml is not generated when doing out-of-source build. As it doesn't provide useful information to users, I think it is better to remove it than fixing the build.
Comment 1 Ting-Wei Lan 2016-06-14 03:07:08 UTC
Created attachment 329749 [details] [review]
build: Fix gtk-doc build when builddir != srcdir

The generated geocode-enum-types.xml is not useful and can be removed.
Comment 2 Ting-Wei Lan 2016-07-16 15:44:20 UTC
Does someone have time to review this trivial patch? It has stayed in bugzilla for 1 month.
Comment 3 Jonas Danielsson 2016-07-16 20:16:52 UTC
Review of attachment 329749 [details] [review]:

Sorry for delay! And thanks for patch!
Comment 4 Ting-Wei Lan 2016-07-17 02:15:12 UTC
Attachment 329749 [details] pushed as a646012 - build: Fix gtk-doc build when builddir != srcdir
Comment 5 Jonas Danielsson 2016-09-20 09:40:15 UTC
This seem to have broken distcheck:

$ cat ./geocode-glib-3.22.0/_build/sub/docs/gtkdoc-check.log
Running suite(s): gtk-doc-geocode-glib
/home/jonas/jhbuild/checkout/geocode-glib/geocode-glib-3.22.0/_build/sub/docs/geocode-glib-docs.xml doesn't appear to include "xml/geocode-enum-types.xml"
75.0%: Checks 4, Failures: 1
FAIL gtkdoc-check.test (exit status: 1)
Comment 6 Ting-Wei Lan 2016-09-27 17:52:05 UTC
I don't know how to reproduce the error message with 'gmake distcheck'. After disabling several failing tests in test-gcglib.c, I saw failure when running gtkdoc-check.test but with different message:

$ cat geocode-glib-3.20.1/_build/sub/docs/gtkdoc-check.log
Running suite(s): gtk-doc-geocode-glib
geocode-glib-3.20.1/_build/sub/docs/geocode-glib-unused.txt:1:E: 10 unused documentation entries
75.0%: Checks 4, Failures: 1
FAIL gtkdoc-check.test (exit status: 1)