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

Update instructions for new macOS guidance and drop source install for now #700

Closed

Conversation

dannycjones
Copy link

@dannycjones dannycjones commented May 25, 2022

In summary, this PR reworks the installation instructions in two ways.

Firstly, it drops the installation from source instructions, with the intention to add an updated version back once working. As #696 mentions, go install should be used instead of go get now. In terms of getting go install working, I'm seeing the same issues as #527, #664.

Secondly, it updates the instructions now that goofys is not available from Homebrew. This is due to a licensing change on the FUSE library used on macOS. Addresses #628.

Note, this does overlap with #618, #668, #687 but they do not address that goofys can no longer be installed via Homebrew (see Homebrew/homebrew-core#64491).

(closes #618, #668, #687)

Copy link
Contributor

@monthonk monthonk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested a little change but everything else looks good for now.

We really need a new release so we will have go install working.

README.md Outdated Show resolved Hide resolved
@monthonk
Copy link
Contributor

Looks good, thanks Danny.

@dannycjones
Copy link
Author

If #680 is merged before this, I think we can re-introduce the building from source options subject to testing.

@dannycjones
Copy link
Author

Closing. Change is outdated and Homebrew is no longer a supported method for install FUSE dependent connectors on macOS.

@dannycjones dannycjones closed this Feb 1, 2023
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