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

Search for libsystemd instead of libsystemd-journal #34

Merged
merged 1 commit into from
Jan 4, 2017

Conversation

bigon
Copy link
Contributor

@bigon bigon commented Dec 30, 2016

libsystemd-journal has been merged in libsystemd at version 209
(2014-02-20), make the configure search for that "new" library.

Closes: #33

@ppwaskie
Copy link
Contributor

ppwaskie commented Jan 1, 2017

While this looks ok overall, have you tried this on an older system with an older version of systemd? I'm more concerned about systems that want to upgrade irqbalance, but keep other things like systemd at an older level.

@bigon
Copy link
Contributor Author

bigon commented Jan 2, 2017

It will indeed not work with the older versions of systemd. I'll see if I can add the detection of the older library as a fallback

@ppwaskie
Copy link
Contributor

ppwaskie commented Jan 3, 2017

Thanks. I would be much more comfortable merging it if you can add that please.

-PJ

libsystemd-journal has been merged in libsystemd at version 209
(2014-02-20), make the configure search for that "new" library first and
then fallback to the old one.

Closes: Irqbalance#33
@bigon
Copy link
Contributor Author

bigon commented Jan 3, 2017

OK, I've update my commit and force pushed it

@ppwaskie
Copy link
Contributor

ppwaskie commented Jan 4, 2017

Looks good, thanks Laurent!

@ppwaskie ppwaskie merged commit 1b999ec into Irqbalance:master Jan 4, 2017
@bigon bigon deleted the libsystemd branch January 4, 2017 14:29
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

Successfully merging this pull request may close these issues.

2 participants