Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-27 Thread Sebastian Dröge
On So, 2013-09-22 at 14:52 +0200, Sebastian Ramacher wrote:
 On 2013-09-13 09:16:35, Sebastian Dröge wrote:
  On Fr, 2013-09-13 at 02:56 +0200, Sebastian Ramacher wrote:
   On 2013-09-12 14:00:43, Sebastian Dröge wrote:
On Do, 2013-09-12 at 12:57 +0200, Sebastian Ramacher wrote:
 On 2013-09-09 20:53:03, Luk Claes wrote:
   Can/should this be removed in favour of gst-libav1.0?
  
   Yes it should probably be just removed. I plan to get rid of all 
   of
   gstreamer 0.10 in the next months, so let's just let this one go 
   now
   already :)
  
  testing removal hint added.
 
 That didn't work - there are reverse dependencies. Seems like
 gstreamer0.10-ffmpeg needs to be fixed.

The rdepends should be ported to GStreamer 1.0 instead probably. Also
most of these rdepends should at most Recommend the package.
   
   Sure, ideally they would be ported to use GStreamer 1.0 instead. But I'd
   rather not add yet another transition on top of the libav transition.
   
   I've started to backport the necessary changes for libav 9 from
   gst-libav1.0. I'm at a state where it compiles and appreciate another
   pair of eyes looking over it as the patch got really large.
  
  From skipping over it this patch looks good, you only remove support for
  paletted video formats and also some more changes are necessary to
  convert from the planar audio formats that are output by some decoders
  now to an interleaved audio format.
 
 Here is a new version of the patch. Support for paletted video formats
 should be back (as far as I can tell from gst-libav1.0's code
 *_set_palette was removed altogether and is no longer needed). Support
 for planar audio formats gives me headaches as too much has been
 changed. I've started to backport it for the encoder but failed to do
 so for the decoder.

Yeah, there's a reason why we did a new version with improved API after
many years of experience with annoyances and unneeded complications like
that ;)

Generally looks good the patch

 In gst-libav1.0 gst_ffmpegauddec_get_buffer gets a GstAudioInfo pointer
 from a GstAudioDecoder* instance. Where do I get that from in
 gstreamer-ffmpeg0.10? Or is there some other way to get the same
 information? The same questions also bugs me for the encoder.

In 0.10 you could create (with later versions of 0.10) a GstAudioInfo
from the GstCaps. See gst/audio/audio.h for the relevant functions.

 Since GstFFmpegDec is not a GstAudioDecoder, what's the proper
 replacement for gst_audio_decoder_allocate_output_buffer? (as used in
 gst-libav1.0's gst_ffmpegauddec_get_buffer)

Just gst_buffer_new_and_alloc()


signature.asc
Description: This is a digitally signed message part


Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-14 Thread Sebastian Ramacher
On 2013-09-13 09:16:35, Sebastian Dröge wrote:
 On Fr, 2013-09-13 at 02:56 +0200, Sebastian Ramacher wrote:
  On 2013-09-12 14:00:43, Sebastian Dröge wrote:
   On Do, 2013-09-12 at 12:57 +0200, Sebastian Ramacher wrote:
On 2013-09-09 20:53:03, Luk Claes wrote:
  Can/should this be removed in favour of gst-libav1.0?
 
  Yes it should probably be just removed. I plan to get rid of all of
  gstreamer 0.10 in the next months, so let's just let this one go now
  already :)
 
 testing removal hint added.

That didn't work - there are reverse dependencies. Seems like
gstreamer0.10-ffmpeg needs to be fixed.
   
   The rdepends should be ported to GStreamer 1.0 instead probably. Also
   most of these rdepends should at most Recommend the package.
  
  Sure, ideally they would be ported to use GStreamer 1.0 instead. But I'd
  rather not add yet another transition on top of the libav transition.
  
  I've started to backport the necessary changes for libav 9 from
  gst-libav1.0. I'm at a state where it compiles and appreciate another
  pair of eyes looking over it as the patch got really large.
 
 From skipping over it this patch looks good, you only remove support for
 paletted video formats and also some more changes are necessary to
 convert from the planar audio formats that are output by some decoders
 now to an interleaved audio format.

