Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

Psych does not handle symbols in YAML, used in ActiveSupport

#6354

Bundler 1.0.10 loads Psych by default. (Bundler commit a318f91) At least ActiveSupport 2.3.10 subtly breaks because of this.

Psych does not seem to understand symbols in inline sequences used in lib/active_support/locale/en.yml. This can be triggered by any action that loads this file, in my case it was Array#to_sentence.

It looks like this was already fixed in 3.0, which uses the block notation for sequences instead. This works fine in Psych.

Reported by Shtééf · February 1st, 2011 @ 04:00 PM

State: committed
Milestone: none
Assigned to: nobody
Importance: Low

Activity

  1. Aaron Patterson
    Aaron Patterson
    • State changed from new to committed
    • Importance changed from to Low

    I've cherry picked the fix to 2-3-stable. It should be released with 2.3.11

    February 1st, 2011 @ 09:31 PM

  2. Nikos Dimitrakopoulos
    Nikos Dimitrakopoulos

    Aaron, could you please point me to the commit that this is fixed (either in 3.x or in 2.x)? I'm asking cause I'm currently on 3.0.4 and I'm getting some troubles with this thing (got here from http://stackoverflow.com/questions/4980877/rails-error-couldnt-pars...)

    February 28th, 2011 @ 08:25 PM

  3. Shtééf
    Shtééf

    Aaron fixed this in: https://github.com/rails/rails/commit/dc94d81

    (For future reference, you can browse to the specific file on GitHub, and click 'history' top-right of the file content view.)

    February 28th, 2011 @ 08:43 PM

  4. Nikos Dimitrakopoulos
    Nikos Dimitrakopoulos

    Thanks!

    February 28th, 2011 @ 09:14 PM

  5. Dave Myron
    Dave Myron

    So is there an explanation why this would be happening in 3.0.4? Would some gem cause this problem to rear its head again?

    February 28th, 2011 @ 10:46 PM