-
Notifications
You must be signed in to change notification settings - Fork 2
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
Default workflow doesn't work out of box on windows #2
Comments
this is not isolated to windows, just recreated the issue on macOS following the instructions in the readme.md |
Can we cast the generator object to a tuple and move on? |
Depending on how it tries to interact with the pickled object yeah, that might be a solution. But if its relying on the data not yet being solidified then it could be wrong. |
In the past i've run pelican server's directly with |
@theodox Do the bat files work for you? Use peldev.bat to generate/auto-reload the website: I made them as a workaround for getpelican/pelican#2400 |
Or do we want to fix the issue and make a PR for Pelican? |
I just installed python 3.7.3 on macOS and I can't reproduce the issue :( @cmcpasserby What python are you using? |
for 3.7 still failing to start the server in the bat file. The site generation appears to work but nobody's listening on the loopback:
|
was using 3.8 |
The fix is to bump Pelican from 4.2.0 to 4.5.0. Here is the fix |
Did that update resove the issue for you? |
Not yet. Site generates but the dev server still seems to fail silently. when I ctrl-c
|
at least now |
I'm having a hard time reproducing the issue, that you are talking about, on Windows with py37. Just to make sure we are on the same page. @theodox |
I made a wrapper script to guarantee Pelican process termination when you send CTRL+C. |
I tried setting this up on my home machine which is running 3.7.7, but
make devserver
fails withIt might ultimately be this issue: getpelican/pelican#2400
Still investigating... but we should definitely offer a windows workflow that works out of the box
The text was updated successfully, but these errors were encountered: