search

archlinuxfr / package-query

Query alpm database and AUR
69 stars 17 forks source link

Fixed to support libalpm v13. #150

Closed egormanga closed 3 years ago

f2404 commented 3 years ago

Thanks!

Morganamilo commented 3 years ago

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 commented 3 years ago

@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.