Bug#752093: [x264] Drop the no longer used B-D libx11-dev

[Peter Pentchev]

Source: x264
Version: 2:0.142.2412+gitd7e6896-1
Severity: wishlist
Tags: patch

Hi,

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

What do you think about the attached trivial patches that drop the build
dependency on libx11-dev?  Since version 0.140.2377 (Git rev. 95d196e)
x264 has no visualization support, and thus it neither checks for nor
uses the X11 library.

If something goes wrong with the patch, it is also available at
https://gitorious.org/roam-debian-bootstrap/x264-debian/commits/roam-drop-x11

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 98cbaed5d5364ed7d94cfd00a34b69f441b7b378 Mon Sep 17 00:00:00 2001
From: Peter Pentchev <roam at ringlet.net>
Date: Thu, 19 Jun 2014 17:17:55 +0300
Subject: [PATCH 1/2] Drop the unneeded build dependecy on libx11-dev.

The visualization option was dropped in x264 0.140.2377 95d196e.
Ever since then, the X11 library is not checked for or used at all.
---
 debian/control.in | 1 -
 1 file changed, 1 deletion(-)

diff --git a/debian/control.in b/debian/control.in
index 7e5f0d5..4e8fb42 100644
--- a/debian/control.in
+++ b/debian/control.in
@@ -12,7 +12,6 @@ Build-Depends:
  libavformat-dev (>= 6:9),
  libffms2-dev,
  libgpac-dev (>= 0.5.0+svn4288~),
- libx11-dev,
  yasm [any-i386 any-amd64]
 Standards-Version: 3.9.5
 Vcs-Git: git://anonscm.debian.org/pkg-multimedia/x264.git
-- 
2.0.0

-------------- next part --------------
From 42baf129ec379e49e9f55d511ddab87683bbc1c2 Mon Sep 17 00:00:00 2001
From: Peter Pentchev <roam at ringlet.net>
Date: Thu, 19 Jun 2014 17:21:50 +0300
Subject: [PATCH 2/2] Regenerate debian/control.

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

diff --git a/debian/control b/debian/control
index 01ee2e0..66ac8c9 100644
--- a/debian/control
+++ b/debian/control
@@ -12,7 +12,6 @@ Build-Depends:
  libavformat-dev (>= 6:9),
  libffms2-dev,
  libgpac-dev (>= 0.5.0+svn4288~),
- libx11-dev,
  yasm [any-i386 any-amd64]
 Standards-Version: 3.9.5
 Vcs-Git: git://anonscm.debian.org/pkg-multimedia/x264.git
-- 
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-multimedia-maintainers/attachments/20140619/2f1d4d22/attachment.sig>