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

Psych can't parse YML in RefineryCMS that syck can #10

Closed
parndt opened this issue Feb 23, 2011 · 3 comments
Closed

Psych can't parse YML in RefineryCMS that syck can #10

parndt opened this issue Feb 23, 2011 · 3 comments

Comments

@parndt
Copy link

parndt commented Feb 23, 2011

Hi,

We've had reports that Psych can't parse the YML inside RefineryCMS even though syck can. Here is the link to the issue on our repository with heaps of conversation around it:

https://github.com/resolve/refinerycms/issues/#issue/457

The simple solution to this issue is to put this in our config/boot.rb:

require 'yaml'
YAML::ENGINE.yamler= 'syck'

AKA disable Psych, no more problem. Weird?

Thanks!
Phil

@tenderlove
Copy link
Member

Hi Phil,

Psych uses libyaml which sticks to the YAML 1.1 spec. Syck does not. Syck is a more loose parser. libyaml will not support invalid yaml, so this is not a bug.

Syck will be removed from Ruby (someday), so I suggest you upgrade the YAML inside RefineryCMS. I can help with that if you like. :-)

@parndt
Copy link
Author

parndt commented Feb 23, 2011

That explains it!
Would appreciate ALL the help we can get with this. Due to the Christchurch Earthquake we're at limited capacity here. Plus, I can't figure out how to make the YAML valid :-)

Thanks for your very swift response, Aaron.

@tenderlove
Copy link
Member

No problem. I will pull down the code and take a look!

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants