Re: [fuse-emulator-devel] [fuse-emulator-cvs] [libspectrum] branch master updated: Updates for a "1.5.0" release

[Fredrick M. <fr...@sp...>]

Hi Sergio,

> On 12 Feb 2021, at 17:41, Sergio Baldoví <ser...@gm...> wrote:
> 
> On 11/2/21 12:05, fredm--- via fuse-emulator-cvs wrote:
>> ## * Always increase the revision value.
>> ## * Increase the age value only if the changes made to the ABI are backward
>> ##   compatible.
>> -libspectrum_la_LDFLAGS = -version-info 16:14:8 -no-undefined @WINDRES_LDFLAGS@
>> +libspectrum_la_LDFLAGS = -version-info 17:15:8 -no-undefined @WINDRES_LDFLAGS@
> 
> Do you think a soname dump is required?
> 
> I was expecting an increase of the age value too.

Hmm I did do an api compatibility check to decide on the appropriate values (see attached) and it reported that both source and binary compatibility has been affected by the changes.

The source isn’t 100% compatible due to the removal of libspectrum_snap_alloc_internal() which isn’t supposed to be part of the public API but is still being flagged.

Binary compatibility is affected by the changes to the libspectrum_snap structure.so its a deliberate choice to not bump the age value.

Have I made a mistake?

Fred

[1] Compatibility report, X is 1.4.4 and Y is the proposed 1.5.0