[FFmpeg-devel] Bump major version of swresample

[Zhao Zhili]

> 在 2023年2月28日，09:43，Wang Bin <wbsecg1 at gmail.com> 写道：
> 
> James Almer <jamrial at gmail.com> 于2023年2月27日周一 20:00写道：
> 
>>> On 2/27/2023 7:03 AM, Michael Niedermayer wrote:
>>> essOn Sat, Feb 25, 2023 at 12:03:02AM +0800, Wang Bin wrote:
>>>> 
>>> 
>>>>  version_major.h |    2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>> a87056c2fe65d68b2cf5d1de06be28ea40c69b73
>> 0001-Bump-major-version-of-swresample.patch
>>>> From e3e6a3833f2fba743ee9c05962e804e9e570dd75 Mon Sep 17 00:00:00 2001
>>>> From: wang-bin <wbsecg1 at gmail.com>
>>>> Date: Fri, 24 Feb 2023 23:54:51 +0800
>>>> Subject: [PATCH] Bump major version of swresample
>>>> 
>>>> ---
>>>>  libswresample/version_major.h | 2 +-
>>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>> 
>>>> diff --git a/libswresample/version_major.h
>> b/libswresample/version_major.h
>>>> index 7f265c2073..dd13f2bbe3 100644
>>>> --- a/libswresample/version_major.h
>>>> +++ b/libswresample/version_major.h
>>>> @@ -26,6 +26,6 @@
>>>>   * Libswresample version macros
>>>>   */
>>>> 
>>>> -#define LIBSWRESAMPLE_VERSION_MAJOR   4
>>>> +#define LIBSWRESAMPLE_VERSION_MAJOR   5
>>> 
>>> No oppinion if this should be changed now before 6.0 or not
>>> but if its done it should be done on master and release/6.0 at the same
>> time
>>> and LIBSWRESAMPLE_VERSION_MINOR needs to be reset too while
>>> LIBSWRESAMPLE_VERSION_MINOR needs to be +1 on master compared to
>> release/6.0
>>> 
>>> oppinon from others is welcome here. Iam not a user of the releases so
>> its
>>> hard for me to really guess which way is better. Its a little messy to
>>> change now
>>> 
>>> thx
>> 
>> I don't think it's a good idea to do it now. No API was removed from it
>> so leaving the major as is should be fine.
>> 
> 
> Currently no api change and even no abi change. But AVFrame is used in the
> public api swr_convert_frame, AVFrame abi changes may break swresample
> binary compatibility without swresample code change.

I don’t think that’s how ABI and so version works.

> All other modules
> except postproc and avutil public apis also depend on structs from another
> module. So it's better to bump major version of all modules.
> 
> Regards
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org wit