Thank you for the feedback. I'll try to finish the patch early next week.

I've missed that the support for paletted video formats was re-added
later. The commits for the planar audio formats are
cab519f37045ec405d4b6539d232dfaea13def80,
e092f3d41183e9e901c0f86961124d237a82895e,
1bec18edce7b7cf639807218dc0f12dc590ae306 and
cb9e40ff4c78014a336b77bd2b142e12758106ef or are there more?

Regards
-- 
Sebastian Ramacher


signature.asc
Description: Digital signature


Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-13 Thread Sebastian Dröge
On Fr, 2013-09-13 at 02:56 +0200, Sebastian Ramacher wrote:
 On 2013-09-12 14:00:43, Sebastian Dröge wrote:
  On Do, 2013-09-12 at 12:57 +0200, Sebastian Ramacher wrote:
   On 2013-09-09 20:53:03, Luk Claes wrote:
 Can/should this be removed in favour of gst-libav1.0?

 Yes it should probably be just removed. I plan to get rid of all of
 gstreamer 0.10 in the next months, so let's just let this one go now
 already :)

testing removal hint added.
   
   That didn't work - there are reverse dependencies. Seems like
   gstreamer0.10-ffmpeg needs to be fixed.
  
  The rdepends should be ported to GStreamer 1.0 instead probably. Also
  most of these rdepends should at most Recommend the package.
 
 Sure, ideally they would be ported to use GStreamer 1.0 instead. But I'd
 rather not add yet another transition on top of the libav transition.
 
 I've started to backport the necessary changes for libav 9 from
 gst-libav1.0. I'm at a state where it compiles and appreciate another
 pair of eyes looking over it as the patch got really large.

From skipping over it this patch looks good, you only remove support for
paletted video formats and also some more changes are necessary to
convert from the planar audio formats that are output by some decoders
now to an interleaved audio format.


signature.asc
Description: This is a digitally signed message part


Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-12 Thread Sebastian Ramacher
On 2013-09-09 20:53:03, Luk Claes wrote:
  Can/should this be removed in favour of gst-libav1.0?
 
  Yes it should probably be just removed. I plan to get rid of all of
  gstreamer 0.10 in the next months, so let's just let this one go now
  already :)
 
 testing removal hint added.

That didn't work - there are reverse dependencies. Seems like
gstreamer0.10-ffmpeg needs to be fixed.

Regards
-- 
Sebastian Ramacher


signature.asc
Description: Digital signature


Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-12 Thread Sebastian Dröge
On Do, 2013-09-12 at 12:57 +0200, Sebastian Ramacher wrote:
 On 2013-09-09 20:53:03, Luk Claes wrote:
   Can/should this be removed in favour of gst-libav1.0?
  
   Yes it should probably be just removed. I plan to get rid of all of
   gstreamer 0.10 in the next months, so let's just let this one go now
   already :)
  
  testing removal hint added.
 
 That didn't work - there are reverse dependencies. Seems like
 gstreamer0.10-ffmpeg needs to be fixed.

The rdepends should be ported to GStreamer 1.0 instead probably. Also
most of these rdepends should at most Recommend the package.


signature.asc
Description: This is a digitally signed message part


Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-12 Thread Luk Claes
On 09/12/2013 02:00 PM, Sebastian Dröge wrote:
 On Do, 2013-09-12 at 12:57 +0200, Sebastian Ramacher wrote:
 On 2013-09-09 20:53:03, Luk Claes wrote:
 Can/should this be removed in favour of gst-libav1.0?

 Yes it should probably be just removed. I plan to get rid of all of
 gstreamer 0.10 in the next months, so let's just let this one go now
 already :)

 testing removal hint added.

 That didn't work - there are reverse dependencies. Seems like
 gstreamer0.10-ffmpeg needs to be fixed.
 
 The rdepends should be ported to GStreamer 1.0 instead probably. Also
 most of these rdepends should at most Recommend the package.

Will you work on that please, TIA?

Cheers

