Bug#890889: stretch-pu: package yorick-av/0.0.4-1

[Thibaut Paumard <thibaut@debian.org>]

Package: release.debian.org
Severity: normal
Tags: patch
User: release.debian.org@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 <thibaut@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 <thibaut@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