Skip to content

Updated pygments.rb to latest to make it build under ruby 2.4#2644

Merged
fabaff merged 1 commit into
home-assistant:nextfrom
auduny:ruby2.4
Jun 27, 2017
Merged

Updated pygments.rb to latest to make it build under ruby 2.4#2644
fabaff merged 1 commit into
home-assistant:nextfrom
auduny:ruby2.4

Conversation

@auduny
Copy link
Copy Markdown
Contributor

@auduny auduny commented May 16, 2017

Description:

On latest OS X, default ruby is 2.4. Older version of Pygments.rb depends on a version of yajl-ruby that is not buildable on latest ruby 2.4. Updated pygments to latest version. Issue #1887 seems to refer to this issue.

@Landrash Landrash added the bug This issue or PR is related to an unexpected behavoir on our website/documentation label May 17, 2017
@Landrash
Copy link
Copy Markdown
Contributor

Does this change affect the availability of building on any of the other platforms?

@auduny
Copy link
Copy Markdown
Contributor Author

auduny commented May 18, 2017

@Landrash Nope I do not think so. I just built it on Ubuntu 16.04 with ruby 2.3.1 Seems to be completely fine.

@Landrash
Copy link
Copy Markdown
Contributor

OK with you @fabaff ?

@tiktok7
Copy link
Copy Markdown
Contributor

tiktok7 commented Jun 23, 2017

Just found this problem the hard way.. tried 2.4 then found this when I was going to log the issue. It might be kinder to newbies to merge the change... 😉

@fabaff
Copy link
Copy Markdown
Member

fabaff commented Jun 27, 2017

Works on an older version of ruby too.

$ ruby -v
ruby 2.3.3p222 (2016-11-21 revision 56859) [x86_64-linux]

Copy link
Copy Markdown
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🐦

@fabaff fabaff merged commit 284e4d7 into home-assistant:next Jun 27, 2017
@balloob
Copy link
Copy Markdown
Member

balloob commented Jun 27, 2017

We should probably try to update all our gems.

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

Labels

bug This issue or PR is related to an unexpected behavoir on our website/documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants