-
-
Notifications
You must be signed in to change notification settings - Fork 222
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
Running into an Error on Windows #5
Comments
Hey, thank you for the issue. I will have a closer look at the windows support soon. But yes, the intention was not to support Windows out of the box. 😁 |
Came back to this after a day or two, but I realized the problem was pretty obvious when looking at what the makefile commands were doing. It's this line -- In particular, the Windows shell doesn't understand what the My quick solution was just to replace the variable with the verbatim working directory path, but this is definitely not ergonomic by any means -- there's probably a better way of doing it. But at the very least this will finish building the webapp image. I've tried
It's a little strange -- from what I understand, startdb works totally fine. The postgres container stays alive and the db migration works. But perhaps it's related to the Dockerfile that creates the image. I'll share more of my findings if I get anywhere closer, but I get the feeling somehow that this should be a very simple fix. |
Is it on general possible to run the application on Windows with |
I won't be able to answer that right now since there's a dependency that breaks actix-web. ring 0.12.1 won't compile on MSVC 2017 (briansmith/ring#670). It's fixed on the latest release, but https://github.com/alexcrichton/cookie-rs needs to be upgraded to use it. Sorry this is so troubling. Someone has already submitted a PR for the fix, it's just up to the maintainers now to get to it (rwf2/cookie-rs#106). |
I didn't look into code, but if you don't need cookies, you could disable temporarily P.s. @saschagrunert you shouldn't treat windows so badly 😄 We poor windows plebs still want to develop |
I'm running into an error trying to deploy this app on Windows:
Though while I'm making this issue, I can provide some extra notes on what I needed to get closer to building the app:
I understand that this might not be a big priority since Windows is somewhat of a second-class citizen for webdev stuff, but I think it would be really cool to see this work there too.
Thanks for making this, by the way!
The text was updated successfully, but these errors were encountered: