-
Notifications
You must be signed in to change notification settings - Fork 139
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
Problems with wget #30
Comments
I have another remote ghost instance running here. Tried with that also on Windows and same error. |
Maybe buster could be updated to rely on urllib or requests instead of wget |
@zlalanne wget for Windows should not be any different than wget on Linux IMO. |
@ibrahim-islam Agreed. But you can't guarantee that. You also can't guarantee that wget is going to be in the user's path on windows or that the user even has wget installed. Or that new versions of wget won't break this functionality. I think a pure python approach using requests/urllib would provide a better solution and would remove the dependency on having wget installed at all. |
@zlalanne You have a point. Why don't you change this and send a pull request? I know I would benefit from that. |
Solved the problem by replacing |
My instance of Ghost is running latest 0.5. There was no mention of installing wget for Windows. Anyway, I did that and after running
buster generate --domain=http://localhost:2368
I get the following error:The text was updated successfully, but these errors were encountered: