
already initialized constant OrderedHash in dm-serializer
Reported by dhf (at f8technologies) | June 3rd, 2010 @ 11:10 AM
In running tests with rspec using autotest under bundler I get the following error: /Users/dhf/.rvm/gems/ruby-1.9.2-preview1/bundler/gems/dm-serializer-335910fccfe74f236018a35e7482369baa9de1e0-f4da7fb505b9df87b7e70863a17baaac44852efd/lib/dm-serializer.rb:6: warning: already initialized constant OrderedHash
Comments and changes to this ticket
-
Martin Gamsjaeger (snusnu) June 4th, 2010 @ 03:44 PM
- State changed from new to unconfirmed
- Assigned user set to Martin Gamsjaeger (snusnu)
-
Dan Kubb (dkubb) June 5th, 2010 @ 01:02 AM
@dhf: What version of ActiveSupport are you using, or are your gems depending on?
-
Dan Kubb (dkubb) June 6th, 2010 @ 12:12 PM
(from [b3423e5c835c6501ab7ae3547ee51137802c7a4a]) Removed duplicate constant aliasing
The same gets done in dm-serializer/common.rb which
gets required by all the various to_* serializers.In general, it might not even be necessary to do
any of these aliasings at all in plugin land, since
dm-core actually does it itself already, and needs
to be required before the plugins do their thing
anyway (most of the times at least).I'm not sure if this change will actually eliminate
all const redefinition warnings, but executing this
code in that place is definitely not necessary.[#1302] http://github.com/datamapper/dm-serializer/commit/b3423e5c835c6501a...
-
Martin Gamsjaeger (snusnu) June 6th, 2010 @ 12:14 PM
dhf, can you test if this eliminates the warnings you saw?
-
dhf (at f8technologies) June 7th, 2010 @ 07:52 AM
I just checked and I do not get the warning any more. Thanks
-
Martin Gamsjaeger (snusnu) June 7th, 2010 @ 08:53 AM
- State changed from unconfirmed to resolved
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
People watching this ticket
Referenced by
-
1302 already initialized constant OrderedHash in dm-serializer [#1302] http://github.com/datamapper/dm-serializer/commi...