Bug#530436: ffmpeg-debian: FTBFS on Hurd

2009-05-25 Thread Andres Mejia
On Monday 25 May 2009 01:34:15 Diego Biurrun wrote:
 On Sun, May 24, 2009 at 08:53:08PM -0400, Andres Mejia wrote:
  On Sunday 24 May 2009 17:08:17 Marc Dequènes (Duck) wrote:
   The configure script does not support the GNU system. The attached
   very small patch adds the required modification to fix this problem.
   Please consider applying it in your next upload and help push this
   change upstream.
 
  Thanks. It's been applied in the git repo for the next upload and
  forwarded upstream.

 Better strategy: Forward upstream first, then merge back whatever
 changes upstream really implements.

 Diego

Oh yeah, got to fix that now. I'll forward these changes upstream first then 
from 
now on.

-- 
Regards,
Andres



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#530436: ffmpeg-debian: FTBFS on Hurd

2009-05-25 Thread Diego Biurrun
On Mon, May 25, 2009 at 03:26:50AM -0400, Andres Mejia wrote:
 On Monday 25 May 2009 01:34:15 Diego Biurrun wrote:
  On Sun, May 24, 2009 at 08:53:08PM -0400, Andres Mejia wrote:
   On Sunday 24 May 2009 17:08:17 Marc Dequènes (Duck) wrote:
The configure script does not support the GNU system. The attached
very small patch adds the required modification to fix this problem.
Please consider applying it in your next upload and help push this
change upstream.
  
   Thanks. It's been applied in the git repo for the next upload and
   forwarded upstream.
 
  Better strategy: Forward upstream first, then merge back whatever
  changes upstream really implements.
 
 Oh yeah, got to fix that now. I'll forward these changes upstream first then 
 from 
 now on.

Thanks.

Diego




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#530436: ffmpeg-debian: FTBFS on Hurd

2009-05-25 Thread Diego Biurrun
On Mon, May 25, 2009 at 03:26:50AM -0400, Andres Mejia wrote:
 On Monday 25 May 2009 01:34:15 Diego Biurrun wrote:
  On Sun, May 24, 2009 at 08:53:08PM -0400, Andres Mejia wrote:
   On Sunday 24 May 2009 17:08:17 Marc Dequènes (Duck) wrote:
The configure script does not support the GNU system. The attached
very small patch adds the required modification to fix this problem.
Please consider applying it in your next upload and help push this
change upstream.
  
   Thanks. It's been applied in the git repo for the next upload and
   forwarded upstream.
 
  Better strategy: Forward upstream first, then merge back whatever
  changes upstream really implements.
 
 Oh yeah, got to fix that now. I'll forward these changes upstream first then 
 from 
 now on.

Note that I changed your patch twice already.  While yours is correct
even if not optimal, committing it prematurely still caused you
duplicated work.  Had it been buggy, there might have been trouble...

With an upstream as active and responsive as FFmpeg I think there is
never a reason to commit a patch before it is accepted upstream.

Diego




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#530436: ffmpeg-debian: FTBFS on Hurd

2009-05-24 Thread Marc Dequènes (Duck)

Package: ffmpeg-debian
Version: 4:0.5+svn20090420-2
Severity: important
Tags: patch


Coin,

The configure script does not support the GNU system. The attached  
very small patch adds the required modification to fix this problem.  
Please consider applying it in your next upload and help push this  
change upstream.


Regards.

--
Marc Dequènes (Duck)
diff -Nur orig/ffmpeg-debian-0.5+svn20090420/debian/changelog ffmpeg-debian-0.5+svn20090420/debian/changelog
--- orig/ffmpeg-debian-0.5+svn20090420/debian/changelog	2009-05-24 22:50:56.0 +0200
+++ ffmpeg-debian-0.5+svn20090420/debian/changelog	2009-05-24 20:17:39.0 +0200
@@ -1,3 +1,9 @@
+ffmpeg-debian (4:0.5+svn20090420-2+hurdfr1) unreleased; urgency=low
+
+  * Added patch 'hurd_support' to add support for Hurd.
+
+ -- Marc Dequènes (Duck) d...@hurdfr.org  Sun, 24 May 2009 20:17:33 +0200
+
 ffmpeg-debian (4:0.5+svn20090420-2) unstable; urgency=low
 
   * debian/control: fix dependencies for libavutil-dev and libavfilter-dev
diff -Nur orig/ffmpeg-debian-0.5+svn20090420/debian/patches/hurd_support ffmpeg-debian-0.5+svn20090420/debian/patches/hurd_support
--- orig/ffmpeg-debian-0.5+svn20090420/debian/patches/hurd_support	1970-01-01 01:00:00.0 +0100
+++ ffmpeg-debian-0.5+svn20090420/debian/patches/hurd_support	2009-05-24 20:12:37.0 +0200
@@ -0,0 +1,14 @@
+Index: ffmpeg-debian-0.5+svn20090420/configure
+===
+--- ffmpeg-debian-0.5+svn20090420.orig/configure	2009-05-24 20:11:17.0 +0200
 ffmpeg-debian-0.5+svn20090420/configure	2009-05-24 20:12:30.0 +0200
+@@ -1609,6 +1609,9 @@
+ linux)
+ enable dv1394
+ ;;
++gnu)
++disable dv1394
++;;
+ irix*)
+ target_os=irix
+ ranlib=echo ignoring ranlib
diff -Nur orig/ffmpeg-debian-0.5+svn20090420/debian/patches/series ffmpeg-debian-0.5+svn20090420/debian/patches/series
--- orig/ffmpeg-debian-0.5+svn20090420/debian/patches/series	2009-05-24 22:50:56.0 +0200
+++ ffmpeg-debian-0.5+svn20090420/debian/patches/series	2009-05-24 20:11:12.0 +0200
@@ -1 +1,2 @@
 900_doxyfile
+hurd_support

pgpeI8fY2VHPX.pgp
Description: PGP Digital Signature


Bug#530436: ffmpeg-debian: FTBFS on Hurd

2009-05-24 Thread Andres Mejia
On Sunday 24 May 2009 17:08:17 Marc Dequènes (Duck) wrote:
 Package: ffmpeg-debian
 Version: 4:0.5+svn20090420-2
 Severity: important
 Tags: patch


 Coin,

 The configure script does not support the GNU system. The attached
 very small patch adds the required modification to fix this problem.
 Please consider applying it in your next upload and help push this
 change upstream.

 Regards.

Thanks. It's been applied in the git repo for the next upload and forwarded 
upstream.

-- 
Regards,
Andres



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#530436: ffmpeg-debian: FTBFS on Hurd

2009-05-24 Thread Diego Biurrun
On Sun, May 24, 2009 at 08:53:08PM -0400, Andres Mejia wrote:
 On Sunday 24 May 2009 17:08:17 Marc Dequènes (Duck) wrote:
 
  The configure script does not support the GNU system. The attached
  very small patch adds the required modification to fix this problem.
  Please consider applying it in your next upload and help push this
  change upstream.
 
 Thanks. It's been applied in the git repo for the next upload and forwarded 
 upstream.

Better strategy: Forward upstream first, then merge back whatever
changes upstream really implements.

Diego




--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org