Backport #4641: Please backport r31418 to 1.9.2 stable branch

• Status: Rejected
• Assignee: yugui (Yuki Sonoda)

[tenderlovemaking (Aaron Patterson)]

Syck cannot load valid YAML times that Psych dumps. For example '2010-10-10 00:00:00.000000000Z' is a valid YAML time, but Syck cannot load it.

r31418 makes Psych dump time in a way that Syck can load.

[yugui (Yuki Sonoda)]

• Status changed from Open to Rejected

I don't want to change behavior unless it is a bug.

This interoperability issue is a bug of syck and I know it is hard to fix this bug in syck. So is it a time to move to psych?

[tenderlovemaking (Aaron Patterson)]

On Tue, May 31, 2011 at 10:47:00AM +0900, Yuki Sonoda wrote:

Issue #4641 has been updated by Yuki Sonoda.

Status changed from Open to Rejected

I don't want to change behavior unless it is a bug.

This interoperability issue is a bug of syck and I know it is hard to fix this bug in syck. So is it a time to move to psych?

I think Psych will work fine as the default YAML engine in 1.9.2.
You've backported the major bugs, and I haven't received any bug reports
other than the ones I've already fixed.

r31715 makes Psych default for 1.9.3. I think it can be easily
backported to 1.9.2.

If Psych is made default, and something doesn't work, people can easily
downgrade by changing the engine.

--
Aaron Patterson
http://tenderlovemaking.com/