Skip to content
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

upgrade fuse to fuser for Mac support #65

Closed
wants to merge 6 commits into from

Conversation

dev-jwel
Copy link

Upgrade to support Mac.
fuse package returns error even osxfuse has installed for Mac.

Package osxfuse was not found in the pkg-config search path.
Perhaps you should add the directory containing `osxfuse.pc'
to the PKG_CONFIG_PATH environment variable
No package 'osxfuse' found
Package osxfuse was not found in the pkg-config search path.
Perhaps you should add the directory containing `osxfuse.pc'
to the PKG_CONFIG_PATH environment variable
No package 'osxfuse' found
', /Users/jwel/.cargo/registry/src/github.com-1ecc6299db9ec823/fuse-0.3.1/build.rs:10:76
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

However, it is works well for fuser which is fork of the fuse.

@gaul gaul force-pushed the master branch 5 times, most recently from 154ee45 to f53f3ee Compare February 26, 2022 08:50
@gaul
Copy link
Collaborator

gaul commented Feb 26, 2022

Could you rebase and push again? I believe I fixed the CI.

@dev-jwel
Copy link
Author

I merged it to run CI.

@gaul gaul requested a review from kahing February 26, 2022 13:24
@gaul
Copy link
Collaborator

gaul commented Jun 12, 2022

This still needs rebasing and please get rid of the merge commits. I'm not too sure what this is doing so it might make sense to send individual PRs.

@dev-jwel
Copy link
Author

#68 follows this. CI will fail if divide the PR as fuse-rs v0.3 use Timespec but fuse-rs v0.4 and fuser does not. I described the change in the commit instead. Feel free to ask me if some change is strange.

@dev-jwel dev-jwel closed this Jun 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants