-
Notifications
You must be signed in to change notification settings - Fork 494
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
Add AppVeyor CI for Cygwin #138
Conversation
Thanks! Will this do any good if I just commit it? I'd rather not burden myself with an AppVeyor account. Is there any way you could set things up? |
I think the only way would be for me to be a member of the libfuse organization or the sshfs project. Even then I am not really certain that I would have the necessary permissions to set it up. But I am happy to try if you want. |
Just sent you an invite. Please continue to check non-Cygwin-related changes check with me before comitting (i.e, send a regular pull request). |
@Nikratio thanks for the invite. I will look whether I can add AppVeyor support as collaborator tomorrow (it is 12:30am here). And rest assured that I will always submit any future changes as PR’s for you to review and not direct commits. |
@Nikratio I have been unable to add libfuse/sshfs as a project in my AppVeyor account. I am working with AppVeyor support to resolve this. For details please see appveyor/ci#2548. |
According to the AppVeyor folks I need libfuse access in order to setup CI for sshfs. So we have the following options:
I am happy to proceed as you see fit. |
Given the effort and care that you have invested in this, I'd be very happy to have you on-board :-). |
@Nikratio sorry for the late response and thanks for adding me. For some reason I still do not seem to have the right permissions to do this. So I have asked for the necessary privileges via GitHub Settings. You should see messages regarding "AppVeyor" and "AppVeyor CI". Thanks. |
Just granted them. |
@Nikratio thank you. I have created an AppVeyor project for https://ci.appveyor.com/project/billziss-gh/sshfs The first build failed, because the project is not setup to build without this PR. See the image below: I believe if you accept this PR, AppVeyor builds will work normally. |
Thank you for merging it in. AppVeyor builds now work correctly: |
This PR adds AppVeyor CI support to SSHFS.
The included scripts build SSHFS against both Cygwin 64-bit and Cygwin 32-bit. This should ensure that Cygwin remains a valid target for SSHFS going forward.
To use this you will need a free AppVeyor account.
EDIT: Here is what the AppVeyor run looks like:
https://ci.appveyor.com/project/billziss-gh/sshfs/build/18