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 90689 - Built documentation should be distributed.
Built documentation should be distributed.
Status: RESOLVED FIXED
Product: gtkmm
Classification: Bindings
Component: reference documentation
2.0
Other All
: Normal normal
: ---
Assigned To: gtkmm-forge
gtkmm-forge
Depends on:
Blocks: 97562
 
 
Reported: 2002-08-14 01:06 UTC by btb
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gtkmm_dist_docs.patch (83.33 KB, patch)
2002-11-06 10:29 UTC, Murray Cumming
none Details | Review
gtkmm_dist_docs2.patch - revised for cvs changes (82.79 KB, patch)
2002-11-06 15:21 UTC, Murray Cumming
none Details | Review

Description btb 2002-08-14 01:06:43 UTC
the 'dot' program used by doxygen is non-free, and therefore is not used
when building debs of gtkmm.  Doxygen seems to be perfectly capable of
building the relevant diagrams without it, though, providing you set the
HAVE_DOT variable correctly.  This will do so.

Index: configure.in
===================================================================
RCS file: /cvs/gnome/gtkmm-root/base/configure.in,v
retrieving revision 1.103
diff -u -r1.103 configure.in
--- configure.in	10 Aug 2002 17:39:37 -0000	1.103
+++ configure.in	14 Aug 2002 01:00:44 -0000
@@ -179,6 +179,9 @@
 # This will be much slower.
 GTKMM_ARG_ENABLE_FULLDOCS()
 
+# Check for the dot program
+AC_CHECK_PROG(HAVE_DOT,[dot],[YES],[NO])
+
 
 # Dummy conditional just to make automake-1.4 happy.
 # We need an always-false condition in docs/Makefile.am.
Index: docs/reference/Doxyfile.in
===================================================================
RCS file: /cvs/gnome/gtkmm-root/base/docs/reference/Doxyfile.in,v
retrieving revision 1.11
diff -u -r1.11 Doxyfile.in
--- docs/reference/Doxyfile.in	23 Apr 2002 15:39:05 -0000	1.11
+++ docs/reference/Doxyfile.in	14 Aug 2002 01:00:44 -0000
@@ -155,7 +155,7 @@
 # Configuration options related to the dot tool   
 #---------------------------------------------------------------------------
 CLASS_DIAGRAMS         = YES
-HAVE_DOT               = YES
+HAVE_DOT               = @HAVE_DOT@
 CLASS_GRAPH            = YES
 COLLABORATION_GRAPH    = NO
 TEMPLATE_RELATIONS     = NO
Comment 1 Murray Cumming 2002-08-14 07:51:19 UTC
Dot builds _better_ diagrams. I don't want 2 versions of the same
documentation.

Do you expect to distribute the documentation in a deb? In a separate
deb? Maybe we should just generate it as part of the whole build.
Comment 2 btb 2002-08-14 18:22:36 UTC
Yes, I am distributing it, in a separate deb for gtkmm (only because
it's so large), and in the -dev package for the other modules.  I
build it with a simple 'make -C docs/reference' (and patching Doxyfile.in)

I noticed that with non-dot, the diagrams are slightly less complete, but
I think distributing reference docs is important, and would not want
to put them into "non-free" or "contrib".

Comment 3 Murray Cumming 2002-08-15 06:57:54 UTC
OK, so let's build the documentation when we build the .h/,cc files. I
think that means building it when we are in maintainer mode.

I don't want to distribute 2 versions of the same documentation, and I
don't want to distribute crippled documentation.
Comment 4 Murray Cumming 2002-08-29 11:16:14 UTC
I'm kind of hoping that someone provides a patch to build the 
documentation when the .h/.cc files are built, and to add them to 
make dist.
Comment 5 Murray Cumming 2002-10-11 10:34:07 UTC
They say that it's open source:
http://www.research.att.com/sw/tools/graphviz/download.html

If their license isn't really OpenSource(TM) then maybe they should be
persuaded to use a different license, or to get it certified.
Comment 6 btb 2002-10-11 19:04:53 UTC
I will see if the Stephen (maintainer of graphviz debs) has
investigated that.  In the meantime, springgraph looks like an
excellent drop-in replacement for dot:
http://www.chaosreigns.com/code/springgraph/

Here is what the ref. docs look like when springgraph is used:
http://pokey.css.washington.edu/debian/gtkmm/gtkmm1.3/gtkmm1.3-1.3.24/docs/reference/html/

-brad
Comment 7 btb 2002-10-11 19:33:02 UTC
I seem to have been mistaken about springgraph.  I can't get it to
work at all, now.
Comment 8 Murray Cumming 2002-10-12 13:37:24 UTC
Look at the ListStore page, for instance. Only dot properly combines
virtual base classes.
Comment 9 Murray Cumming 2002-10-21 12:53:32 UTC
Did you get any response from the graphviz maintainer?
Comment 10 Murray Cumming 2002-10-26 02:37:28 UTC
Please respond.
Comment 11 btb 2002-10-28 19:31:12 UTC
Here's what he said:

Bradley Bell wrote:
> 
> Hi, are there any plans to move graphviz into main?  It would be
useful to
> build the reference documentation that goes into the gtkmm debs.

	I need to have our legal group review the license and get
	a more definitive answer to AT&T.  At this time we consider
	this to be non-free due to the licensing issues.

Regards,
Stephen
Comment 12 Murray Cumming 2002-10-29 08:28:44 UTC
OK, so it looks like it's still best to distribute the built
documentation. And we need a patch for that.
Comment 13 btb 2002-10-29 21:45:27 UTC
I'll see what I can do.
Comment 14 Murray Cumming 2002-11-06 10:28:11 UTC
Here is a patch to do this, but:
1) I am concerned that it more than doubles the size of the tarball -
to almost 5Mb.
2) The reference docs are not rebuilt unless you do a manual "make
clean all" in docs/reference. I'd like the reference docs to rebuild
always when in "maintainer mode".
Comment 15 Murray Cumming 2002-11-06 10:29:34 UTC
Created attachment 12087 [details] [review]
gtkmm_dist_docs.patch
Comment 16 Murray Cumming 2002-11-06 15:21:30 UTC
Created attachment 12097 [details] [review]
gtkmm_dist_docs2.patch - revised for cvs changes
Comment 17 Murray Cumming 2002-11-10 02:11:48 UTC
Then again, GTK+ does the same thing and is 8Mb. Applied.