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 #3

Open
zielmicha opened this issue Mar 4, 2016 · 6 comments
Open

Windows support #3

zielmicha opened this issue Mar 4, 2016 · 6 comments

Comments

@zielmicha
Copy link
Owner

Make reactor.nim compile on Windows.

@hcorion
Copy link

hcorion commented Sep 6, 2016

Not exactly 'trivial'.
A lot of the code includes posix (at least in the uv folder), and there are some things used that are posix only. I'm unfamiliar with using sockets in both windows and unix so I'm at a loss here.
To get started you would have to detect windows or unix/mac and import winlean if windows and posix if posix. Then you'd have to find the windows equivalent for the posix functions and variables used.
P.S. It doesn't appear to compile on the latest devel.

@zielmicha
Copy link
Owner Author

Actually most of these posix imports are unused (I have no idea why I've added them) and rest are used only for errno.

@zielmicha
Copy link
Owner Author

Anyway, it compiles for me on latest devel (emit tons of warnings, I've just fixed them). Could you post an error message?

@hcorion
Copy link

hcorion commented Sep 8, 2016

Oh, interesting. It appears to be my installation of nim that is the problem. I'm getting casting errors,
reactor\uv\uvstream.nim(33, 18) Error: type mismatch: got (int, culong)
I'll look into it, probably a problem with my installation.

@zielmicha
Copy link
Owner Author

This is caused by bad definition of uv_buf_t for Windows - this is one of these things that need adjusting.

@zielmicha
Copy link
Owner Author

698a9f1 is nearly there, but there are still some problems with linker.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants