search

WinFF / winff

Automatically exported from code.google.com/p/winff
213 stars 50 forks source link

ffmpeg doesn't accept values in kb #109

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. any conversion
2.
3.

What is the expected output? What do you see instead?
ffmpeg refuses to accept rate value in kbs, only in b, so ffmpeg fails to parse 
command line parameters provided by presets.xml

What version of the product are you using? On what operating system?

FFmpeg version SVN-r25838, winff 1.3.1 debian 64, presets-libavcodec52-v5.xml, 
libavcodec52.
Please provide any additional information below.

Original issue reported on code.google.com by ste.mont...@gmail.com on 15 Feb 2011 at 4:59

GoogleCodeExporter commented 9 years ago 
Do I understand correctly that you are reporting the issue with the 
libavcodec52-v5.xml presets? I will check if it indeed need an update.

Original comment by poipodec...@hotmail.com on 16 Feb 2011 at 6:49

GoogleCodeExporter commented 9 years ago 
it's a newer ffmpeg than the presets. It not only requires preset updates but 
code updates as well

Original comment by bgg...@gmail.com on 16 Feb 2011 at 7:23

GoogleCodeExporter commented 9 years ago 
Ok, i'll wait for; at the moment, i'm converting by command line.

Original comment by ste.mont...@gmail.com on 17 Feb 2011 at 9:42

GoogleCodeExporter commented 9 years ago 
Matt, in revision 480 [1] you implemented code to not use kb in -b and -ab 
options, but k. Is this what is meant here as well? If the answer is yes, we 
still need to update our preset files to NOT use kb anymore, but k instead. 
Then the code update is already done and the "only" change is to presets.xml 
and the libavcodecXX-vX.xmls. Of course I might be mistaken and not 
understanding well.

[1] http://code.google.com/p/winff/source/detail?r=480

Original comment by poipodec...@hotmail.com on 17 Feb 2011 at 8:28

GoogleCodeExporter commented 9 years ago 
oh yes, i forgot. I believe that is correct

Original comment by bgg...@gmail.com on 17 Feb 2011 at 11:38

GoogleCodeExporter commented 9 years ago 
Matt, I will update our presets files this weekend or early next week, if you 
don't beat me to it. I try to get the new version into Ubuntu Natty and want 
this fix there too.

Original comment by poipodec...@hotmail.com on 18 Feb 2011 at 5:41

GoogleCodeExporter commented 9 years ago 
Updated in svn and download section:
http://winff.googlecode.com/files/presets-libavcodec51-v5.xml.gz
http://winff.googlecode.com/files/presets-libavcodec52-v6.xml.gz

Original comment by poipodec...@hotmail.com on 19 Feb 2011 at 1:37