Luk


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



Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-09 Thread Sebastian Dröge
On Fr, 2013-09-06 at 15:34 +0200, Moritz Muehlenhoff wrote:
 On Sun, Aug 25, 2013 at 03:08:25PM +0200, David Suárez wrote:
  Source: gstreamer0.10-ffmpeg
  Version: 0.10.13-5
  Severity: serious
  Tags: jessie sid
  User: debian...@lists.debian.org
  Usertags: qa-ftbfs-20130825 qa-ftbfs
  Justification: FTBFS on amd64
  
  Hi,
  
  During a rebuild of all packages in sid, your package failed to build on
  amd64.
  
  Related to current libav9 transition (see #706798).
 
 Can/should this be removed in favour of gst-libav1.0?

Yes it should probably be just removed. I plan to get rid of all of
gstreamer 0.10 in the next months, so let's just let this one go now
already :)


signature.asc
Description: This is a digitally signed message part


Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-09 Thread Luk Claes
 Can/should this be removed in favour of gst-libav1.0?

 Yes it should probably be just removed. I plan to get rid of all of
 gstreamer 0.10 in the next months, so let's just let this one go now
 already :)

testing removal hint added.

Cheers

Luk


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



Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-09-06 Thread Moritz Muehlenhoff
On Sun, Aug 25, 2013 at 03:08:25PM +0200, David Suárez wrote:
 Source: gstreamer0.10-ffmpeg
 Version: 0.10.13-5
 Severity: serious
 Tags: jessie sid
 User: debian...@lists.debian.org
 Usertags: qa-ftbfs-20130825 qa-ftbfs
 Justification: FTBFS on amd64
 
 Hi,
 
 During a rebuild of all packages in sid, your package failed to build on
 amd64.
 
 Related to current libav9 transition (see #706798).

Can/should this be removed in favour of gst-libav1.0?

Cheers,
Moritz 


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



Bug#720796: gstreamer0.10-ffmpeg: FTBFS with libav9: gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'

2013-08-25 Thread David Suárez
Source: gstreamer0.10-ffmpeg
Version: 0.10.13-5
Severity: serious
Tags: jessie sid
User: debian...@lists.debian.org
Usertags: qa-ftbfs-20130825 qa-ftbfs
Justification: FTBFS on amd64

Hi,

During a rebuild of all packages in sid, your package failed to build on
amd64.

Related to current libav9 transition (see #706798).

Relevant part (hopefully):
 /bin/bash ../../libtool  --tag=CC --tag=disable-static  --mode=compile gcc 
 -std=gnu99 -DHAVE_CONFIG_H -I. -I../..   -D_FORTIFY_SOURCE=2  -pthread 
 -I/usr/include/gstreamer-0.10 -I/usr/include/glib-2.0 
 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/libxml2   
 -I../../gst-libs -I../../gst-libs -pthread -I/usr/include/gstreamer-0.10 
 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include 
 -I/usr/include/libxml2-Wall -Wdeclaration-after-statement -Wvla 
 -Wpointer-arith -Wmissing-declarations -Wmissing-prototypes -Wredundant-decls 
 -Wundef -Wwrite-strings -Wformat-nonliteral -Wformat-security 
 -Wold-style-definition -Winit-self -Wmissing-include-dirs -Waddress 
 -Waggregate-return -Wno-multichar -Wnested-externs   -g -O2 
 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security 
 -Wall -Wno-error -c -o libgstffmpeg_la-gstffmpeg.lo `test -f 'gstffmpeg.c' || 
 echo './'`gstffmpeg.c
 libtool: compile:  gcc -std=gnu99 -DHAVE_CONFIG_H -I. -I../.. 
 -D_FORTIFY_SOURCE=2 -pthread -I/usr/include/gstreamer-0.10 
 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include 
 -I/usr/include/libxml2 -I../../gst-libs -I../../gst-libs -pthread 
 -I/usr/include/gstreamer-0.10 -I/usr/include/glib-2.0 
 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/libxml2 -Wall 
 -Wdeclaration-after-statement -Wvla -Wpointer-arith -Wmissing-declarations 
 -Wmissing-prototypes -Wredundant-decls -Wundef -Wwrite-strings 
 -Wformat-nonliteral -Wformat-security -Wold-style-definition -Winit-self 
 -Wmissing-include-dirs -Waddress -Waggregate-return -Wno-multichar 
 -Wnested-externs -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat 
 -Werror=format-security -Wall -Wno-error -c gstffmpeg.c  -fPIC -DPIC -o 
 .libs/libgstffmpeg_la-gstffmpeg.o
 In file included from gstffmpeg.c:38:0:
 gstffmpeg.h:63:1: error: unknown type name 'URLProtocol'
  extern URLProtocol gstreamer_protocol;
  ^
 gstffmpeg.h:64:1: error: unknown type name 'URLProtocol'
  extern URLProtocol gstpipe_protocol;
  ^
 In file included from gstffmpeg.c:39:0:
 gstffmpegutils.h: In function 'gst_ffmpeg_time_ff_to_gst':
 gstffmpegutils.h:59:5: warning: implicit declaration of function 
 'av_rescale_q' [-Wimplicit-function-declaration]
  out = av_rescale_q (pts, base, bq);
  ^
 gstffmpegutils.h:59:5: warning: nested extern declaration of 'av_rescale_q' 
 [-Wnested-externs]
 gstffmpegutils.h: At top level:
 gstffmpegutils.h:90:26: warning: 'enum SampleFormat' declared inside 
 parameter list [enabled by default]
  av_smp_format_depth(enum SampleFormat smp_fmt);
   ^
 gstffmpegutils.h:90:26: warning: its scope is only this definition or 
 declaration, which is probably not what you want [enabled by default]
 gstffmpeg.c: In function 'gst_ffmpeg_avcodec_open':
 gstffmpeg.c:51:3: warning: 'g_static_mutex_get_mutex_impl' is deprecated 
 (declared at /usr/include/glib-2.0/glib/deprecated/gthread.h:149): Use 
 'GMutex' instead [-Wdeprecated-declarations]
g_static_mutex_lock (gst_avcodec_mutex);
^
 gstffmpeg.c:52:3: warning: implicit declaration of function 'avcodec_open' 
 [-Wimplicit-function-declaration]
ret = avcodec_open (avctx, codec);
^
 gstffmpeg.c:52:3: warning: nested extern declaration of 'avcodec_open' 
 [-Wnested-externs]
 gstffmpeg.c:53:3: warning: 'g_static_mutex_get_mutex_impl' is deprecated 
 (declared at /usr/include/glib-2.0/glib/deprecated/gthread.h:149): Use 
 'GMutex' instead [-Wdeprecated-declarations]
g_static_mutex_unlock (gst_avcodec_mutex);
^
 gstffmpeg.c: In function 'gst_ffmpeg_avcodec_close':
 gstffmpeg.c:63:3: warning: 'g_static_mutex_get_mutex_impl' is deprecated 
 (declared at /usr/include/glib-2.0/glib/deprecated/gthread.h:149): Use 
 'GMutex' instead [-Wdeprecated-declarations]
g_static_mutex_lock (gst_avcodec_mutex);
^
 gstffmpeg.c:65:3: warning: 'g_static_mutex_get_mutex_impl' is deprecated 
 (declared at /usr/include/glib-2.0/glib/deprecated/gthread.h:149): Use 
 'GMutex' instead [-Wdeprecated-declarations]
g_static_mutex_unlock (gst_avcodec_mutex);
^
 gstffmpeg.c: In function 'gst_ffmpeg_av_find_stream_info':
 gstffmpeg.c:75:3: warning: 'g_static_mutex_get_mutex_impl' is deprecated 
 (declared at /usr/include/glib-2.0/glib/deprecated/gthread.h:149): Use 
 'GMutex' instead [-Wdeprecated-declarations]
g_static_mutex_lock (gst_avcodec_mutex);
^
 gstffmpeg.c:76:3: warning: implicit declaration of function 
 'av_find_stream_info' [-Wimplicit-function-declaration]
ret = av_find_stream_info (ic);
^
 gstffmpeg.c:76:3: warning: nested extern