Control: severity -1 important Control: tags -1 patch
On Thu, 31 Jan 2013 00:43:44 +0100 chrysn <chr...@fsfe.org> wrote: > Package: performous-tools > Version: 0.7.0-1 > Severity: minor > File: /usr/games/ss_extract > > when running `ss_extract --dvd ../some_singstar_style_dvd`, ss_extract > uses ffmpeg in a deprecated way: This is not considered to be an issue by the developers of Performous because other distributions still use ffmpeg instead of avconv. Their intention is to fix it for all systems sometimes by using the corresponding library instead of command-line options. Unfortunately ffmpeg will not be shipped for Jessie hence we should just change the ffmpeg command to avconv which does basically the same thing. I am attaching a patch for this bug. Markus
From: Markus Koschany <a...@gambaru.de>
Date: Sun, 2 Nov 2014 00:57:00 +0100
Subject: use avconv
Use avconv command instead of ffmpeg because the latter is not available on
Debian systems. This should be revisited for Jessie+1.
Forwarded: not-needed
---
tools/ss_extract.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/ss_extract.cc b/tools/ss_extract.cc
index 2a94110..6677b8f 100644
--- a/tools/ss_extract.cc
+++ b/tools/ss_extract.cc
@@ -213,7 +213,7 @@ struct Process {
}
if (g_mkvcompress) {
std::cerr << ">>> Compressing video into video.m4v" << std::endl;
- std::string cmd = "ffmpeg -i \"" + (path / "video.mpg").string() + "\" -vcodec libx264 -profile main -crf 25 -threads 0 -metadata album=\"" + song.edition + "\" -metadata author=\"" + song.artist + "\" -metadata comment=\"" + song.genre + "\" -metadata title=\"" + song.title + "\" \"" + (path / "video.m4v\"").string();
+ std::string cmd = "avconv -i \"" + (path / "video.mpg").string() + "\" -vcodec libx264 -profile main -crf 25 -threads 0 -metadata album=\"" + song.edition + "\" -metadata author=\"" + song.artist + "\" -metadata comment=\"" + song.genre + "\" -metadata title=\"" + song.title + "\" \"" + (path / "video.m4v\"").string();
std::cerr << cmd << std::endl;
if (std::system(cmd.c_str()) == 0) { // FIXME: std::system return value is not portable
fs::remove(path / "video.mpg");
signature.asc
Description: OpenPGP digital signature
