Bug#751702: libtheora: Drop the unused graphviz build dependency.

[Peter Pentchev]

Source: libtheora
Severity: wishlist
Tags: patch

Hi,

First of all, thanks for taking care of libtheora in Debian!

What do you think about the attached trivial patch that removes the
build dependency on graphviz?  As far as I could see, the libtheora
build does not really check for or use any of the graphviz tools at all;
it seems that the packages built with and without the graphviz
dependency are identical.

This patch was extracted from my Git repository at
https://gitorious.org/roam-debian-bootstrap/libtheora-debian/commits/roam-drop-graphviz

I originally started working on this as part of the "Bootstrappable
Debian" Google Summer of Code project, as some of our tools seemed to
indicate the presence of a circular build dependency that could be
removed if src:libtheora stopped build-depending on graphviz, but
right now this is not part of the Feedback Arc Set.  Still, dropping
an unused build dependency looks like a welcome simplification to me :)

Thanks again for your work on libtheora and Debian in general!

G'luck,
Peter

-- System Information:
Debian Release: jessie/sid
  APT prefers testing
  APT policy: (990, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.14-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=bg_BG.UTF-8, LC_CTYPE=bg_BG.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-- 
Peter Pentchev  roam at ringlet.net roam at FreeBSD.org p.penchev at storpool.com
PGP key:        http://people.FreeBSD.org/~roam/roam.key.asc
Key fingerprint 2EE7 A7A5 17FC 124C F115  C354 651E EFB0 2527 DF13
-------------- next part --------------
From ca208c6a8c3399fa7ddc7748d198e99197c3584f Mon Sep 17 00:00:00 2001
From: Peter Pentchev <roam at ringlet.net>
Date: Fri, 13 Jun 2014 15:25:36 +0300
Subject: [PATCH] Remove the unused graphviz build dependency.

---
 debian/control | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/debian/control b/debian/control
index a6771b9..1830818 100644
--- a/debian/control
+++ b/debian/control
@@ -4,7 +4,7 @@ Priority: optional
 Maintainer: Debian Xiph.org Maintainers <pkg-xiph-maint at lists.alioth.debian.org>
 Uploaders: John Francesco Ferlito <johnf at inodes.org>
 Build-Depends: debhelper (>> 9), dh-autoreconf, devscripts,
- doxygen, graphviz, libogg-dev, libsdl1.2-dev, libvorbis-dev, python,
+ doxygen, libogg-dev, libsdl1.2-dev, libvorbis-dev, python,
  libtool, automake, quilt, dpkg (>= 1.14.8), libpng-dev, pkg-config
 Build-Depends-Indep: transfig, texlive-latex-base, texlive-latex-extra,
  texlive-latex-recommended, texlive-fonts-recommended, ghostscript
-- 
2.0.0

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-xiph-maint/attachments/20140615/d6593114/attachment.sig>