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
keybase-gui: fix kbfs mount path check #40080
Conversation
While this does look like it'll work in most scenarios it doesn't solve the overall problem in #31261 as it's now just looking at two options of hardcoded mountpoint. A better solution would be to have the wrapper (I may have used the wrong word, if so, sorry) check for where keybase is mounted and then use that. |
Success on x86_64-linux (full log) Attempted: keybase-gui Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: keybase-gui Partial log (click to expand)
|
Updated the check to look for |
Success on aarch64-linux (full log) Attempted: keybase-gui Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: keybase-gui Partial log (click to expand)
|
merge? |
@puffnfresh can you take a look and merge please? Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested with kbfs mounted on /mnt/kbfs, /keybase, ~/keybase, looks good.
(cherry picked from commit b3c3df4)
Motivation for this change
Fixes #31261
keybase-gui did not work out of the box with the default configuration of the keybase nixos service.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)