From: Yehuda Katz Date: 2010-05-19T03:22:30+09:00 Subject: [ruby-core:30304] Re: [Bug #3112] require "yaml" doesn't use psych as default --00c09f93d92c891b530486e26c5f Content-Type: text/plain; charset=ISO-8859-1 Yugui, I am worried about possible encoding issues that can come in Syck that will be hard to fix. For instance, Aaron was able to fix Psych to honor default_internal in just a few days, and there is no real maintainer for Syck. Since Aaron is the maintainer for Psych, and there is no maintainer for Syck, I think it makes sense to make Psych the default, with the yamler= option for backward compatibility. Yehuda Katz Architect | Engine Yard (ph) 718.877.1325 On Tue, May 18, 2010 at 3:30 PM, Yuki Sonoda wrote: > Issue #3112 has been updated by Yuki Sonoda. > > Target version changed from 1.9.2 to 1.9.x > > I think syck works more or less correctly unlike dl2. So I want to decide > it conservatively. I don't want to switch the default yaml engine to psych. > ---------------------------------------- > http://redmine.ruby-lang.org/issues/show/3112 > > ---------------------------------------- > http://redmine.ruby-lang.org > > --00c09f93d92c891b530486e26c5f Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Yugui,

I am worried about possible encoding issues that = can come in Syck that will be hard to fix. For instance, Aaron was able to = fix Psych to honor default_internal in just a few days, and there is no rea= l maintainer for Syck.

Since Aaron is the maintainer for Psych, and there is n= o maintainer for Syck, I think it makes sense to make Psych the default, wi= th the yamler=3D option for backward compatibility.

Yehuda Katz
Architect | Engine Yard
(ph) 718.877.1325


On Tue, May 18, 2010 at 3:30 PM, Yuki So= noda <redmine= @ruby-lang.org> wrote:
Issue #3112 has been updated by Yuki Sonoda.

Target version changed from 1.9.2 to 1.9.x

I think syck works more or less correctly unlike dl2. So I want to decide i= t conservatively. I don't want to switch the default yaml engine to psy= ch.
----------------------------------------
----------------------------------------

--00c09f93d92c891b530486e26c5f--