-
Notifications
You must be signed in to change notification settings - Fork 357
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
Wraith latest mode failure #546
Comments
Juan, you're right. This is related to |
I pushed a docker image "superwese/wraith_chrome". |
Didn't work for me really, was getting bunch of errors when executing
on saving images step and fatal error on the end. Also manually executing steps like
Is not working. I think because of a true parameter passed in The only reliable way to make this work was to manually patch cli.rb as in 4f43aca. |
+1 on the solution above. Adding setup_folders(config) to cli.rb worked for me. |
I came across the same problem. |
Fixed in v4.2.2. |
Hello!
I'm trying to use Wraith in history mode but I've quickly run into an issue. The
history
step works well, but trying to runlatest
errors out. Relevant part of the error log:I checked out the code and poking around found that calling
setup_folders(config)
beforesave_images(config, true)
inlib/wraith/cli.rb
~L159 seems to sort the issue (command runs and can see expected results in the generated gallery).Now, this is a total stab in the dark and I have no idea about possible ramifications, but thought of letting you know!
To reproduce:
gem install wraith
and sufficiently new Chrome on Mac for headless mode (note doubt this is related to Chrome, but I don't want to install Phantom since it's been mostly abandoned).wraith history history.yaml
.wraith latest history.yaml
.Expected: commands run without problems, gallery is created.
Actual:
latest
command crashes, no gallery created.Hope it helps.
Juan
ps: if adding that line of code is the right fix let me know and I'll send a PR over.
Issue checklist:
wraith info
command and pasted the output below:verbose: true
to my config) and pasted the output below:The text was updated successfully, but these errors were encountered: