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

WIP: start porting alpm backend to adjust to breaking changes in alpm #407

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pontaoski
Copy link
Collaborator

No description provided.

@hughsie
Copy link
Collaborator

hughsie commented Jun 26, 2020

Are the alpm changes upstream already? Do you want to support both APIs with #ifdefs or just rely on the new version?

@pontaoski
Copy link
Collaborator Author

Are the alpm changes upstream already?

Yes; PackageKit alpm backend fails to build from source with current libalpm master due to API changes.

Do you want to support both APIs with #ifdefs or just rely on the new version?

Given that Arch is rolling, I don't think it makes much sense to keep the old code path around.

Copy link
Collaborator

@hughsie hughsie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PackageKit alpm backend fails to build from source with current libalpm master due to API changes

Then most certainly makes sense.

@eworm-de
Copy link
Collaborator

The alpm header file do not contain version information, so you can not detect at built time. It just builds or fails.

@eworm-de
Copy link
Collaborator

Now that pacman 6.0.0 has been released... Do you want to finish your work?

@progandy
Copy link

progandy commented Jun 11, 2021

The alpm header file do not contain version information, so you can not detect at built time. It just builds or fails.

If it is necessary to support the old and new library (e.g. some other distributions stay with old pacman), then it could be done with some meson configuration that converts the version number from pkg-config (alpm_dep.version()) to a c define. Either make it numeric like -DALPM_VER=XXXYYYZZZ or if(n)def with -DALPM_V13=1

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

4 participants