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

netlink library required but not checked in configure step #28

Closed
neophiliac opened this issue May 19, 2020 · 5 comments
Closed

netlink library required but not checked in configure step #28

neophiliac opened this issue May 19, 2020 · 5 comments

Comments

@neophiliac
Copy link

After installing the missing libraries that configure pointed out, 'make' gave this error:

/usr/bin/ld: cannot find -lnl-genl-3

After installing libnl-genl-3-dev, the build completed.

I expected that after configure completed with no errors, the make should also complete with no errors.

@nhorman
Copy link
Owner

nhorman commented May 19, 2020

can you attach the config.log file that got generated when you ran configure? configure.ac has a PKG_CHECK_MODULES macro for libnl3 as just about the first operation in the configure script.

@neophiliac
Copy link
Author

config.log

It's checking for libnl-3.0, but the linker failed when looking for libnl-gen-3.

Here's the output of make after a 'make clean'.

make.log

"-lnl-3 -lnl-genl-3" always appear together.

@nhorman
Copy link
Owner

nhorman commented May 20, 2020

ah, I see the problem. When I added the configure script, libnl3 had a bit of a broken pkg-config setup, in that it had a pkg-config for libnl, but not libnl-genl. That seems to have been corrected now.

I've pushed this branch:
https://github.com/nhorman/dropwatch/tree/pkg-config-fix

with a correction to the configure script. Can you please attempt to build that branch and confirm that the issue is resolved? Not, depending on your version of ubuntu, you may need to install the libnl-genl-3.0-dev pacakge for it to work (configure will fail if its not installed)

@neophiliac
Copy link
Author

That works great. No errors or warnings.

Thanks!

@nhorman
Copy link
Owner

nhorman commented May 20, 2020

awesome, I'll merge it today, 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

2 participants