Skip to content

EY::Config should require yaml, but not#1

Open
ursm wants to merge 2 commits into
engineyard:masterfrom
ursm:yaml
Open

EY::Config should require yaml, but not#1
ursm wants to merge 2 commits into
engineyard:masterfrom
ursm:yaml

Conversation

@ursm

@ursm ursm commented Jul 5, 2013

Copy link
Copy Markdown

No description provided.

@shaiguitar

Copy link
Copy Markdown

When/where is this error happening?

@ursm

ursm commented Jul 5, 2013

Copy link
Copy Markdown
Author

As you can see.

$ ey_config_local foo
/home/ursm/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/ey_config-0.0.6/lib/ey_config/local.rb:29:in `existing_contents': uninitialized constant Class::YAML (NameError)
    from /home/ursm/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/ey_config-0.0.6/lib/ey_config/local.rb:12:in `generate'
    from /home/ursm/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/ey_config-0.0.6/bin/ey_config_local:14:in `<top (required)>'
    from /home/ursm/.rbenv/versions/2.0.0-p247/bin/ey_config_local:23:in `load'
    from /home/ursm/.rbenv/versions/2.0.0-p247/bin/ey_config_local:23:in `<main>'

@ursm

ursm commented Jul 5, 2013

Copy link
Copy Markdown
Author

In other case:

$ mkdir config
$ touch config/ey_services_config_local.yml 
$ irb
>> require 'ey_config'
=> true
>> EY::Config.get('foo')
NameError: uninitialized constant Class::YAML
    from /home/ursm/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/ey_config-0.0.6/lib/ey_config.rb:34:in `init'
    from /home/ursm/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/ey_config-0.0.6/lib/ey_config.rb:42:in `get'
    from (irb):2
    from /home/ursm/.rbenv/versions/2.0.0-p247/bin/irb:12:in `<main>'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants