search

dtao / safe_yaml

Parse YAML safely
MIT License
217 stars 63 forks source link

Remove warning about libyaml < 0.1.17 for jruby #59

Closed printercu closed 10 years ago

printercu commented 10 years ago

I'm new to jruby, but seems like it's using its own version of yaml: https://github.com/wayneeseguin/rvm/issues/2604

So warning from https://github.com/dtao/safe_yaml/commit/8dc900a6eeab61559fd98b940807c76e0f54f7cd can't be removed by updating psych.

mkristian commented 10 years ago

the future releases of jruby will use the version of the underlying yaml implementation (snakeyaml) instead of a some hardcoded libyaml version. https://github.com/jruby/jruby/commit/5e74ee22bb75eb4f352e1c847214a26c7e9d2425

so checking the libyaml with jruby not work like this and has not needed at all - jruby does not use libyaml !!!