bug-gnupod
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-gnupod] Bug fixes for gnupod.


From: H. Langos
Subject: Re: [Bug-gnupod] Bug fixes for gnupod.
Date: Wed, 1 Jun 2011 01:34:15 +0200
User-agent: Mutt/1.5.20 (2009-06-14)

Hi Brad & Richard,

On Tue, May 31, 2011 at 06:11:10PM -0400, Brad wrote:
> On 31/05/11 2:55 PM, Richard van den Berg wrote:
> >Hello Brad,
> >
> >Thanks for the ffmpeg fixes, they look very useful.

@Richard/Brad: Do you happen to know when ffmpeg changed their output?
It's not like gnupod was wrong using "-formats". I just checked a
debian lenny and there "-codecs" yields an error:
=============
$ ffmpeg -codecs
FFmpeg version SVN-r13582, Copyright (c) 2000-2008 Fabrice Bellard, et al.
  configuration: --prefix=/usr --libdir=${prefix}/lib
--shlibdir=${prefix}/lib --bindir=${prefix}/bin
--incdir=${prefix}/include/ffmpeg --enable-shared --enable-libmp3lame
--enable-gpl --enable-libfaad --mandir=${prefix}/share/man
--enable-libvorbis --enable-pthreads --enable-libfaac --enable-libxvid
--enable-postproc --enable-libamr-nb --enable-libamr-wb --enable-x11grab
--enable-libgsm --enable-libx264 --enable-liba52 --enable-libtheora
--extra-cflags=-Wall -g -fPIC -DPIC --cc=ccache cc --enable-swscale
--enable-libdc1394 --enable-nonfree --disable-mmx --disable-stripping
--enable-avfilter --enable-libdirac --disable-decoder=libdirac
--enable-libschroedinger --disable-encoder=libschroedinger --disable-altivec
--disable-armv5te --disable-armv6 --disable-vis
  libavutil version: 49.7.0
  libavcodec version: 51.58.0
  libavformat version: 52.16.0
  libavdevice version: 52.0.0
  libavfilter version: 0.0.0
  built on May  3 2009 12:02:42, gcc: 4.3.2
ffmpeg: missing argument for option '-codecs'
==============

It seems we might have to support both "-formats" and "-codecs" if we want
to keep support for older systems alive.

> >On 30-5-11 22:39 , Brad wrote:
> >>  printf "%s" "checking for ImageMagick..."
> >>  convert --version>  /dev/null 2>&1
> >>- if test $? = 0; then
> >>+ if test $? = 1; then
> >

I think we've seen that issue before ... let me google that
for you :-)

http://lmgtfy.com/?q=gnupod+convert+exit+code

I never heard back from chris or the ImageMagick developers
so it remains a mystery why they changed the exit code.
But as Richard mentioned below, the 6.6 version (just checked
the one on ubuntu 11.04 and debian squeeze) seem to have reverted
to the old behaviour:
=====
address@hidden:~$ convert -version
Version: ImageMagick 6.6.2-6 2011-03-16 Q16 http://www.imagemagick.org
Copyright: Copyright (C) 1999-2010 ImageMagick Studio LLC
Features: OpenMP 

address@hidden:~$ echo $?
0
=====
address@hidden:~$ convert -version
Version: ImageMagick 6.6.0-4 2010-11-16 Q16 http://www.imagemagick.org
Copyright: Copyright (C) 1999-2010 ImageMagick Studio LLC
Features: OpenMP 

address@hidden:~$ echo $?
0
=====

So I guess 6.4 was a glitch.

> Anyway, I'll keep that part as a local change for our OpenBSD port just
> so developers and so on are not confused about the autoconf script not
> finding ImageMagick even though it is there.

If you implement changes in your OpenBSD port I suggest you
set yourself a flag to change it back when you upgrade to a newer
ImageMagick version.

BTW: If you have suggestions to improve the portability of gnupod, please
feel free to contact us. I have been busy (more than usual) in the last half
year, but I hope I'll get around to rewrite the backend to support more
ipod models ... eventually.

cheers
-henrik





reply via email to

[Prev in Thread] Current Thread [Next in Thread]