Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Audiowmark fails to build on Arch #62

Open
tim-janik opened this issue Aug 20, 2024 · 0 comments · May be fixed by #68
Open

Audiowmark fails to build on Arch #62

tim-janik opened this issue Aug 20, 2024 · 0 comments · May be fixed by #68

Comments

@tim-janik
Copy link
Contributor

Recently, Arch has upgraded to ffmpeg-7, which removed deprecated APIs from ffmpeg-6.1..

Atm, building the repo on Arch fails as follows:

$ docker build -t audiowmark-arch -f misc/Dockerfile-arch .
15.97 g++ -std=gnu++14 -DHAVE_CONFIG_H -I. -I..    -I/usr/include/opus     -g -O2 -Wall -pthread -MT testrandom.o -MD -MP -MF .deps/testrandom.Tpo -c -o testrandom.o testrandom.cc
15.99 hlsoutputstream.cc: In member function ‘Error HLSOutputStream::add_stream(const AVCodec**, AVCodecID)’:
15.99 hlsoutputstream.cc:116:26: error: ‘av_get_channel_layout’ was not declared in this scope; did you mean ‘set_channel_layout’?
15.99   116 |   uint64_t want_layout = av_get_channel_layout (m_channel_layout.c_str());
15.99       |                          ^~~~~~~~~~~~~~~~~~~~~
15.99       |                          set_channel_layout
15.99 hlsoutputstream.cc:119:10: error: ‘AVCodecContext’ {aka ‘struct AVCodecContext’} has no member named ‘channel_layout’; did you mean ‘ch_layout’?
15.99   119 |   m_enc->channel_layout = want_layout;
15.99       |          ^~~~~~~~~~~~~~
15.99       |          ch_layout

> pacman -Q
0.452 ffmpeg 2:7.0.2-1
tim-janik added a commit to tim-janik/audiowmark that referenced this issue Oct 29, 2024
Fixes swesterfeld#62: swesterfeld#62

Starting ffmpeg-7, the old av_get_channel_layout() API
is gone and use of AVChannelLayout is mandatory.

Signed-off-by: Tim Janik <timj@gnu.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant