search

winfsp / cgofuse

Cross-platform FUSE library for Go - Works on Windows, macOS, Linux, FreeBSD, NetBSD, OpenBSD
https://winfsp.dev
MIT License
514 stars 82 forks source link

Update README file with instructions for Alpine Linux dependencies, b… #29

Closed andersjohansenange closed 4 years ago

andersjohansenange commented 5 years ago

…uilding in a container

billziss-gh commented 5 years ago

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 commented 5 years ago

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 commented 5 years ago

@andersjohansenange no problem :)