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 README file with instructions for Alpine Linux dependencies, b… #29

Closed
wants to merge 1 commit into from

Conversation

andersjohansenange
Copy link

…uilding in a container

@billziss-gh
Copy link
Collaborator

There is a build failure, but it is unrelated to your change. It appears to be a build failure of iozone, likely due to some change in the Travis macOS build environment.

(Iozone is not actually used by cgofuse or for its testing, it is simply built by the default testsuite Makefile. So this failure is entirely unimportant to cgofuse. Nevertheless I will fix this in a future change).

Overall the information in the PR is fine, but I think we should add the Alpine/Kubernetes information is a separate README (e.g. README-Alpine.md), rather than as a section in the main README.

@andersjohansenange
Copy link
Author

Sorry for dropping off the map a bit, I will get this set straight this week. Been super busy.. writing cgofuse FUSE drivers :)

@billziss-gh
Copy link
Collaborator

@andersjohansenange no problem :)

@billziss-gh billziss-gh closed this Jul 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants