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 604892 - checks fail
checks fail
Status: RESOLVED FIXED
Product: gtk-doc
Classification: Platform
Component: general
1.12
Other Linux
: Normal normal
: 1.14
Assigned To: Stefan Sauer (gstreamer, gtkdoc dev)
gtk-doc maintainers
Depends on:
Blocks:
 
 
Reported: 2009-12-18 10:17 UTC by Götz Waschk
Modified: 2009-12-19 20:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
complete build log (58.94 KB, text/plain)
2009-12-18 14:05 UTC, Götz Waschk
  Details
append to the perl path so that it stays overrideable (2.39 KB, patch)
2009-12-18 19:52 UTC, Stefan Sauer (gstreamer, gtkdoc dev)
committed Details | Review

Description Götz Waschk 2009-12-18 10:17:53 UTC
This is on Mandriva cooker with gtk-doc 1.12:

100 %: Checks 8, Failures: 0
PASS: tools.sh
Running suite(s): gtk-doc-tester
Cannot open tester-undocumented.txt: No such file or directory
FAIL: gobject.sh
Running suite(s): gtk-doc-tester
Cannot open tester-undocumented.txt: No such file or directory
FAIL: bugs.sh
Running suite(s): gtk-doc-tester
Cannot open tester-undocumented.txt: No such file or directory
FAIL: annotations.sh
Running suite(s): gtk-doc-fail
grep: tester-undocumented.txt: No such file or directory
grep: tester-undocumented.txt: No such file or directory
grep: tester-undocumented.txt: No such file or directory
100 %: Checks 3, Failures: 0
PASS: fail.sh
========================================================================
3 of 5 tests failed
Please report to http://bugzilla.gnome.org/enter_bug.cgi?product=gtk-doc
========================================================================
Comment 1 Stefan Sauer (gstreamer, gtkdoc dev) 2009-12-18 13:25:21 UTC
those files should be produced by the test runs. Could you have a deeper look?
Comment 2 Stefan Sauer (gstreamer, gtkdoc dev) 2009-12-18 13:32:52 UTC
could you attach the whole make check output?
Comment 3 Götz Waschk 2009-12-18 14:05:28 UTC
Created attachment 149997 [details]
complete build log
Comment 4 Stefan Sauer (gstreamer, gtkdoc dev) 2009-12-18 15:36:05 UTC
Undefined subroutine &main::CreateValidSGMLID called at /home/goetz/svn/gtk-doc/BUILD/gtk-doc-1.13/gtkdoc-mkdb line 2603, <INPUT> line 2.

This function is in gtkdoc-common.pl. You probably have an older version of gtk-doc installed and the way you build the rpm breaks the PERLPATH setting I do in make check to use the uninstalled version. Could you try make check from the tarball?

Also i sthat is supposed to beome a official package:
gtk-doc was configured with the following options:
==================================================
configure: ** Python based tools enabled, using /usr/bin/python
configure: ** SGML support enabled, using /usr/bin/openjade
configure:    XML PDF support disabled, no fop available
configure: ** Gnome-doc-utils support enabled
configure: ** Scrollkeeper support enabled
configure: ** Syntax highlighting of examples enabled, using /bin/vim
configure: ** Building regression tests

- please install dblatex to have pdf support (I'll fix the messae in the sumary)
- please install source-highlight or highlight to get decent syntax highlighting
Comment 5 Götz Waschk 2009-12-18 18:02:08 UTC
You are right, uninstalling gtk-doc 1.11 fixes make check. The rpm build doesn't try anything special, just configure, make and make check.
Comment 6 Stefan Sauer (gstreamer, gtkdoc dev) 2009-12-18 19:29:37 UTC
Found it! Its because of this construct in most of the scripts:

unshift @INC, '/usr/share/gtk-doc/data';
require "gtkdoc-common.pl";

I'll try to find a way that they can detect that they are running uninstalled. Can you proceed with a 1.13 release. I would include that fix with a 1.14, okay?
Comment 7 Götz Waschk 2009-12-18 19:39:53 UTC
Sure, I can use 1.13 for now. And I'll enable dblatex and source-highlight.
Comment 8 Stefan Sauer (gstreamer, gtkdoc dev) 2009-12-18 19:52:43 UTC
Created attachment 150022 [details] [review]
append to the perl path so that it stays overrideable

Götz, could you try this patch? If it works I'll push it and you can add it localy to the rpm.
Comment 9 Götz Waschk 2009-12-18 20:55:00 UTC
Yes, with the patch the checks all succeed.
Comment 10 Stefan Sauer (gstreamer, gtkdoc dev) 2009-12-19 20:09:49 UTC
commit 1f5d323d9ecf6384980c8e51e9d5b30376ce3536
Author: Stefan Kost <ensonic@users.sf.net>
Date:   Fri Dec 18 21:47:23 2009 +0200

    tools: append to the perl path so that it stays overrideable. Fixes #604892
    
    We were prepending datadir to the INC path for perl to find gtkdoc-common.pl.
    This also caused the unit tests to run with the installed version, despite
    that we had set an local path to INC from the Makefile.