Merge ~liushuyu-011/ubuntu/+source/janus:ubuntu/devel into ubuntu/+source/janus:ubuntu/devel

Proposed by Zixing Liu
Status: Merged
Merged at revision: 4d2fcec1d1119e334e2690ae37a6f33f7c77838c
Proposed branch: ~liushuyu-011/ubuntu/+source/janus:ubuntu/devel
Merge into: ubuntu/+source/janus:ubuntu/devel
Diff against target: 56 lines (+34/-0)
3 files modified
debian/changelog (+6/-0)
debian/patches/2010_fix_ffmpeg_7.patch (+27/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Graham Inggs (community) Approve
Review via email: mp+473389@code.launchpad.net

Description of the change

This merge proposal fixes incompatibilities with FFmpeg 7 for Janus.

To post a comment you must log in.
Revision history for this message
Graham Inggs (ginggs) wrote :

Looks good to me, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index e195bd0..86de9f8 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,9 @@
6+janus (1.1.2-1ubuntu1) oracular; urgency=medium
7+
8+ * d/p/2010_fix_ffmpeg_7.patch: fix FTBFS with FFmpeg 7
9+
10+ -- Zixing Liu <zixing.liu@canonical.com> Tue, 17 Sep 2024 13:01:53 -0600
11+
12 janus (1.1.2-1build5) noble; urgency=medium
13
14 * No-change rebuild against libssl3t64.
15diff --git a/debian/patches/2010_fix_ffmpeg_7.patch b/debian/patches/2010_fix_ffmpeg_7.patch
16new file mode 100644
17index 0000000..3a5f0f1
18--- /dev/null
19+++ b/debian/patches/2010_fix_ffmpeg_7.patch
20@@ -0,0 +1,27 @@
21+Description: Fix FTBFS with FFmpeg 7
22+Author: Zixing Liu <zixing.liu@canonical.com>
23+Forwarded: no
24+Last-Update: 2024-09-17
25+---
26+--- janus-1.1.2.orig/src/postprocessing/pp-avformat.c
27++++ janus-1.1.2/src/postprocessing/pp-avformat.c
28+@@ -64,7 +64,7 @@ AVStream *janus_pp_new_audio_avstream(AV
29+ c->codec_id = codec_id;
30+ c->codec_type = AVMEDIA_TYPE_AUDIO;
31+ c->sample_rate = samplerate;
32+- c->channels = channels;
33++ av_channel_layout_default(&c->ch_layout, channels);
34+ if(extradata) {
35+ c->extradata_size = size;
36+ c->extradata = av_memdup(extradata, size);
37+--- janus-1.1.2.orig/src/postprocessing/pp-g722.c
38++++ janus-1.1.2/src/postprocessing/pp-g722.c
39+@@ -212,7 +212,7 @@ int janus_pp_g722_process(FILE *file, ja
40+ int data_size = av_get_bytes_per_sample(dec_ctx->sample_fmt);
41+ int i=0, ch=0;
42+ for(i=0; i<frame->nb_samples; i++) {
43+- for(ch=0; ch<dec_ctx->channels; ch++) {
44++ for(ch=0; ch<dec_ctx->ch_layout.nb_channels; ch++) {
45+ fwrite(frame->data[ch] + data_size*i, 1, data_size, wav_file);
46+ }
47+ }
48diff --git a/debian/patches/series b/debian/patches/series
49index 61ae7e0..1383b83 100644
50--- a/debian/patches/series
51+++ b/debian/patches/series
52@@ -11,3 +11,4 @@
53 2007_use_shared_janus.patch
54 2008_avoid_virtualbg.patch
55 2009_rollup_3.patch
56+2010_fix_ffmpeg_7.patch

Subscribers

People subscribed via source and target branches

to all changes: