Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Save the URI the user passes in bin/nokogiri. #745

Closed
wants to merge 1 commit into from

3 participants

Eero Saynatkari Don't Add Me To Your Organization a.k.a The Travis Bot Lee Jarvis
Eero Saynatkari

It’s awkward not to store the URI the user gives. Also awkward to not have tests…should probably add some. But I’ll do that after/if this change is deemed appropriate at all.

Don't Add Me To Your Organization a.k.a The Travis Bot

This pull request fails (merged 30fedb3 into c3953c2).

Lee Jarvis
Owner

Closing this per our discussion on IRC. Please re-open with a problem reproduction if needed.

Lee Jarvis leejarvis closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Aug 12, 2012
  1. Eero Saynatkari
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  bin/nokogiri
2  bin/nokogiri
View
@@ -50,7 +50,7 @@ if uri.to_s.strip.empty?
exit 1
end
-@doc = parse_class.parse(open(uri).read, nil, encoding)
+@doc = parse_class.parse(open(uri).read, uri, encoding)
if @rng
@rng.validate(@doc).each do |error|
Something went wrong with that request. Please try again.