Merge lp:~scarneiro/ubuntu/oneiric/setpwc/fix-for-755945 into lp:ubuntu/oneiric/setpwc

Proposed by Sebastian Carneiro
Status: Merged
Merged at revision: 6
Proposed branch: lp:~scarneiro/ubuntu/oneiric/setpwc/fix-for-755945
Merge into: lp:ubuntu/oneiric/setpwc
Diff against target: 79 lines (+54/-2)
3 files modified
debian/changelog (+12/-0)
debian/control (+3/-2)
debian/patches/20-ftbfs-old-video4linux (+39/-0)
To merge this branch: bzr merge lp:~scarneiro/ubuntu/oneiric/setpwc/fix-for-755945
Reviewer Review Type Date Requested Status
Luke Yelavich (community) Approve
Ubuntu branches Pending
Review via email: mp+69743@code.launchpad.net

Description of the change

Replaced old Video4Linux kernel interfaces with the ones from libv4l1 in the sources, Makefile, and Build-Depends in control file to fix FTBFS.

Thanks.

To post a comment you must log in.
Revision history for this message
Luke Yelavich (themuso) wrote :

Thanks for your work. i see you linked to the Debian bug report in the patch. If you could push the patch to that bug report in Debian, that would be much appreciated.

Thanks again.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2009-06-11 10:58:09 +0000
3+++ debian/changelog 2011-07-29 04:11:30 +0000
4@@ -1,3 +1,15 @@
5+setpwc (1.2-3ubuntu1) oneiric; urgency=low
6+
7+ * debian/patches/20-ftbfs-old-video4linux: Fixed FTBFS regarding
8+ use of deprecated interfaces to Video4Linux (LP: #755945)
9+ - Makefile: added v4l1 libraries to linker parameters and changed
10+ order of parameters to build with ld --as-needed.
11+ - setpwc.c: replaced old kernel includes of Video4Linux with
12+ the current ones from v4l1 libraries.
13+ * debian/control: Added libv4l1-dev to Build-Depends.
14+
15+ -- Sebastian Carneiro <scarneiro@fibertel.com.ar> Fri, 29 Jul 2011 00:12:00 -0300
16+
17 setpwc (1.2-3) unstable; urgency=low
18
19 * Bumped standards version to 3.8.1 (no changes).
20
21=== modified file 'debian/control'
22--- debian/control 2009-06-11 10:58:09 +0000
23+++ debian/control 2011-07-29 04:11:30 +0000
24@@ -1,8 +1,9 @@
25 Source: setpwc
26 Section: graphics
27 Priority: optional
28-Maintainer: Anders Lennartsson <anders@lennartsson.se>
29-Build-Depends: debhelper (>= 4.0.0), dbs (>= 0.21)
30+Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
31+XSBC-Original-Maintainer: Anders Lennartsson <anders@lennartsson.se>
32+Build-Depends: debhelper (>= 4.0.0), dbs (>= 0.21), libv4l-dev
33 Standards-Version: 3.8.1
34
35 Package: setpwc
36
37=== added file 'debian/patches/20-ftbfs-old-video4linux'
38--- debian/patches/20-ftbfs-old-video4linux 1970-01-01 00:00:00 +0000
39+++ debian/patches/20-ftbfs-old-video4linux 2011-07-29 04:11:30 +0000
40@@ -0,0 +1,39 @@
41+Description: Replaced old Video4Linux kernel interfaces with
42+ the current ones from v4l1 library to fix FTBFS.
43+Origin/Author: Sebastian Carneiro <scarneiro@fibertel.com.ar>
44+Bug: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=621961
45+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/setpwc/+bug/755945
46+Last-Update: 2011-07-28
47+diff -ruN setpwc-1.2-old/Makefile setpwc-1.2/Makefile
48+--- setpwc-1.2-old/Makefile 2011-07-29 00:13:27.895300001 -0300
49++++ setpwc-1.2/Makefile 2011-07-29 00:13:29.645300001 -0300
50+@@ -14,14 +14,14 @@
51+
52+ VERSION=1.2
53+ CFLAGS+=-Wall -O2 -DVERSION=\"$(VERSION)\"
54+-LDFLAGS=
55++LDFLAGS=-lv4l1
56+
57+ OBJS=setpwc.o
58+
59+ all: setpwc
60+
61+ setpwc: $(OBJS)
62+- $(CC) -Wall -W $(OBJS) $(LDFLAGS) -o setpwc
63++ $(CC) -Wall -W $(OBJS) -o setpwc $(LDFLAGS)
64+ #
65+ # Oh, blatant plug: http://keetweej.vanheusden.com/wishlist.html
66+
67+diff -ruN setpwc-1.2-old/setpwc.c setpwc-1.2/setpwc.c
68+--- setpwc-1.2-old/setpwc.c 2006-11-27 06:14:03.000000000 -0300
69++++ setpwc-1.2/setpwc.c 2011-07-29 00:13:29.665300001 -0300
70+@@ -23,8 +23,7 @@
71+ #include <fcntl.h>
72+ #include <unistd.h>
73+ #define _LINUX_TIME_H 1 /* to get things compile on kernel 2.6.x */
74+-#include <linux/videodev.h>
75+-
76++#include <libv4l1-videodev.h>
77+ #include "pwc-ioctl.h"
78+
79+ char *device = "/dev/video0";

Subscribers

People subscribed via source and target branches