diff options
author | Jon Leighton <j@jonathanleighton.com> | 2011-08-13 13:54:54 +0100 |
---|---|---|
committer | Jon Leighton <j@jonathanleighton.com> | 2011-08-13 13:54:54 +0100 |
commit | 34689c40a03c9921b5c43ac1e120a9885edded73 (patch) | |
tree | c1b70ae5e71a48daa96b357787c9a56085e169fe /activemodel/lib/active_model/observing.rb | |
parent | 6b56f059bfb5e0c9ff4198864cf469077bb91b71 (diff) | |
download | rails-34689c40a03c9921b5c43ac1e120a9885edded73.tar.gz rails-34689c40a03c9921b5c43ac1e120a9885edded73.tar.bz2 rails-34689c40a03c9921b5c43ac1e120a9885edded73.zip |
Work around for lolruby bug. (Read on for explanation.)
We were experiencing CI test failures, for example:
* 3-1-stable: http://travis-ci.org/#!/rails/rails/builds/79473/L407
* master: http://travis-ci.org/#!/rails/rails/builds/79507/L80
These failures only happened on 1.8.7-p352, and we were only able to
reproduce on the Travis CI VM worker. We even tried creating a new 32
bit Ubuntu VM and running the tests on that, and it all worked fine.
After some epic trial and error, we discovered that replacing the
following:
fuu = Marshal.load(Marshal.dump(fuu))
with:
marshalled = Marshal.dump(fuu)
fuu = Marshal.load(marshalled)
seemed to prevent the failure.
We have NO IDEA why this is. If anyone has some great insight to
contribute then that is welcome. Otherwise, hopefully this will just
help us get the CI green again.
Many thanks to @joshk for help with sorting this out.
Diffstat (limited to 'activemodel/lib/active_model/observing.rb')
0 files changed, 0 insertions, 0 deletions