Skip to content
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

update nokogiri to 1.8.0 for ruby >= 2.1 #2380

Merged
merged 1 commit into from
Aug 23, 2017

Conversation

miltador
Copy link

This fixes dependencies installation on Windows with ruby >= 2.4.

fixes dependencies installation on Windows with ruby >= 2.4
@mojavelinux mojavelinux changed the title update nokogiri to 1.8.0 for ruby >= 2.2 update nokogiri to 1.8.0 for ruby >= 2.1 Aug 23, 2017
@mojavelinux
Copy link
Member

I updated it to apply to Ruby 2.1 as well since that is still compatible.

@mojavelinux
Copy link
Member

Thanks!

@mojavelinux mojavelinux merged commit 33d6ec7 into asciidoctor:master Aug 23, 2017
@miltador
Copy link
Author

miltador commented Aug 23, 2017

x64-mingw32 and x86-mingw32 versions require Ruby 2.2 while general version requires 2.1. That's why I put the higher denominator of 2.2 version.

@mojavelinux
Copy link
Member

that's kind of insane. hmm, I'd rather make an exception for that one case then.

@mojavelinux
Copy link
Member

see if that works

@miltador
Copy link
Author

It works, thanks!

mojavelinux pushed a commit that referenced this pull request Mar 16, 2018
fixes dependencies installation on Windows with ruby >= 2.4
mojavelinux pushed a commit that referenced this pull request Mar 16, 2018
fixes dependencies installation on Windows with ruby >= 2.4
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.

None yet

2 participants