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

Install FUSE headers alongside rest #37

Closed
raymond-h opened this issue Aug 7, 2015 · 3 comments
Closed

Install FUSE headers alongside rest #37

raymond-h opened this issue Aug 7, 2015 · 3 comments

Comments

@raymond-h
Copy link

I made a fork of mafintosh/fuse-bindings to add support for Windows using the FUSE API of this repo. It's available at raymond-h/fuse-bindings.

It works, however at the moment, you need to both install Dokany for binary files, and clone the repo for the dokan-fuse headers.

It'd be great if the dokan-fuse headers were bundled with the install as well.

@Liryna
Copy link
Member

Liryna commented Aug 7, 2015

Hi,

Thats a very Great PR you have made 👍
I see no problem of adding the fuse header with the installer.

Do you need all headers of dokan_fuse/include/ ?
I will add them into \Dokan\DokanLibrary\include\fuse if you see no objection.

I will make the changes when your PR will be merged.

@raymond-h
Copy link
Author

Yeah, all headers would be great.

@Liryna
Copy link
Member

Liryna commented Aug 9, 2015

Well done for your merged. 🏆
I will add includes in the next release as you asked.

Liryna added a commit that referenced this issue Aug 17, 2015
@Liryna Liryna closed this as completed Aug 21, 2015
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

No branches or pull requests

2 participants