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

Build fails when glibc 2.17 is used #40

Closed
TZ86 opened this issue Jan 8, 2013 · 5 comments
Closed

Build fails when glibc 2.17 is used #40

TZ86 opened this issue Jan 8, 2013 · 5 comments

Comments

@TZ86
Copy link

TZ86 commented Jan 8, 2013

When I try build eudev with latest glibc, it fails on udevd linking:

  CCLD     udevd
./.libs/libudev-core.a(libudev_core_la-sd-daemon.o): In function `sd_is_mq':
sd-daemon.c:(.text+0x94a): undefined reference to `mq_getattr'
collect2: error: ld returned 1 exit status

I fixes it by adding -lrt flag to libudev_core_la target: http://pastebin.com/hLfGSy9i

@lu-zero
Copy link
Contributor

lu-zero commented Jan 8, 2013

Flameeyes already pointed why in his review, if it uses posix message
queues configure should check for them.

On Tue, Jan 8, 2013 at 10:28 AM, Alexey D. notifications@github.com wrote:

When I try build eudev with latest glibc, it fails on udevd linking:

CCLD udevd
./.libs/libudev-core.a(libudev_core_la-sd-daemon.o): In function sd_is_mq': sd-daemon.c:(.text+0x94a): undefined reference tomq_getattr'
collect2: error: ld returned 1 exit status

I fixes it by adding -lrt flag to libudev_core_la target: http://pastebin.com/hLfGSy9i


Reply to this email directly or view it on GitHub.

@blueness
Copy link
Contributor

blueness commented Jan 8, 2013

lu-zero: yes and no. If we needed mq_* we would have to check for it, but that code is now dead. It appears in sd_is_mq() in src/udev/sd-daemon.c which is no longer used. Commenting it out and the prototype in the header file leaves eudev building okay. Nor is the symbol exported, so we should be able to just comment it out.

Any comments before I do the hacking!

@blueness
Copy link
Contributor

blueness commented Jan 8, 2013

Actually AxS has convinced me pretty much all the code in sd-daemon.c is dead. We're just going to remove it all and test.

@blueness
Copy link
Contributor

blueness commented Jan 9, 2013

@ reporter, can you please test again. This should be fixed in master/HEAD.

Thanks for the report!

@TZ86
Copy link
Author

TZ86 commented Jan 9, 2013

blueness: Yes, it works.

Thanks!

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

3 participants