-
Notifications
You must be signed in to change notification settings - Fork 56
No such file or directory #3
Comments
Are you sure your repository is clean ? I've tried cloning it and I can successfully generate the website with jekyll. Same for glynn, which uses the jekyll methods to generate the site and tries to send it over ftp (of course I can't send it as I don't have your ftp login/password). |
You mean the _site dir needs to be empty when I run glynn? Did that, and glynn sure enough generated everything as expected, but still get the error when it tries to upload the files :/ I'm trying to investigate your code.. you're running a "file" command? I'm running on Windows - will this work there? |
Oooh you're running it on windows. This explains everything :) That reason is simple. To check if a file is binary (in which case it needs to be sent with the putbinaryfile method) or is text (in which case it needs to be sent with the puttextfile method), I do a file command. How would you do it on windows ? (check if a file is binary or not) ? |
I've forked the project, committed a fix, and requested a pull.. The change: http://github.com/tormaroe/glynn/commit/4c0c0225b663dfc566187285d7a94ab54106ac51 |
Thank you. I'll take a look at it asap. |
When you say it works, does it mean the tests pass ? Because they don't here (mainly because .blksize isn't implemented in fakefs). Moreover instead of dumping the File module, you can call a method outside of the current module with the following : |
No, sorry, I wasn't able to run the tests at all for some reason. It worked in the sense that I was able to generate and upload my jekyll site successfully. I'll have another go at it when time permits.. And thanks for the tip on calling outside current. |
Closing as stale. |
When running glynn I get the following exception. It happens after ftp-login, and the root folder I specified in the _config.yml was created on the server by glynn. The file in the exception DOES exist.
You can access my source here: http://github.com/tormaroe/barnesang.com
The text was updated successfully, but these errors were encountered: