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

[libcommhistory] Remove unneeded and incorrect service name references. JB#62154 #14

Merged
merged 1 commit into from
Jun 4, 2024

Conversation

pvuorela
Copy link
Contributor

Commhistoryd is having its own service name which is not this. There should be nothing using it generally. A few places were mistaking service and interface names, though the actual strings are the same.

Spotted while checking @dcaliste's dbus PRs.

…s. JB#62154

Commhistoryd is having its own service name which is not this.
There should be nothing using it generally. A few places were mistaking
service and interface names, though the actual strings are the same.
@pvuorela pvuorela requested a review from mlehtima May 30, 2024 20:02
@dcaliste
Copy link
Contributor

That is a sensible change. There is no service here provided by a central object.

@pvuorela pvuorela requested a review from spiiroin June 1, 2024 10:04
@pvuorela pvuorela merged commit 3cc96dd into master Jun 4, 2024
@mlehtima mlehtima deleted the remove_service_name branch June 10, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants