-
Notifications
You must be signed in to change notification settings - Fork 16
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
pacsift --repo misses packages compared to pacman -Sl #9
Comments
The shaman-git package is the problem; libarchive cannot parse the |
I don't really think it's libalpm's job to set the locale, feels like something that's up to the program using it. |
Correct, but it should be able to parse its databases regardless of the current locale. |
Good point. I guess libalpm would have to read it in as binary data instead of trying to parse it using the current locale, since currently anything that isn't colons or hyphens are fair game in pkgvers. Though I wonder if it would be reasonable to have a UTF-8 locale as a requirement, hmm... |
pacsift is definitely not the right place to fix this; any alpm frontend will miss those packages when running under the C locale, including pacman. libalpm needs to be updated to either be able to read the database regardless of the locale or at least throw an error if that's not possible. I tried to replicate the error via pacman's test suite and was unable to, so it may also have something to do with how the db was created. Please open a bug report against pacman, and, if you used something other than repo-add to build the db, maybe poke them too. |
pacman.conf: https://ptpb.pw/Z3oc
include file: https://ptpb.pw/htKj
% pacsift --repo=custom | wc -l
241
% pacman -Slq custom | wc -l
278
Missing packages: https://ptpb.pw/sKrS
pacman strace: https://ptpb.pw/3DMr
pacsift strace: https://ptpb.pw/CX9x
database: https://ptpb.pw/u3fb
The text was updated successfully, but these errors were encountered: