From: Monty Williams <monty@...>
Date: 2010-05-25T13:18:12+09:00
Subject: [ruby-core:30409] Re: [Bug #3112] require "yaml" doesn't use 	psych as default

This is a multi-part message in MIME format.
--------------000309060306080502030306
Content-Type: text/plain; charset=iso-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

I'm with Yehuda on this one. I have talked to people at pdx.rb who had 
to deal with invalid YAML problems arising from Syck. I don't see value 
in perpetuating invalid YAML. It would seem better to default to Psych 
and be able to better interoperate with other systems that generate 
correct YAML.

Monty Williams
MagLev Team

On 5/23/10 4:29 PM, Yehuda Katz wrote:
> Bump.
>
> I think the lack of a Syck maintainer, yet making Syck the default in 
> 1.9.2 needs to be addressed. 1.9.2 is probably going to be the 
> de-facto release for some time. What's the rationale for keeping an 
> unmaintained major component when a maintained version is available in 
> the standard library?
>
> Yehuda Katz
> Architect | Engine Yard
> (ph) 718.877.1325
>
>
> On Tue, May 18, 2010 at 11:31 AM, Aaron Patterson 
> <aaron@tenderlovemaking.com <mailto:aaron@tenderlovemaking.com>> wrote:
>
>     On Tue, May 18, 2010 at 08:30:14PM +0900, 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.
>
>     I'm afraid that if we use Syck as default, no one will see errors from
>     invalid YAML files.  If we make Psych as default, people will see
>     errors
>     with their YAML files but *have the option* to move back to Syck.
>
>     Are we going to remove Syck in 1.9.3?  If we don't make Psych
>     default in
>     1.9.2, it seems to me that we have to wait until 1.9.4 to remove Syck.
>
>     Since Syck has no active maintainer, it seems bad to wait so long to
>     remove it.
>
>     --
>     Aaron Patterson
>     http://tenderlovemaking.com/
>
>



--------------000309060306080502030306
Content-Type: text/html; charset=iso-8859-1
Content-Transfer-Encoding: 7bit

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=iso-8859-1"
 http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
I'm with Yehuda on this one. I have talked to people at pdx.rb who had
to deal with invalid YAML problems arising from Syck. I don't see value
in perpetuating invalid YAML. It would seem better to default to
Psych and be able to better interoperate with other systems that
generate correct YAML.<br>
<br>
Monty Williams<br>
MagLev Team<br>
<br>
On 5/23/10 4:29 PM, Yehuda Katz wrote:
<blockquote
 cite="mid:AANLkTikqGc8mP9UCsPDMDDsgmL1s7AVTngY7iYaNkTSz@mail.gmail.com"
 type="cite">Bump.
  <div><br>
  </div>
  <div>I think the lack of a Syck maintainer, yet making Syck the
default in 1.9.2 needs to be addressed. 1.9.2 is probably going to be
the de-facto release for some time. What's the rationale for keeping an
unmaintained major component when a maintained version is available in
the standard library?</div>
  <div><br clear="all">
Yehuda Katz<br>
Architect | Engine Yard<br>
(ph) 718.877.1325<br>
  <br>
  <br>
  <div class="gmail_quote">On Tue, May 18, 2010 at 11:31 AM, Aaron
Patterson <span dir="ltr">&lt;<a moz-do-not-send="true"
 href="mailto:aaron@tenderlovemaking.com">aaron@tenderlovemaking.com</a>&gt;</span>
wrote:<br>
  <blockquote class="gmail_quote"
 style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
    <div class="im">On Tue, May 18, 2010 at 08:30:14PM +0900, Yuki
Sonoda wrote:<br>
&gt; Issue #3112 has been updated by Yuki Sonoda.<br>
&gt;<br>
&gt; Target version changed from 1.9.2 to 1.9.x<br>
&gt;<br>
&gt; 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.<br>
    <br>
    </div>
I'm afraid that if we use Syck as default, no one will see errors from<br>
invalid YAML files. &nbsp;If we make Psych as default, people will see errors<br>
with their YAML files but *have the option* to move back to Syck.<br>
    <br>
Are we going to remove Syck in 1.9.3? &nbsp;If we don't make Psych default in<br>
1.9.2, it seems to me that we have to wait until 1.9.4 to remove Syck.<br>
    <br>
Since Syck has no active maintainer, it seems bad to wait so long to<br>
remove it.<br>
    <div>
    <div class="h5"><br>
--<br>
Aaron Patterson<br>
    <a moz-do-not-send="true" href="http://tenderlovemaking.com/"
 target="_blank">http://tenderlovemaking.com/</a><br>
    </div>
    </div>
  </blockquote>
  </div>
  <br>
  </div>
</blockquote>
<br>
<br>
</body>
</html>

--------------000309060306080502030306--