<div dir="ltr"><div><div>Dominic - I see you have forked the Perl libcdio code. I have just updated it with something that provisionally works with <a href="http://rocky.github.io/libcdio-1.1.0rc1.tar.bz2">http://rocky.github.io/libcdio-1.1.0rc1.tar.bz2</a>  the current 1.1.0 release candidate.  That Perl code still needs fixups for (not recent) CD-Text API changes. <br><br>In the last libcdio release, I removed the rather old CD-Text API but we were stull using that in the Perl bindings.<br><br></div>In general what has to happen is 1.1.0 needs to get out there. Then another Device::Cdio will be released which uses that and will depend on 1.1.0 (not 1.0.0). <br><br></div>Sorry everyone for all of the hassle. With luck, perhaps this will be the last set of libcdio stuff I do.<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 6, 2017 at 7:59 AM, Dominic Hargreaves <span dir="ltr"><<a href="mailto:dom@earth.li" target="_blank">dom@earth.li</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Dec 06, 2017 at 11:46:31AM +0100, Matthias Klose wrote:<br>
> Package: src:libdevice-cdio-perl<br>
> Version: 0.4.0-2<br>
> Severity: serious<br>
> Tags: sid buster patch<br>
><br>
> One driver is gone upstream.<br>
><br>
> patch at<br>
> <a href="http://launchpadlibrarian.net/348274120/libdevice-cdio-perl_0.4.0-2build1_0.4.0-2ubuntu1.diff.gz" rel="noreferrer" target="_blank">http://launchpadlibrarian.net/<wbr>348274120/libdevice-cdio-perl_<wbr>0.4.0-2build1_0.4.0-2ubuntu1.<wbr>diff.gz</a><br>
<br>
</div></div>Hi Matthew,<br>
<br>
I tried applying this patch (both to the Debian package and to upstream<br>
git ready for forwarding) but I get the following build failures on sid:<br>
<br>
./perlcdio_wrap.c: In function 'get_cdtext':<br>
./perlcdio_wrap.c:2439:14: error: too many arguments to function 'cdio_get_cdtex<br>
t'<br>
     cdtext = cdio_get_cdtext (p_cdio, (track_t) track);<br>
              ^~~~~~~~~~~~~~~<br>
In file included from /usr/include/cdio/cdio.h:62:0,<br>
                 from ./perlcdio_wrap.c:1562:<br>
/usr/include/cdio/disc.h:77:<wbr>13: note: declared here<br>
   cdtext_t *cdio_get_cdtext (CdIo_t *p_cdio);<br>
             ^~~~~~~~~~~~~~~<br>
./perlcdio_wrap.c:2448:29: warning: passing argument 1 of 'cdtext_get_const' makes pointer from integer without a cast [-Wint-conversion]<br>
         if(cdtext_get_const(num,<wbr>cdtext)) {<br>
                             ^~~<br>
In file included from /usr/include/cdio/cdio.h:59:0,<br>
                 from ./perlcdio_wrap.c:1562:<br>
/usr/include/cdio/cdtext.h:<wbr>262:13: note: expected 'const cdtext_t * {aka const struct cdtext_s *}' but argument is of type 'int'<br>
 const char *cdtext_get_const (const cdtext_t *p_cdtext, cdtext_field_t field,<br>
             ^~~~~~~~~~~~~~~~<br>
./perlcdio_wrap.c:2448:33: error: incompatible type for argument 2 of 'cdtext_get_const'<br>
         if(cdtext_get_const(num,<wbr>cdtext)) {<br>
                                 ^~~~~~<br>
<br>
[...]<br>
<br>
(full log attached).<br>
<br>
Do you have any suggestions about how to resolve this?<br>
<br>
Cheers,<br>
Dominic.<br>
</blockquote></div><br></div>