Request for feedback on libcdio update (soname bump)
Gabriel F. T. Gomes
gabriel at debian.org
Fri Jan 31 15:40:20 GMT 2025
Dear Perl Team,
Upstream libcdio released a new version with ABI and API changes.
I uploaded the new version to experimental which provides the following
packages (among others):
libiso9660-12
libiso9660-dev
libdevice-cdio-perl depends on them, so I also built it against these
dependencies. The build is fine; however, I manually reviewed the uses
of the new ABI within this perl package but I could not determine if
they are fine.
Could you help me review it?
The new ABI added a new struct member/field to the following type,
which is used in libdevice-cdio-perl:
iso9660_stat_t
For instance, in libdevice-cdio-perl-2.0.0/perliso9660_wrap.c, the
function XS(_wrap_get_posix_filemode) declares an object of type
iso9660_stat_t, and assigns data to it. The data seems to be generated
by the SWIG_ConvertPtr function (or maybe the storage for it is
allocated within this function?).
Since the new ABI changed the size of the struct, we could have a
silent problem here.
Are you familiar with this SWIG thing?
Do the tests defined in autopkgtest test this specifically?
Are there other tests you could run?
I would appreciate your help very much! :)
Cheers,
Gabriel
PS: I'm planning a transition for libcdio[1], but I'll only upload to
unstable once I have confirmed with you that this is OK.
[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1094736
More information about the pkg-perl-maintainers
mailing list