Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2017-03-19 Thread Ivo De Decker
Control: severity -1 important Hi, On Wed, Mar 23, 2016 at 04:50:34PM +0100, Matthias Klose wrote: > On 22.03.2016 17:12, Santiago Vila wrote: > >Dear Matthias: > > > >If the default HAVE_DOT is changed to "no", it is relatively easy to > >check which packages would need to add "graphviz" to

Processed: Re: Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2017-03-19 Thread Debian Bug Tracking System
Processing control commands: > severity -1 important Bug #818787 [doxygen] doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line. Severity set to 'important' from 'serious' -- 818787: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=818787 Debian Bug Tracking

Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2016-09-13 Thread Anders Kaseorg
Hi, I’m one of the nobodies that looks at my build logs. I noticed five instances of sh: 1: dot: not found error: Problems running dot: exit code=127, command='dot', arguments='"…/graph_legend.dot" -Tpng -o "…/graph_legend.png"' in the openafs build log, and indeed Doxygen is generating

Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2016-03-23 Thread Matthias Klose
On 22.03.2016 17:12, Santiago Vila wrote: Dear Matthias: If the default HAVE_DOT is changed to "no", it is relatively easy to check which packages would need to add "graphviz" to their build-depends. (See the thread in -devel about this). In other words: If you think that's the right fix (I

Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2016-03-22 Thread Santiago Vila
Dear Matthias: If the default HAVE_DOT is changed to "no", it is relatively easy to check which packages would need to add "graphviz" to their build-depends. (See the thread in -devel about this). In other words: If you think that's the right fix (I do), don't worry about the additional bugs

Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2016-03-20 Thread Santiago Vila
I believe the attached patch is closer to the "right" fix. Presumably, packages actually requiring the dot command will have an override for this variable in their Doxyfiles. Thanks.--- a/src/config.xml +++ b/src/config.xml @@ -3104,7 +3104,7 @@ where `loc1` and `loc2` can be relative or

Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2016-03-20 Thread Santiago Vila
Note 2: The easy fix does not necessariñy have to be the right fix. As Bas Wijnen points out in -devel: > If the program can be used without it, it should not be a Depends. That's > what > Recommends are for.

Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2016-03-20 Thread Santiago Vila
Note: The changelog says (year 2000) graphviz is in the Suggests because it was non-free, which clearly is not the case anymore. So, the easy fix for this bug is to move graphviz to the Depends line (see the attached patch). (Well, easy, but even after this, we would still have to rebuild all

Bug#818787: doxygen: Changes default HAVE_DOT to YES without having graphviz in the Depends line.

2016-03-20 Thread Santiago Vila
Package: doxygen Version: 1.8.11-1 Severity: serious Hello again Matthias. If packages having a "Build-Depends: doxygen" may assume that graphviz is installed, then doxygen should have a line like this: Depends: graphviz Currently, this is only a Suggests. If, on the other side, packages