Request 1200898 (revoked)

You're not reviewing the full diff of request 1200898 , but the diff to the superseded request 1197967 (Show full diff)

Overview

Request 1200898 revoked

- fix build for Tumbleweed, by using ffmpeg-6 not ffmpeg-7

I resubmit this, as my comment didn't get any traction in the old SR.
Because ffmpeg-6-mini-devel doesn't (for some reason) expose the pkgconfig
files. Which therefore means, there is no way for me to actually use pkgconfig() < 61

Created by rrahl0 2 months ago
In state revoked
Supersedes 1197967
Open review for openSUSE:Factory:Staging:adi:17

Submit cosmic-player

Comments for request 1200898 1
Comments for request 1197967 3

Marcus Rueckert (darix) - 2 months ago

reviewer

you can actually differentiate at pkgconfig level

pkgconfig(libavcodec) = 60.31.102

pkgconfig(libavcodec) = 61.3.100

so you could do something like

pkgconfig(libavcodec) < 61

we would really want to get rid of ffmpeg older than 7 and many of the patches i have seen for other apps, it is not much work to actually port from 6 to 7. did you contact upstream about it?

Marcus Rueckert (darix) - 3 months ago

reviewer

can you please check if you can patch this to use ffmpeg-7? there is a few packages already have patches which you can use as inspiration.

Fabian Vogt (favogt) - 3 months ago

IMO the correct way is to

BuildRequires: pkgconfig(libav...) < 5x.0

Richard Rahl (rrahl0) - 3 months ago

author source maintainer

so I looked into this, and sadly it doesn't work that way, as for some reason ffmpeg-6-mini-devel doesn't expose the pkgconfig(). (while still there). so I can't pkgconfig() < 61