From: Andreas Schneider <asn@cryptomilk.org>
To: FFmpeg development discussions and patches
<ffmpeg-devel@ffmpeg.org>, Jan Engelhardt <jengelh@inai.de>,
Andreas Schneider <asn@cryptomilk.org>
Subject: Re: [FFmpeg-devel] FFmpeg 5.0
Date: Mon, 31 Oct 2022 18:51:23 +0100
Message-ID: <12093289.O9o76ZdvQC@magrathea> (raw)
In-Reply-To: <166722140107.12287.4175132527138383610@lain.khirnov.net>
On Monday, 31 October 2022 14:03:21 CET Anton Khirnov wrote:
> Quoting Neal Gompa (2022-10-30 22:04:42)
>
> > On Sun, Oct 30, 2022 at 4:10 PM Michael Niedermayer
> >
> > <michael@niedermayer.cc> wrote:
> > > On Sat, Oct 29, 2022 at 02:29:56PM -0400, Neal Gompa wrote:
> > > > On Fri, Oct 28, 2022 at 2:23 PM Michael Niedermayer
> > > >
> > > > <michael@niedermayer.cc> wrote:
> > > > > Hi
> > > > >
> > > > > According to our
> > > > > https://trac.ffmpeg.org/wiki/Downstreams
> > > > >
> > > > > Noone and nothing is using 5.0
> > > > > should i make another release of 5.0 ?
> > > > > should i move 5.0 to olddownloads ?
> > > > >
> > > > > does anyone use it ? plan to use it or know of someone using it ?
> > > >
> > > > Fedora 36 still uses FFmpeg 5.0 as I discovered there was an ABI break
> > > > that made upgrading to FFmpeg 5.1 not possible for F36. FFmpeg 5.1 is
> > > > used for Fedora 37, though.
> > > >
> > > > This had apparently been also discovered by openSUSE some time ago:
> > > > https://build.opensuse.org/package/view_file/multimedia:libs/ffmpeg-5/
> > > > work-around-abi-break.patch?expand=1> >
> > > You can replace 5.0 by 5.1 but not 5.1 by 5.0, The compatibility is only
> > > in one way.
> > > Iam assuming here you talk about the addition of functions and there is
> > > not some other issue iam not aware of.
> >
> > My understanding is that when using symbol versions, modifying the
> > symbol table creates a breakage on its own.
>
> Do you have some authoritative source for this claim? So far all the
> arguments I've seen were along the lines of "because I say so".
>
> > > > Do we have ABI testing in place for submitted patches? I haven't seen
> > > > any evidence of CI testing of patches submitted to the mailing list,
> > > > but maybe I'm looking in the wrong place? If there is, maybe we can
> > > > consider adding some kind of ABI testing for release branches, using
> > > > tools like libabigail[1] with abidiff[2]?
> > > >
> > > > [1]: https://sourceware.org/libabigail/
> > > > [2]: https://www.mankier.com/1/abidiff
> > >
> > > iam not sure there is agreement between you and others of what is a ABI
> > > break so the tool maybe will not help.
> > >
> > > I have generaly done testing with replacing old libraries by new when
> > > doing
> > > releases. But for me a ABI break is if replacing a library by another
> > > breaks some binary that is not rebuild and linked to the new lib.
> > >
> > > More testing is always good and welcome of course.
> >
> > Yeah, I think that qualifying how ABI is validated in a reproducible
> > way would be useful. The abigail tooling can help here, In Fedora,
> > every update runs through abigail validation too. I vaguely recall
> > that it warned me when I did the update in Rawhide, which is how I wound
> > up talking to Jan in openSUSE and RPM Fusion maintainers, who both
> > didn't refresh FFmpeg on stable branches with 5.0 to 5.1.
> >
> > It's also entirely possible that I was *too* cautious, and I'm okay
> > with having a conversation that leads me to do differently in the future.
>
> Having automated ABI testing would be definitely a good idea, not all
> developers understand how ABI compatibility works and there have been
> breakages (according to our definition) recently.
>
> I also have https://github.com/lvc/abi-compliance-checker/ on my
> to-look-at list, but as always time is lacking.
At libssh we use https://github.com/ansasaki/abimap for symbol versioning.
Ask if master could be added to ABI Laboratory:
https://abi-laboratory.pro/index.php?view=timeline&l=ffmpeg
Just my 2c.
Andreas
--
Andreas Schneider asn@cryptomilk.org
GPG-ID: 8DFF53E18F2ABC8D8F3C92237EE0FC4DCC014E3D
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2022-10-31 17:51 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-10-28 18:23 Michael Niedermayer
2022-10-28 19:10 ` Timo Rothenpieler
2022-10-28 20:17 ` AV Preservation by reto.ch (lists)
2022-10-29 18:29 ` Neal Gompa
2022-10-29 19:42 ` Timo Rothenpieler
2022-10-30 20:10 ` Michael Niedermayer
2022-10-30 21:04 ` Neal Gompa
2022-10-31 13:03 ` Anton Khirnov
2022-10-31 17:51 ` Andreas Schneider [this message]
2022-11-03 22:23 ` Michael Niedermayer
2022-10-30 20:16 ` Michael Niedermayer
2022-10-30 20:53 ` Neal Gompa
[not found] ` <ss9225p7-9rpo-r229-5914-4n555p66sos@vanv.qr>
2022-10-31 14:33 ` Michael Niedermayer
2022-11-03 22:28 ` Michael Niedermayer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=12093289.O9o76ZdvQC@magrathea \
--to=asn@cryptomilk.org \
--cc=ffmpeg-devel@ffmpeg.org \
--cc=jengelh@inai.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git