Skip to content

Depends on constant that is only in nokogiri 1.5+#78

Merged
stouset merged 1 commit intoSAML-Toolkits:masterfrom
danielfarrell:nokogiri-fix
Apr 5, 2013
Merged

Depends on constant that is only in nokogiri 1.5+#78
stouset merged 1 commit intoSAML-Toolkits:masterfrom
danielfarrell:nokogiri-fix

Conversation

@danielfarrell
Copy link
Copy Markdown
Contributor

I ran into this when including into an app that already was locked at a nokogiri 1.4.x version. It gave the following error:

uninitialized constant Nokogiri::XML::XML_C14N_EXCLUSIVE_1_0

That constant came into being with nokogiri 1.5, so this should depend on that.

stouset added a commit that referenced this pull request Apr 5, 2013
Depends on constant that is only in nokogiri 1.5+
@stouset stouset merged commit 26cb5aa into SAML-Toolkits:master Apr 5, 2013
@danielfarrell danielfarrell deleted the nokogiri-fix branch April 5, 2013 01:29
@ericenphase
Copy link
Copy Markdown

Those constants are actually defined in 1.5.1. They do not exist in Nokogiri 1.5.0.

@phene
Copy link
Copy Markdown
Contributor

phene commented Dec 4, 2013

Just ran into this issue. @ericenphase is correct that 1.5.1 is the required Nokogiri

@danielfarrell
Copy link
Copy Markdown
Contributor Author

One of you guys should make a pull request requiring 1.5.1 or above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants