Package: release.debian.org Severity: normal Tags: patch User: release.debian....@packages.debian.org Usertags: pu
Dear release team, yorick-av has an important bug (important impact on usability, does not make the package totally useless) that I only notice now while working on porting this package to the upcoming FFmpeg in experimental: https://bugs.debian.org/890880 The fix is one-line in the C code (at two places). While I am working on more details for unstable, I would like to be given the opportunity to upload this one-liner fix to stable. I attach a source diff. Kind regards, Thibaut. -- System Information: Debian Release: 9.3 APT prefers stable-updates APT policy: (900, 'stable-updates'), (900, 'stable'), (1, 'experimental') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 4.9.0-5-amd64 (SMP w/8 CPU cores) Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8), LANGUAGE=fr_FR.UTF-8 (charmap=UTF-8)
diff --git a/debian/changelog b/debian/changelog index 5a2c216..188f477 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,10 @@ +yorick-av (0.0.4-2) stable; urgency=low + + * Bug fix: "AVPacket timestamps need to be rescaled for most codecs" + (Closes: #890880). + + -- Thibaut Paumard <thib...@debian.org> Tue, 20 Feb 2018 11:37:10 +0100 + yorick-av (0.0.4-1) unstable; urgency=low * New upstream release diff --git a/debian/patches/rescale-ts b/debian/patches/rescale-ts new file mode 100644 index 0000000..bf934fc --- /dev/null +++ b/debian/patches/rescale-ts @@ -0,0 +1,29 @@ +Description: Rescale frame timestamps + Without this, the timestamps are wrong with most codecs, making the + output garbage. +Author: Thibaut Paumard <thib...@debian.org> +Origin: vendor +Bug-Debian: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=890880 +Forwarded: not-needed +Applied-Upstream: ed6b754e03f280708991f579db42dca136431c35 +Last-Update: 2018-02-20 +--- +This patch header follows DEP-3: http://dep.debian.net/deps/dep3/ +--- a/yav.c ++++ b/yav.c +@@ -391,6 +391,7 @@ + AVPacket pkt; + av_init_packet(&pkt); + pkt.flags |= AV_PKT_FLAG_KEY; ++ av_packet_rescale_ts(&pkt, c->time_base, obj->video_st->time_base); + pkt.stream_index = obj->video_st->index; + pkt.data= obj->video_outbuf; + // pkt.size= out_size; +@@ -408,6 +409,7 @@ + } + /* If size is zero, it means the image was buffered. */ + if (!ret && got_packet && pkt.size) { ++ av_packet_rescale_ts(&pkt, c->time_base, obj->video_st->time_base); + pkt.stream_index = obj->video_st->index; + /* Write the compressed frame to the media file. */ + ret = av_interleaved_write_frame(obj->oc, &pkt); diff --git a/debian/patches/series b/debian/patches/series index e69de29..6c46fab 100644 --- a/debian/patches/series +++ b/debian/patches/series @@ -0,0 +1 @@ +rescale-ts