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

libudev.pc for libudev-compat #65

Open
fbt opened this issue Sep 5, 2015 · 4 comments
Open

libudev.pc for libudev-compat #65

fbt opened this issue Sep 5, 2015 · 4 comments

Comments

@fbt
Copy link
Contributor

fbt commented Sep 5, 2015

libudev-compat should probably provide a .pc file for pkg-config, otherwise configure scripts can't find libudev.
I've tried to just put in the one from eudev and it works perfectly. Should I create a pull request with it? There's nothing eudev-specific in it. It even has systemd's standard “this file is part of systemd” header (although that should probably be removed without any licensing considerations, it's a config, not code).

Low priority probably.

UPD: also a libudev.so -> libudev.so.$ver link should exist.

@suedi
Copy link
Contributor

suedi commented Sep 5, 2015

regarding libudev.so

I have with original libudev in arch

libudev.so -> libudev.so.1.6.4
libudev.so.1 -> libudev.so.1.6.4
libudev.so.1.6.4

is there a correct way to do this?

@fbt
Copy link
Contributor Author

fbt commented Sep 5, 2015

That's the correct way, yeah, edited my comment.

@jcnelson
Copy link
Owner

jcnelson commented Sep 7, 2015

Hey, sorry for the late reply.

@fbt libudev-compat will need a .pc file at some point. It's probably best to derive it from libudev or libeudev, and track upstream. If you want to send a PR for it, I'd be happy to merge it :). Otherwise, I'll do to it sometime later this week (once eventfs is working with libudev-compat).

@jcnelson
Copy link
Owner

Added in e2bbff3

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

No branches or pull requests

3 participants