michaelni changed the topic of #ffmpeg-devel to: Welcome to the FFmpeg development channel | Questions about using FFmpeg or developing with libav* libs should be asked in #ffmpeg | This channel is publicly logged | FFmpeg 7.1 has been released! | Please read ffmpeg.org/developer.html#Code-of-conduct
rvalue has quit [Ping timeout: 252 seconds]
MyNetAz has quit [Remote host closed the connection]
Traneptora has joined #ffmpeg-devel
<BBB>
wbs: yes, I suppose that would be good. michaelni: are you in charge of backports?
ngaullier has quit [Remote host closed the connection]
<Traneptora>
it's 1 a.m. in Europe rn so he may be asleep
<BtbN>
nobody is directly in charge of backport
<BtbN>
he just picks a ton of fixes every release on an old branch
<BtbN>
If a patch is fix for a bug that does not introduce new features or breaks anything, just push it
<Traneptora>
I was under the impression the usual method is that if you fix bugs without changing (non-bug) behavior then you just backport it as far as it merges
<Traneptora>
i.e. as far as it applies cleanly
<BtbN>
yeah, just cherry-pick away for pure bugfixes
<BBB>
don't have a lot of experience with backports, afaik I've never backported anything
<cone-141>
ffmpeg Leo Izen master:f3c408264554: avcodec/libjxl: add animated JPEG XL encoder
<cone-141>
ffmpeg Leo Izen master:07e54f9b5c1c: avformat/jpegxl_anim_dec: use new animated JPEG XL codec ID
thilo has quit [Ping timeout: 246 seconds]
<michaelni>
BBB, i try to backport my bugfixes but i have never tried to backport everyones bugfixes. So unless we find someone that does that, everyone has to backport their bugfixes which they want backported
thilo has joined #ffmpeg-devel
<michaelni>
Traneptora, sleep? what is that ?
System_Error has quit [Remote host closed the connection]
rvalue has quit [Read error: Connection reset by peer]
rvalue has joined #ffmpeg-devel
<BBB>
michaelni: what are the currently maintained release branches?
<BBB>
and what is the process? is cherry-pick 060464105b -3 enough? or does it need more information?
<beastd>
IMHO try the cherry pick (see if it runs through without problems, git will stop and explain if things don't fit). Check if all expected commits are on top of the branch now. then build and run fate. if you think all is ok try to push the branch. continue with the next release branch.
<BBB>
I can't run fate, the changes are on arm
<BBB>
I don't have arm
<beastd>
it's been a while since i backported stuff for ffmpeg, but IIRC that's how i did it
<michaelni>
BBB, maintained release branches are the ones on https://ffmpeg.org/download.html, though when backporting has been done to release n and m then anything between is probably "free" and no work
<michaelni>
also please add -x to the cherry pick so the commit hash of the source commit is in the commit message (this makes tracking things easer)
<BBB>
git cherry-pick -x $hash -3
<michaelni>
yes
<BBB>
ok
<beastd>
yeah, -x is good for this picks. it's not default because it often makes no sense. but especially when backporting it's good because you are picking from a stable branch
<BBB>
2.8, 3.4, 4.2, 4.3, 4.4, 5.1, 6.1, 7.0, 7.1
<beastd>
BBB: ok i can give it a run on aarch64 after it's pushed. think i could also run on arm but that will be a little more involved.
<BBB>
oh wow it applies to 3.4 cleanly
<BBB>
(2.8 did not)
<michaelni>
if you do a lot of testing then seting up qemu+cross build is not hard, ubuntu at least has all packages one needs for several architectures like arm and mips
<kurosu>
Sounds like that should be automated, but I imagine fate+samples are only valid for master and not versioned, so no easy to fully validate a branch ?
<kurosu>
*not easy
<kurosu>
(branch or tag)
<beastd>
kurosu: iirc samples shold not be changed in fate collection. so it should work also for older fate.
<michaelni>
fate works with old checkouts
<kurosu>
ah right, it's the output that needs updating, and that's under the regular git
<beastd>
only add new samples, never change or remove old ones (at some point we could consider deleting old samples, but as long as it's not needed i would not)
<BBB>
man, sometimes shell scripting feels like cheating
cone-369 has joined #ffmpeg-devel
<cone-369>
ffmpeg Janne Grunau release/3.4:180f8216cdb6: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/3.4:80ef3328b559: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/3.4:970ca1e0f259: vp9: recon: Use emulated edge to prevent buffer overflows
<cone-369>
ffmpeg Janne Grunau release/4.2:bfed437be84e: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/4.2:a342536d6b1b: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/4.2:655b6f887788: vp9: recon: Use emulated edge to prevent buffer overflows
<kurosu>
So it would be nice that whatever populates fate.ffmpeg.org can also be run on tags. But maybe it can already, or it's worthy of some sponsorship to make the needed changes
<cone-369>
ffmpeg Janne Grunau release/4.3:69107495c5e4: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/4.3:131bd9436c2a: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/4.3:6cd0bdf3b064: vp9: recon: Use emulated edge to prevent buffer overflows
<cone-369>
ffmpeg Janne Grunau release/4.4:d3b9fd51b82d: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/4.4:fe955a7d779f: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<BBB>
beastd: ok pushed, please test if you can on one-or-some of these branches
<cone-369>
ffmpeg Janne Grunau release/4.4:7b0213a111db: vp9: recon: Use emulated edge to prevent buffer overflows
<cone-369>
ffmpeg Janne Grunau release/5.1:0dac8251f741: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/5.1:3562311c3027: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/5.1:14afc43c27d9: vp9: recon: Use emulated edge to prevent buffer overflows
<cone-369>
ffmpeg Janne Grunau release/6.1:7f0d4aa61c55: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/6.1:6a2b9d4c29be: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/6.1:2a29fe87c42a: vp9: recon: Use emulated edge to prevent buffer overflows
<cone-369>
ffmpeg Janne Grunau release/7.0:7f1e2090285e: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/7.0:0407cd499058: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/7.0:bb1a9c5932d3: vp9: recon: Use emulated edge to prevent buffer overflows
<cone-369>
ffmpeg Janne Grunau release/7.1:1a254c5354fb: aarch64: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/7.1:535a8262ccf6: arm: vp9mc: Load only 12 pixels in the 4 pixel wide horizontal filter
<cone-369>
ffmpeg Janne Grunau release/7.1:7d1532f75297: vp9: recon: Use emulated edge to prevent buffer overflows
<BBB>
beastd: (and thanks)
<BBB>
wbs: backport done
<beastd>
kurosu: agree we need more automation for this. would definitely give it a try when we setup a git forge for ffmpeg. think it would be good to run fate for topic branches (similar on how it works with patchwork) on a wide subset of platforms/architectures. would of course need suitably isolated fate clients. not sure how much it is a problems with current fate clients setups.
<beastd>
BBB: ok, will give it a try and report back
<kurosu>
Several years ago, I tried asking Intel to provide nodes to perform any hw test. I'm a bit baffled by how much they invest in adding things to ffmpeg, only for this to not be testable by other devs
<kurosu>
(maybe I have to mention 2 other brands, such AMD and Nvidia, to not be taxed of advertisement)
<beastd>
kurosu: yeah, think it would be really cool if all $big_corp would not only donate code but also hw to run it long term
<beastd>
this will also need probably some thought in regards to extending fate. don't think we test much more than our internal stuff and os integration, but i could be wrong. would be good to also do some more integration testing with 3rd party stuff
HarshK23 has joined #ffmpeg-devel
<beastd>
does someone remember GCC14 fixes for ffmpeg? would maybe be good to backport them to older branches.
<beastd>
Kind of OT, but maybe someone knows by chance (wbs ?). Do most compilers guarantee to only produce output files (objects, libs, executables etc.) only either complete or not at all? (I would expect so, but never checked.)
<cone-369>
ffmpeg Leandro Santiago master:9d9ac8e2ca6b: avfilter/vf_dnn_detect: fix loading anchors when labels file is set
IndecisiveTurtle has joined #ffmpeg-devel
<beastd>
BBB: 3.4, 4.2, 4.3, 4.4, 5.1, 6.1, 7.0, 7.1 <-- all tested on aarch64 without problems. so if the original changes are correct the backports should probably be fine as well
<beastd>
had to make some tradeoffs regarding a bunch of fate tests because of zlib-ng vs fate. but i checked and it seems it's only the usual culprits with embedded png. could not see how it could be affected by vp9 aarch64 changes.
<beastd>
also tested that on current master works fine with the help of my fate .alt file patch set.
<BBB>
cool, tnx
ngaullier has joined #ffmpeg-devel
<fflogger>
[editedticket] quinkblack: Ticket #11363 ([avcodec] [Android] MediaCodec decoders/encoders do not work on Pixel 8 Pro (No output buffer available)) updated https://trac.ffmpeg.org/ticket/11363#comment:5
jalsdflkadfadsf has quit [Ping timeout: 240 seconds]
<beastd>
wbs: today i have backported fate-list-failing locally. as expected no conflicts. you are fine with backporting to all releases listed on our download page?
___nick___ has quit [Ping timeout: 260 seconds]
<wbs>
beastd: I'm a little undecided on that; it's not a pure bugfix, but it also quite safe and harmless... I would raise the question on the mailing list and ask if there are any objections to backporting it
<beastd>
wbs: good point. i just find it useful on releases as well. e.g. for doing backports. will send a mail to the list...
witchymary has quit [Remote host closed the connection]
witchymary has joined #ffmpeg-devel
witchymary has quit [Remote host closed the connection]