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 141899 - Use gtk-doc.make
Use gtk-doc.make
Status: VERIFIED INCOMPLETE
Product: libgsf
Classification: Core
Component: Documentation
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Jody Goldberg
Jody Goldberg
Depends on:
Blocks:
 
 
Reported: 2004-05-05 11:58 UTC by James Cape
Modified: 2009-08-15 18:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to use gtk-doc.make (4.11 KB, patch)
2004-05-05 11:59 UTC, James Cape
needs-work Details | Review
Updated patch for a nicer docs build all-around. (16.90 KB, patch)
2004-05-05 18:12 UTC, James Cape
committed Details | Review
Patch to fix type macros and <protected> (2.64 KB, patch)
2004-05-07 13:46 UTC, James Cape
accepted-commit_now Details | Review

Description James Cape 2004-05-05 11:58:41 UTC
Currently, when compiled without --with-html-dir, the docs are installed to
$(prefix)/gtk-doc/html, as opposed to $(prefix)/gtk-doc/html/gsf, the "standard"
location. This can be easily fixed (and give devhelp support for free) by using
the gtk-doc.make file which is pulled in by gnome-autogen.sh.
Comment 1 James Cape 2004-05-05 11:59:42 UTC
Created attachment 27388 [details] [review]
Patch to use gtk-doc.make
Comment 2 Jody Goldberg 2004-05-05 16:36:17 UTC
The core of this problem has already been fixed in cvs (thanks JHM and fcrozat)

I tried to move to gtk-doc.make a few weeks back which is why it is included in
the build.  That was why we moved to automake-1.7.  Unfortunately it had two
major flaws

1) distcheck failed
2) the docs rebuilt every time even if nothing had changed

If you can fix these problems I'd be happy to apply the result, but frankly
auto* is too much of a time sink for me to bother with it right now.
Comment 3 James Cape 2004-05-05 18:12:29 UTC
Created attachment 27400 [details] [review]
Updated patch for a nicer docs build all-around.

This patch does spew a lot of warnings from the gsf-scan.c compilation, but it
that should be fixed in gtk-doc.
Comment 4 Jody Goldberg 2004-05-07 03:34:41 UTC
Very nice.   This worked mostly smoothly I tweaked the remaining warnings.

There is still a complaint about the
         'No template matches protected in para'

which suggests I'm missing something minor somewhere.

Additionally any thoughts on the
     16% symbol docs coverage (56 symbols documented, 305 not documented)

It seems to be including things like the standard type macros.  Is there a way
to disable that ?

Comment 5 Jody Goldberg 2004-05-07 03:35:14 UTC
Comment on attachment 27400 [details] [review]
Updated patch for a nicer docs build all-around.

applied
Comment 6 James Cape 2004-05-07 13:45:18 UTC
The "protected" element doesn't exist in the Docbook DTD AFAICT. I think "note"
may be a suitable replacement, though.

So far as the coverage goes, I only saw a 6 type macros which are being included
(from GsfInput and GsfOutput), because they are in a <SUBSECTION
StandardImplementation> rather than <SUBSECTION Standard> (which is hidden
pending some hackage to gtk-doc to handle that automatically). I'm not sure
about the other macros which are showing up.

I'm attaching another patch to fix these issues.
Comment 7 James Cape 2004-05-07 13:46:28 UTC
Created attachment 27474 [details] [review]
Patch to fix type macros and <protected>
Comment 8 Jody Goldberg 2004-05-07 14:52:51 UTC
Comment on attachment 27474 [details] [review]
Patch to fix type macros and <protected>

Looks good please commit
Comment 9 Jody Goldberg 2004-05-07 14:55:18 UTC
I suspect the worst of the coverage issues stem from the plethora of #define
offsets for msole and zip files.  I don't see alot of benefit in documenting
them.  People who need them know their names.  Is there a way to add a block
comment to them to indicate 'these all format specific offsets' ?
Comment 10 James Cape 2004-05-09 18:17:55 UTC
There's no way I'm aware of to block-comment these, though a script could
probably be used to auto-generate them. They could also be placed in the
Standard or Private SUBSECTION to ignore them for now, but they won't show up in
the docs then...

AFAIK, what's needed is some scriptable "template for SUBSECTION" support in
gtk-doc (which could also handle the type macros). Until then, it's got to be
done by the module (either manually or by some script). :-(
Comment 11 Jody Goldberg 2004-05-11 16:07:57 UTC
Lets stick a fork in this one.
Thanks for all your help.  This is much better.