-
-
Notifications
You must be signed in to change notification settings - Fork 197
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
No such file or directory @ rb_sysopen #41
Comments
After adding the missing file the issue is gone. But it is just a workaround. The question is, why Proofer tries to proof an non-existent file? |
It's more a feature then a bug. Basically, like every browser ever, Proofer expects there to be a top-level index.html file when you navigate to a folder. For pages with clean URLs (or just using Jekyll), checking for an index.html is really handy. The workaround suggested was to add |
If the link is to a folder, then it can be any number of things ( |
shifty eyes html-proofer/lib/html/proofer/checkable.rb Line 108 in 4904bf8
Yes. But now your mention of all those other file types has me in a tizzy. 😩 |
I think HTMLProofer can assume either
If |
|
Yeah, I just ran into this one as well https://travis-ci.org/nschonni/wet-boew/builds/32812844#L2463 |
@nschonni Can you please provide the exact link? This is your situation now, right? # does not exist
dist/demos/texthighlight/index.html
# do exist
dist/demos/texthighlight/texthighlight-fr.html
dist/demos/texthighlight/texthighlight-en.html |
@penibelst I believe the link it's having trouble with is You can grab the prebuilt site if you want to test it out https://github.com/wet-boew/wet-boew-dist Oddly enough, if I add |
I think #93 fixes all the mentioned issues. |
@nschonni The version 1.3.0 fixes your issue. |
Thanks @penibelst! It's not crashing anymore, but it just stalls out completely https://travis-ci.org/nschonni/wet-boew/builds/33367787 |
@nschonni Can you please start Proofer with only one file?
|
Yeah, running a smaller set help, but it's taking almost 10minutes to run on less than 200 pages now. Getting false positives for the href links, not sure if it's tripping on HTML5 elements, but this is the wrong thread for that 😉 |
@nschonni External URLs are unpredictable. Try the HTML::Proofer.new(dir, {
:timeout => 10 # seconds
}).run If you see false positives, please open a new issue with examples. |
Proofer crashes on a Travis test with:
The PR is IIIF/api#105. The traced Travis build is https://travis-ci.org/IIIF/iiif.io/builds/25216529
I can reproduce the issue on my local machine with Ruby 1.9.3.
The text was updated successfully, but these errors were encountered: