Bug#587898: grammar error makes sentence one big question mark
Package: ffmpeg Version: 4:0.6-2 Severity: wishlist File: /usr/share/doc/ffmpeg/README.Debian.gz This important sentence Currently the following video encoders are disabled in the ffmpeg package: H263, H264, MPEG2 video, MPEG4 and MS-MPEG4. No *decoders* are disabled in any the ffmpeg package! ^^^has a grammar error here making it a guess to what it means. ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Bug#587904: error message should say what to try next if codec not found
Package: ffmpeg Version: 4:0.6-2 Severity: wishlist $ ffmpeg -i file.flv o.mp3 ... Input #0, flv, from 'file.flv':... Duration: 00:03:50.22, start: 0.00, bitrate: 269 kb/s Stream #0.0: Video: flv, yuv420p, 320x240, 261 kb/s, 29.92 tbr, 1k tbn, 1k tbc Stream #0.1: Audio: mp3, 22050 Hz, 2 channels, s16, 8 kb/s Output #0, mp3, to 'o.mp3': Stream #0.0: Audio: 0x, 22050 Hz, 2 channels, s16, 64 kb/s Stream mapping: Stream #0.1 - #0.0 Encoder (codec id 86017) not found for output stream #0.0 Well then the package is broken, or some message saying what to try next should be given. The reason may be documented in /usr/share/doc/ffmpeg/README.Debian.gz but it is very hard to tell. OK, probably the message should be **codec disabled for legal reasons by Debian. Try putting debian-multimedia... on top of your /etc/apt/sources.list... and do apt-get update, then reinstall their version of this package** P.S., does this really turn 8 kb/s into 64 kb/s? Is that a good thing to do? Well I tried -ab 8k and found out that the Stream #0.1: Audio: mp3, 22050 Hz, 2 channels, s16, 8 kb/s must be wrong. ffprobe gives the same wrong answer. P.S.S., the man page, -ac channels Set the number of audio channels (default = 1). is wrong. As you see above two channels were the output, even without this option. If it just extracts the mp3 then it should say so in several of the affected places on the man page. ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Re: Bug#587904: error message should say what to try next if codec not found
Am 02.07.2010 15:34, schrieb jida...@jidanni.org: The reason may be documented in /usr/share/doc/ffmpeg/README.Debian.gz but it is very hard to tell. I fail to see what is difficult to understand in this sentence from README.Debian: I did not activate MP3 encoding (through LAME) in libavcodec [...] However, I agree that the error message Encoder (codec id 86017) not found for output stream #0.0 should be improved to at least call the encoder by name. OK, probably the message should be **codec disabled for legal reasons by Debian. Try putting debian-multimedia... on top of your /etc/apt/sources.list... and do apt-get update, then reinstall their version of this package** We have very good reasons to never ever recommend using packages from this repository. Furthermore, the MP3 encoder has never been actively *disabled* in Debian. ffmpeg is simply not built against LAME, because this is not available in Debian at all. P.S., does this really turn 8 kb/s into 64 kb/s? Is that a good thing to do? Well I tried -ab 8k and found out that the Stream #0.1: Audio: mp3, 22050 Hz, 2 channels, s16, 8 kb/s must be wrong. ffprobe gives the same wrong answer. This is a completely different issue and should get its own bug report. P.S.S., the man page, -ac channels Set the number of audio channels (default = 1). is wrong. As you see above two channels were the output, even without this option. If it just extracts the mp3 then it should say so in several of the affected places on the man page. dito. - Fabian ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Re: Bug#587904: error message should say what to try next if codec not found
Am 02.07.2010 15:59, schrieb Fabian Greffrath:
However, I agree that the error message
Encoder (codec id 86017) not found for output stream #0.0
should be improved to at least call the encoder by name.
Which will be done by the attached patch. Would you please try it out?
- Fabian
diff --git a/ffmpeg.c b/ffmpeg.c
index 6b9380f..e83e1a5 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -2133,8 +2133,8 @@ static int av_transcode(AVFormatContext **output_files,
if (!codec)
codec = avcodec_find_encoder(ost-st-codec-codec_id);
if (!codec) {
-snprintf(error, sizeof(error), Encoder (codec id %d) not found for output stream #%d.%d,
- ost-st-codec-codec_id, ost-file_index, ost-index);
+snprintf(error, sizeof(error), Encoder (codec name %s) not found for output stream #%d.%d,
+ ost-st-codec-codec_name, ost-file_index, ost-index);
ret = AVERROR(EINVAL);
goto dump_format;
}
@@ -2156,8 +2156,8 @@ static int av_transcode(AVFormatContext **output_files,
if (!codec)
codec = avcodec_find_decoder(ist-st-codec-codec_id);
if (!codec) {
-snprintf(error, sizeof(error), Decoder (codec id %d) not found for input stream #%d.%d,
-ist-st-codec-codec_id, ist-file_index, ist-index);
+snprintf(error, sizeof(error), Decoder (codec name %s) not found for input stream #%d.%d,
+ist-st-codec-codec_name, ist-file_index, ist-index);
ret = AVERROR(EINVAL);
goto dump_format;
}
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Bug#587898: grammar error makes sentence one big question mark
Am 02.07.2010 13:22, schrieb jida...@jidanni.org: Currently the following video encoders are disabled in the ffmpeg package: H263, H264, MPEG2 video, MPEG4 and MS-MPEG4. No *decoders* are disabled in any the ffmpeg package! ^^^has a grammar error here making it a guess to what it means. Well, I guess the meaning of this sentence is not *that* hard to guess because of this error, but thanks for reporting anyway. ;) Fixed in git. - Fabian ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Bug#587898: Bug#587904: error message should say what to try next if codec not found
[Dammit, this didn't go to the bug and submitter by accident.] Am 02.07.2010 15:34, schrieb jida...@jidanni.org: The reason may be documented in /usr/share/doc/ffmpeg/README.Debian.gz but it is very hard to tell. I fail to see what is difficult to understand in this sentence from README.Debian: I did not activate MP3 encoding (through LAME) in libavcodec [...] However, I agree that the error message Encoder (codec id 86017) not found for output stream #0.0 should be improved to at least call the encoder by name. OK, probably the message should be **codec disabled for legal reasons by Debian. Try putting debian-multimedia... on top of your /etc/apt/sources.list... and do apt-get update, then reinstall their version of this package** We have very good reasons to never ever recommend using packages from this repository. Furthermore, the MP3 encoder has never been actively *disabled* in Debian. ffmpeg is simply not built against LAME, because this is not available in Debian at all. P.S., does this really turn 8 kb/s into 64 kb/s? Is that a good thing to do? Well I tried -ab 8k and found out that the Stream #0.1: Audio: mp3, 22050 Hz, 2 channels, s16, 8 kb/s must be wrong. ffprobe gives the same wrong answer. This is a completely different issue and should get its own bug report. P.S.S., the man page, -ac channels Set the number of audio channels (default = 1). is wrong. As you see above two channels were the output, even without this option. If it just extracts the mp3 then it should say so in several of the affected places on the man page. dito. - Fabian ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Bug#587898: Bug#587904: error message should say what to try next if codec not found
Am 02.07.2010 15:59, schrieb Fabian Greffrath:
However, I agree that the error message
Encoder (codec id 86017) not found for output stream #0.0
should be improved to at least call the encoder by name.
Which will be done by the attached patch. Would you please try it out?
- Fabian
diff --git a/ffmpeg.c b/ffmpeg.c
index 6b9380f..e83e1a5 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -2133,8 +2133,8 @@ static int av_transcode(AVFormatContext **output_files,
if (!codec)
codec = avcodec_find_encoder(ost-st-codec-codec_id);
if (!codec) {
-snprintf(error, sizeof(error), Encoder (codec id %d) not found for output stream #%d.%d,
- ost-st-codec-codec_id, ost-file_index, ost-index);
+snprintf(error, sizeof(error), Encoder (codec name %s) not found for output stream #%d.%d,
+ ost-st-codec-codec_name, ost-file_index, ost-index);
ret = AVERROR(EINVAL);
goto dump_format;
}
@@ -2156,8 +2156,8 @@ static int av_transcode(AVFormatContext **output_files,
if (!codec)
codec = avcodec_find_decoder(ist-st-codec-codec_id);
if (!codec) {
-snprintf(error, sizeof(error), Decoder (codec id %d) not found for input stream #%d.%d,
-ist-st-codec-codec_id, ist-file_index, ist-index);
+snprintf(error, sizeof(error), Decoder (codec name %s) not found for input stream #%d.%d,
+ist-st-codec-codec_name, ist-file_index, ist-index);
ret = AVERROR(EINVAL);
goto dump_format;
}
___
pkg-multimedia-maintainers mailing list
pkg-multimedia-maintainers@lists.alioth.debian.org
http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Bug#587898: Bug#587904: error message should say what to try next if codec not found
All I know is it seems to the average user that ffmpeg can't even make .mp3s, a rather common format. So they spread the word forget ffmpeg. So that's why your patch should at least say ***Probable causes:... There's only a small chance that the user will be nurd enough to dig out the README files and figure out what they mean. There's a much bigger chance that they will spread the word ffmpeg is busted. Skip it. So it appears your version is doing more harm than good in the world, maybe. Therefore in order to not damage ffmpeg's reputation, there should be a runtime ***Probable causes:... that tells the user all hope is not lost because they can simply install this other package as hinted to in the readme file... FG We have very good reasons to never ever recommend using packages from FG this repository. Furthermore, the MP3 encoder has never been actively FG *disabled* in Debian. ffmpeg is simply not built against LAME, because FG this is not available in Debian at all. All I know is you mentioned it in the readme, so you can have the error message mention to see the readme at least. FG This is a completely different issue and should get its own bug report. OK, I made https://roundup.ffmpeg.org/msg11039 https://roundup.ffmpeg.org/msg11041 ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
jack-rack 1.4.7-2 MIGRATED to testing
FYI: The status of the jack-rack source package in Debian's testing distribution has changed. Previous version: 1.4.7-1 Current version: 1.4.7-2 -- This email is automatically generated once a day. As the installation of new packages into testing happens multiple times a day you will receive later changes on the next day. See http://release.debian.org/testing-watch/ for more information. ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Cecilia 4 released
I'd like to take cecilia into pkg-multimedia, and import the latest version. Free, do you agree? -- Saludos, Felipe Sateler signature.asc Description: OpenPGP digital signature ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
Re: Cecilia 4 released
On 02/07/10 14:17, Felipe Sateler wrote: I'd like to take cecilia into pkg-multimedia, and import the latest version. Free, do you agree? And now with the correct address. -- Saludos, Felipe Sateler signature.asc Description: OpenPGP digital signature ___ pkg-multimedia-maintainers mailing list pkg-multimedia-maintainers@lists.alioth.debian.org http://lists.alioth.debian.org/mailman/listinfo/pkg-multimedia-maintainers
