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

Fixed to support libalpm v13. #150

Merged
merged 1 commit into from
Jun 2, 2021
Merged

Fixed to support libalpm v13. #150

merged 1 commit into from
Jun 2, 2021

Conversation

egormanga
Copy link
Contributor

No description provided.

@f2404 f2404 merged commit 767ada2 into archlinuxfr:master Jun 2, 2021
@f2404
Copy link
Collaborator

f2404 commented Jun 2, 2021

Thanks!

@egormanga egormanga deleted the fix-libalpm13 branch June 2, 2021 15:30
@Morganamilo
Copy link

Is there a reason architecture needs to be set in the first place? AFAIK it's only used for transactions which package query won't be doing.

@egormanga
Copy link
Contributor Author

@Morganamilo Honestly this fork was the first time when I saw the source of package-query, so it was more of a quick fix for yaourt not to crash. Let us not bump this already-merged PR, feel free to open a new issue regarding this (probably just a code cleanup, as setting the architecture should not make any matter if you're correct).

By the way, in process of updating I tried to comment out these lines and it worked out, so you must be right. Anyway, this PR is not a great place to discuss that.

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.

3 participants