Skip to content
This repository has been archived by the owner on Mar 12, 2019. It is now read-only.

Remove the already initialized constant error #23

Closed
wants to merge 1 commit into from
Closed

Remove the already initialized constant error #23

wants to merge 1 commit into from

Conversation

retr0h
Copy link

@retr0h retr0h commented Aug 27, 2012

emopop:chef-repo(master)% spiceweasel infrastructure.yml
/Users/jdewey/.rvm/gems/ruby-1.9.3-p194@chef-repo/gems/spiceweasel-1.1.2/bin/spiceweasel:33: warning: already initialized constant DEBUG

Tracked this down to bundler's no_exec_wrapper.
It defines DEBUG on the global namespace.
https://github.com/mpapis/rubygems-bundler/blob/master/lib/rubygems-bundler/noexec.rb

Maybe bundler is in the wrong for defining this on the global namespace. However,
we are doing the same thing in spiceweasel. Decided to namespace all the constants
under ::Spiceweasel.

emopop:chef-repo(master)% spiceweasel infrastructure.yml
/Users/jdewey/.rvm/gems/ruby-1.9.3-p194@chef-repo/gems/spiceweasel-1.1.2/bin/spiceweasel:33: warning: already initialized constant DEBUG

Tracked this down to bundler's no_exec_wrapper.
It defines DEBUG on the global namespace.
  https://github.com/mpapis/rubygems-bundler/blob/master/lib/rubygems-bundler/noexec.rb

Maybe bundler is in the wrong for defining this on the global namespace.
However, we are doing the same thing in spiceweasel.  Decided to namespace
all the constants under ::Spiceweasel.
@mattray mattray closed this Dec 10, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants