[ruby-core:29372] Re: [Bug #1247] YAML::load converts some dates into strings

On 4/8/10, Yusuke ENDOH <[e-mail]> wrote:
> The above uses Psych.
> I thought you wanted to fix Syck.  Did I misunderstand you?

I want to see it fixed, one way or another. I don't much care about
what underlying implementation is used. #1311 has been a thorn in my
side for a long, long time.

My comment on #1311 was intended as a kudo to Aaron (and yourself) for
resolving this long-standing irritation. Not as a prod to make you
guys work more.

So, thanks very much for fixing it, Aaron! #1311 is dead! Huzzah!

> ... Ah, Sorry.  I didn't realize that there seems to be already
> some patches provided in #1311. (I cannot look them because of
> glitch of redmine, though)
> Aaron, why don't import already provided patches for Syck?

Aaron, if you're reading this, merging those patches is a low priority
for me now that psych is in place.

>> There are a few more testcases on #1311 way down in the comments that
>> ought to be checked.... I'll be happy to see if they're still borken
>> after I get out from under my other obligations for this morning.
>
> I didn't realize them too.  Thanks.
> Because of redmine, I cannot check them now.
> Will do later.

The two additional cases I had were that these 2 strings would not round-trip:
  "\n"
  "\r\n"

There was at least one other test case in a comment by someone else
that I cannot remember now. I was gonna check on ruby-versions.net and
see if they're fixed, but evidently ruby-versions.net does not have
the psych library installed. :( Guess I gotta actually install it all
myself.