Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

10.2+2.0.0: cdio_cddap_track_audiop missing #21

Closed
kloczek opened this issue Sep 15, 2019 · 4 comments
Closed

10.2+2.0.0: cdio_cddap_track_audiop missing #21

kloczek opened this issue Sep 15, 2019 · 4 comments
Assignees

Comments

@kloczek
Copy link

kloczek commented Sep 15, 2019

I have problem with building gvs with libcdio-paranoia.
gvfs fails with:

gcc  -o daemon/gvfsd-cdda 'daemon/f77b12a@@gvfsd-cdda@exe/daemon-main.c.o' 'daemon/f77b12a@@gvfsd-cdda@exe/daemon-main-generic.c.o' 'daemon/f77b12a@@gvfsd-cdda@exe/gvfsbackendcdda.c.o' -Wl,--no-undefined -Wl,--as-needed -g -O2 -g -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fstack-protector-strong -grecord-gcc-switches -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -flto -Os -Wl,-z,relro -Wl,--as-needed -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -flto -fuse-linker-plugin -Wl,--start-group daemon/libgvfsdaemon.so common/libgvfscommon.so /usr/lib64/libgio-2.0.so /usr/lib64/libgobject-2.0.so /usr/lib64/libglib-2.0.so /usr/lib64/libgudev-1.0.so /usr/lib64/libcdio_paranoia.so /usr/lib64/libcdio.so -Wl,--end-group '-Wl,-rpath,$ORIGIN/:$ORIGIN/../common' -Wl,-rpath-link,/home/tkloczko/rpmbuild/BUILD/gvfs-1.42.0/x86_64-redhat-linux-gnu/daemon -Wl,-rpath-link,/home/tkloczko/rpmbuild/BUILD/gvfs-1.42.0/x86_64-redhat-linux-gnu/common
/usr/bin/ld: /tmp/gvfsd-cdda.9RGuZe.ltrans0.ltrans.o: undefined reference to symbol 'cdio_cddap_track_audiop'
/usr/bin/ld: /usr/lib64//libcdio_cdda.so.2: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.
error: Bad exit status from /var/tmp/rpm-tmp.rAf0WG (%build)

Just checked libcdio-paranoia tree and it is something strange about only that symbol

[tkloczko@barrel libcdio-paranoia-10.2+2.0.0]$ grep -r cdio_cddap_track_audiop
lib/cdda_interface/libcdio_cdda.sym:cdio_cddap_track_audiop
include/cdio/paranoia/cdda.h:extern int     cdio_cddap_track_audiop(cdrom_drive_t *d, track_t i_track);
include/cdio/paranoia/cdda.h:#define cdda_track_audiop       cdio_cddap_track_audiop

Looks like there is no actual code behind exactly that symbol.

May I ask for help or maybe it is a bug?

@kloczek
Copy link
Author

kloczek commented Sep 15, 2019

I have modified .pc files

[tkloczko@barrel libcdio-paranoia-10.2+2.0.0]$ cat libcdio_paranoia.pc 
prefix=/usr
exec_prefix=/usr
libdir=/usr/lib64
includedir=/usr/include

Name: libcdio_paranoia
Description: CD paranoia library from libcdio
Version: 10.2+2.0.0
Requires: libcdio
Libs: -L${libdir} -lcdio_paranoia
Cflags: -I${includedir}/cdio

[tkloczko@barrel libcdio-paranoia-10.2+2.0.0]$ cat libcdio_cdda.pc 
prefix=/usr
exec_prefix=/usr
libdir=/usr/lib64
includedir=/usr/include

Name: libcdio_cdda
Description: CD paranoia CD-DA library from libcdio
Version: 10.2+2.0.0
Requires: libcdio
Libs: -L${libdir} -lcdio_cdda
Cflags: -I${includedir}/cdio

However looks like libcdio_cdda uses symbols from libcdio_paranoia and vice versa.
With such loopy dependencies looks like have two libraries does not make to much sense.

@rocky rocky self-assigned this Sep 15, 2019
@rocky
Copy link
Collaborator

rocky commented Sep 15, 2019

This, the lack of track_audiop (and other booleans) is a bug. Should be an easy fix which I'll do when I am able.

@rocky
Copy link
Collaborator

rocky commented Sep 15, 2019

Please file one issue per issue. That makes it possible to track independent issues. Thanks.

@rocky rocky changed the title 10.2+2.0.0: libcdio-paranoia is not LTO ready 10.2+2.0.0: cdio_cddap_track_audiop missing Sep 15, 2019
@rocky rocky closed this as completed in 97d216a Sep 16, 2019
@rocky
Copy link
Collaborator

rocky commented Sep 16, 2019

I have modified .pc files

The .pc files are derivied files. I looked over the .pc.in files and don't see anything I want changed.

As for the loopiness, if you have specific suggestions feel free to make a PR for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants