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