FTBFS when tcl8.4 is installed

Bug #414057 reported by Martin Olsson
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
graphviz (Ubuntu)
Fix Released
Undecided
Unassigned
Nominated for Karmic by Martin Olsson

Bug Description

When tcl8.4 is installed graphviz FTBFS because it tries to use tcl8.4 instead of tcl8.5.
This is because tclsh points to tclsh8.4. The fix is to use explicitly tclsh8.5.

Revision history for this message
Martin Olsson (mnemo) wrote :

The FTBFS happens because "/usr/share/man/man3/gv_tcl.3" is missing. There is this package called "libgv-tcl" which contains "/usr/share/man/man3/gv_tcl.3.gz" which is almost the same filename.

It sounds to me that A) libgv-tcl should be added as a build-dep for libgraphviz4, and B) the filename reference should be changed to account for the fact that this man page is nowadays gzipped?

Changed in graphviz (Ubuntu):
status: New → In Progress
assignee: nobody → Simon Olofsson (simono)
Revision history for this message
No (11111nonononono-deactivatedaccount-deactivatedaccount-deactivatedaccount) wrote :

Martin, the linked branch contains a fix. If you like to, you could try building from it. I'll upload a fixed package to my PPA later today.

Changed in graphviz (Ubuntu):
status: In Progress → Incomplete
summary: - FTBFS: libgraphviz4 fails to compile in karmic due to missing
- debian/tmp/usr/share/man/man3/gv_tcl.3
+ FTBFS when tcl8.4 is installed
Revision history for this message
Martin Olsson (mnemo) wrote :

Hmm, I did "bzr branch lp:~simono/ubuntu/karmic/graphviz/fixes-bug-414057" and then "debuild -b -us -uc" and then I got this error:

/bin/bash ../../libtool --tag=CC --mode=compile x86_64-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I../.. -I../.. -I../../lib/gvc -I../../lib/pack -I../../lib/fdpgen -I../../lib/pathplan -I../../lib/graph -I../../lib/cdt -I/usr/include -g -O2 -Wno-unknown-pragmas -Wstrict-prototypes -Wpointer-arith -Wall -ffast-math -MT globals.lo -MD -MP -MF .deps/globals.Tpo -c -o globals.lo globals.c
 x86_64-linux-gnu-gcc -DHAVE_CONFIG_H -I. -I../.. -I../.. -I../../lib/gvc -I../../lib/pack -I../../lib/fdpgen -I../../lib/pathplan -I../../lib/graph -I../../lib/cdt -I/usr/include -g -O2 -Wno-unknown-pragmas -Wstrict-prototypes -Wpointer-arith -Wall -ffast-math -MT globals.lo -MD -MP -MF .deps/globals.Tpo -c globals.c -fPIC -DPIC -o .libs/globals.o
mv -f .deps/globals.Tpo .deps/globals.Plo
bison -y -dv ../../lib/common/htmlparse.y
../../lib/common/htmlparse.y:601.11-12: $$ for the midrule at $3 of `table' has no declared type
make[4]: *** [y.output] Error 1
make[4]: Leaving directory `/home/mnemo/src/fixes-bug-414057/lib/common'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/home/mnemo/src/fixes-bug-414057/lib'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/mnemo/src/fixes-bug-414057'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/home/mnemo/src/fixes-bug-414057'
make: *** [build-stamp] Error 2
dpkg-buildpackage: error: debian/rules build gave error exit status 2
debuild: fatal error at line 1334:
dpkg-buildpackage -rfakeroot -D -us -uc -b failed
mnemo@kingfish:~/src/fixes-bug-414057$

Revision history for this message
Martin Olsson (mnemo) wrote :

Also I can't install "tclsh8.5" it seems?

mnemo@kingfish:~/src/fixes-bug-414057$ sudo apt-get install tclsh
Reading package lists... Done
Building dependency tree
Reading state information... Done
Package tclsh is a virtual package provided by:
  tcl8.3 8.3.5-14
  tcl8.5 8.5.7-1
  tcl8.4 8.4.19-3
You should explicitly select one to install.
E: Package tclsh has no installation candidate
mnemo@kingfish:~/src/fixes-bug-414057$ sudo apt-get install tclsh8.5
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Couldn't find package tclsh8.5
mnemo@kingfish:~/src/fixes-bug-414057$

Revision history for this message
Martin Olsson (mnemo) wrote :

Seems like I already have tcl8.5 package installed though:

mnemo@kingfish:~/src/fixes-bug-414057$ dpkg -l | g tcl
ii libgv-tcl 2.20.2-3ubuntu5 Tcl bindings for graphviz
ii tcl 8.4.16-2 The Tool Command Language (default version)
ii tcl8.4 8.4.19-3 Tcl (the Tool Command Language) v8.4 - run-t
ii tcl8.5 8.5.7-1 Tcl (the Tool Command Language) v8.5 - run-t
ii tcl8.5-dev 8.5.7-1 Tcl (the Tool Command Language) v8.5 - devel
mnemo@kingfish:~/src/fixes-bug-414057$

Revision history for this message
Martin Olsson (mnemo) wrote :

mnemo@kingfish:~/src/fixes-bug-414057$ dpkg -L tcl8.5 | g tclsh
/usr/share/man/man1/tclsh8.5.1.gz
/usr/bin/tclsh8.5

Revision history for this message
No (11111nonononono-deactivatedaccount-deactivatedaccount-deactivatedaccount) wrote :

tcl8.5 is the package which contains tclsh8.5.

To build the package do the following:
sudo apt-get build-dep graphviz
sudo aptitude install bzr-builddeb
bzr branch lp:~simono/ubuntu/karmic/graphviz/fixes-bug-414057
cd fixes-bug-414057
bzr-buildpackage -e
cd ../build-area/graphviz-2.20.2
debuild -us -uc -b

Revision history for this message
Martin Olsson (mnemo) wrote :

That worked, it built all the DEBs now. Thanks for taking some time to look into this.

Changed in graphviz (Ubuntu):
status: Incomplete → Confirmed
Changed in graphviz (Ubuntu):
assignee: Simon Olofsson (simono) → nobody
Revision history for this message
No (11111nonononono-deactivatedaccount-deactivatedaccount-deactivatedaccount) wrote :

The package is now available in my PPA:
https://launchpad.net/~simono/+archive/ppa
It built fine.

Revision history for this message
No (11111nonononono-deactivatedaccount-deactivatedaccount-deactivatedaccount) wrote :

Here is the diff between 2.20.2-3ubuntu5 and 2.20.2-3ubuntu6:
https://launchpadlibrarian.net/33157429/4bADQ5fEy1gj9JEFGhq5FLWEPzz.txt

Revision history for this message
Daniel Holbach (dholbach) wrote :

We're already at
graphviz (2.20.2-8ubuntu2) lucid; urgency=low

  * No-change rebuild for OCaml 3.11.2 transition (LP: #522363).

 -- Luke Yelavich <email address hidden> Fri, 19 Feb 2010 12:39:35 +1100

... and it builds fine. Is there anything still required?

Changed in graphviz (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
No (11111nonononono-deactivatedaccount-deactivatedaccount-deactivatedaccount) wrote :

Daniel, this was fixed in Debian in 2.20.2-4. Which has already been merged.

Changed in graphviz (Ubuntu):
status: Incomplete → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.