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

Windows Support? #86

Closed
ramkumarkb opened this issue Apr 12, 2021 · 9 comments
Closed

Windows Support? #86

ramkumarkb opened this issue Apr 12, 2021 · 9 comments

Comments

@ramkumarkb
Copy link

Hi,

First of all, thank you for this wonderful project.

I was wondering if there are any limitations to support Windows platform for unFTP?

@hannesdejager
Copy link
Collaborator

Hi there. Thank you! No reason except human resources :-). We’ve heard it compiles on windows, not sure if it runs. Do you want to try it and give feedback?

@ramkumarkb
Copy link
Author

@hannesdejager - Thank you for your reply. I am new to Rust but definitely would like to try and build-test this on Windows.

@hannesdejager
Copy link
Collaborator

Would be nice to get some feedback on this. I don’t have a windows machine or vm set up at the moment to test on windows, but would be nice if it works there too

@J-F-Liu
Copy link

J-F-Liu commented Jul 16, 2021

Build on Windows failed:

error[E0432]: unresolved import `tokio::signal::unix`
  --> src\main.rs:39:13
   |
39 |     signal::unix::{signal, SignalKind},
   |             ^^^^ could not find `unix` in `signal`

error[E0599]: no method named `poll` found for struct `Pin<&mut _>` in the current scope
   --> src\main.rs:422:20
    |
422 |       let sig_name = tokio::select! {
    |  ____________________^
423 | |         Some(_signal) = term_sig.recv() => {
424 | |             "SIG_TERM"
425 | |         },
...   |
428 | |         },
429 | |     };
    | |_____^ method not found in `Pin<&mut _>`
    |
    = note: `fut` is a function, perhaps you wish to call it
    = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0599]: no method named `poll` found for struct `Pin<_>` in the current scope
   --> src\main.rs:422:20
    |
422 |       let sig_name = tokio::select! {
    |  ____________________^
423 | |         Some(_signal) = term_sig.recv() => {
424 | |             "SIG_TERM"
425 | |         },
...   |
428 | |         },
429 | |     };
    | |_____^ method not found in `Pin<_>`
    |
    = note: `fut` is a function, perhaps you wish to call it
    = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to 3 previous errors

hannesdejager added a commit that referenced this issue Feb 20, 2023
This fixes building on Windows by putting the unix specific signal
handling behind a cfg attribute and adding Windows specific handling
too.

Adresses #86
hannesdejager added a commit that referenced this issue Feb 20, 2023
This fixes building on Windows by putting the unix specific signal
handling behind a cfg attribute and adding Windows specific handling
too.

Adresses #86
hannesdejager added a commit that referenced this issue Feb 21, 2023
This fixes building on Windows by putting the unix specific signal
handling behind a cfg attribute and adding Windows specific handling
too.

Adresses #86
@hannesdejager
Copy link
Collaborator

Hi @ramkumarkb , @J-F-Liu . I've fixed the windows builds and there is a build available here now:

https://github.com/bolcom/unFTP/actions/runs/4246819517

Do you perhaps want to test it out for us and give feedback?

@J-F-Liu
Copy link

J-F-Liu commented Feb 23, 2023

Did a simple test for download and upload files, works well, thanks.

@hannesdejager
Copy link
Collaborator

@J-F-Liu Great thank you for checking

@ramkumarkb
Copy link
Author

Hi @ramkumarkb , @J-F-Liu . I've fixed the windows builds and there is a build available here now:

https://github.com/bolcom/unFTP/actions/runs/4246819517

Do you perhaps want to test it out for us and give feedback?

@hannesdejager - Thank you very much for this Windows build. I will test the same in the next couple of days and will report back.

@hannesdejager
Copy link
Collaborator

I'm closing this ticket. If new issues are found on windows we can log more specific issues.

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

3 participants