Skip to content

Commit

Permalink
update fuse readme
Browse files Browse the repository at this point in the history
  • Loading branch information
mmaxim committed Oct 1, 2024
1 parent 66d3bc4 commit 8d3f934
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 4 deletions.
2 changes: 1 addition & 1 deletion go/updater/keybase/platform_darwin.go
Original file line number Diff line number Diff line change
Expand Up @@ -331,7 +331,7 @@ func (c context) Apply(update updater.Update, options updater.UpdateOptions, tmp
case nil:
case *os.LinkError:
if err.Op == "rename" && err.Old == "/Applications/Keybase.app" {
c.log.Infof("The error was a problem renaming (moving) the app, let's trying installing the app via keybase install --components=app which has more privileges")
c.log.Infof("The error was a problem renaming (moving) the app, let's trying installing the app via keybase install --components=app which has more privileges: %s", err)

// Unzip and get source path
unzipPath, err := util.UnzipPath(localPath, c.log)
Expand Down
11 changes: 8 additions & 3 deletions osx/Fuse/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ oldest possible (compatible) kernel, to have a fewer kernel extensions built as
possible.

It's possible to build for many macOS versions but it requrues a patched Xcode.
So are targeting 11+ now. We'll build for macOS 11, and make a symlink of
So are targeting 11+ now. We'll build for macOS 11, and make a symlink of
macOS 12 All these are handled by the build script, but we'll need the
appropriate SDKs to build against.

Expand Down Expand Up @@ -49,7 +49,13 @@ which you can submit for PR.
This bundle is included in the KeybaseInstaller.app, so you'll need to build a new
installer, see [Building the Installer](/osx/Scripts/README.md).

Be sure to switch back to latest Xcode after you build.
Versioning can be tricky here, here are some tips from last time:

- Build against the newest macOS SDK
- Specify an old build target (like 12.3)
- Manually fixup Info.plist for the extension, since the Fuse build script puts the wrong values in for the minimum version and kext dependencies.
- Make sure the name in the Info.plist doesn't have a version number after it.
- `sudo kmutil log show` is a great way to see what is happening.

### Manual Install

Expand Down Expand Up @@ -78,4 +84,3 @@ After install if you are having problems loading the kext:
View kext status:

sudo kextstat -b com.github.kbfuse.filesystems.kbfuse

0 comments on commit 8d3f934

Please sign in to comment.