search

illiliti / libudev-zero

Daemonless replacement for libudev
ISC License
201 stars 22 forks source link

libudev.sym: provide all currently known libudev symbols versions #40

Closed phkrl closed 3 years ago

phkrl commented 3 years ago

some libs expect particular version >183

Rio6 commented 3 years ago

These stub symbols probably don't need to depend on each other. I tested with only {}; on all of them and they still work as expected.

Rio6 commented 3 years ago

I was building arcan recently, and it required versioning on each symbols rather than the whole library. I had to replace libudev.sym with the one from systemd repo to make it work (it complained with /usr/bin/ld: /usr/lib/gcc/x86_64-pc-linux-gnu/11.1.0/../../../../lib/libusb-1.0.so: undefined reference to 'udev_enumerate_add_match_property@LIBUDEV_183' with stub symbol versions).

This might be something to consider too when adding symbol versioning compatibility to libudev-zero.

phkrl commented 3 years ago

@Rio6 could you check if adding *; to LIBUDEV_183 definition fixes the issue?

Rio6 commented 3 years ago

Yep that works!

illiliti commented 3 years ago

https://github.com/illiliti/libudev-zero/issues/38