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
uhub: re-enable sqlite plugin / fix aarch64 build #48021
uhub: re-enable sqlite plugin / fix aarch64 build #48021
Conversation
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: uhub Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: uhub Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: uhub Partial log (click to expand)
|
the project changed the code repository to github
@GrahamcOfBorg build uhub |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: uhub Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: uhub Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: uhub Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: uhub Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: uhub Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: uhub Partial log (click to expand)
|
@makefu Any reference to an upstream issue where the patch you're adding is discussed? This would be helpful to be able to eventually remove it, I think :) |
5034d21
to
ae83f87
Compare
@Ekleog i've added a reference to the debian bug (open since 2014) |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: uhub Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: uhub Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: uhub Partial log (click to expand)
|
I've just opened janvidar/uhub#46 upstream based on the bug you linked (maybe the author never saw the debian bug), you may want to link to it instead? :) |
ae83f87
to
7e861a8
Compare
i've removed the patch file and instead pull it from the github PR i've opened |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: uhub Partial log (click to expand)
|
Unexpected error: command failed with exit code 1 on aarch64-linux (full log) Attempted: uhub Partial log (click to expand)
|
Unexpected error: command failed with exit code 1 on x86_64-linux (full log) Attempted: uhub Partial log (click to expand)
|
@makefu Looks like a problem with the patch hash now :/ (don't know if you read ofborg's emails) |
@Ekleog weird thing, let me try again. I do not receive ofborg mails for some reason, is there any button where i can enable it? |
7e861a8
to
d74ea28
Compare
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: uhub Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: uhub Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: uhub Partial log (click to expand)
|
@makefu I don't know, sorry, it's sent through the same mechanism as all github comments theoretically :/ maybe in your spam folder? Anyway, I think that's ready to be merged, thanks! :) |
ah you mean these mails :D yes, i get them but i was busy doing daily work |
anything holding this PR to be merged? |
Motivation for this change
This PR re-enables the sqlite authentication plugin for uhub. It was disabled because of a failed build, but this seems to have been solved.
Additionally it fixes the build of uhub for aarch64
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)