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

Allow setting systemd unit dirs separately, and get sane defaults from pkg-config #834

Closed
wants to merge 1 commit into from

Conversation

mgorny
Copy link
Contributor

@mgorny mgorny commented Feb 1, 2018

Currently blueman uses custom-made --with-systemdunitdir that sets top directory for systemd units. This doesn't work for split /usr systems because system units need to go into /lib/systemd while user units need to go into /usr/lib/systemd.

Replace the custom option with quasi-standard --with-systemdsystemunitdir and --with-systemduserunitdir that allow setting both necessary directories separately. Furthermore, default to querying pkg-config for the correct paths from systemd.pc.

Support setting a different system & user unit directories for systemd.
This is necessary since on split /usr systems, system units belong
in /lib, while user units belong in /usr/lib.
@mgorny
Copy link
Contributor Author

mgorny commented Feb 1, 2018

Hmm, I see that the PKG_CHECK_VAR macro is not present in older versions of pkg-config, and I don't care enough to implement it manually.

@infirit
Copy link
Contributor

infirit commented Feb 1, 2018

I "borrowed" some code (see 8ba8bcb) to add the pkgconfig function and merged your original commits.

thx

@infirit infirit closed this Feb 1, 2018
@mgorny
Copy link
Contributor Author

mgorny commented Feb 1, 2018

Thanks a lot!

cschramm referenced this pull request May 6, 2018
Unit files may also get installed to /etc or /lib. Instead pass working
configure flags to distcheck.
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.

None yet

2 participants