Skip to content
This repository has been archived by the owner on Feb 2, 2019. It is now read-only.

Packagekit1.0 #3

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Packagekit1.0 #3

wants to merge 4 commits into from

Conversation

Thaodan
Copy link

@Thaodan Thaodan commented Feb 15, 2018

Updated to PackageKit 1.0 (or 0.9 for that matter).

For some reason pkgconfig returns the wrong include path so I added
the include path direct.
No longer create transactions ourself.
Instead call the daemon with the requested action.
@mentaljam
Copy link
Owner

Hi,
Storeman no longer uses PackageKit client libraries but direct d-bus calls to PackageKit daemon. PackageKit-Qt5 still is a build dependency because of using its headers to retrieve values of enumerations but no linking is required.

@Thaodan
Copy link
Author

Thaodan commented Feb 15, 2018

Yes I saw that to late, Is this better than using the clientlib or a workaround?

@mentaljam
Copy link
Owner

As for me it's better. I was not satisfied with using PackageKit-Qt5 so I decided to drop it.

@Thaodan
Copy link
Author

Thaodan commented Feb 15, 2018

Ok, although much was reworked.
And retrieving icons and desktopfiles was removed from packagekit in general.

@mentaljam
Copy link
Owner

I've tried to port Storeman to the new PackageKit some weeks ago and after that I finally made a decision to switch to direct d-bus calls. And it looks for me as a more clear solution.

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

Successfully merging this pull request may close these issues.

2 participants