Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

* Bug fix, remove each_line method #121

Merged
merged 1 commit into from
Aug 25, 2013
Merged

Conversation

abraham1901
Copy link
Contributor

Class Array don't have method each_line on ruby 1.8.x, 1.9.x, 2.0.x

proxy_set_header - array
try_files - array
index_files -array
include_files - array
vhost_cfg_append - array

Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Failed to parse template nginx/vhost/vhost_location_directory.erb:
Filepath: /usr/lib/ruby/vendor_ruby/rgen/array_extensions.rb
Line: 25
Detail: undefined method `each_line' for ["index.php", "index.html", "index.htm"]:Array
at /etc/puppet/modules/nginx/manifests/resource/location.pp:148 on node xxx
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run

@gorsuch
Copy link

gorsuch commented Aug 24, 2013

I believe I'm getting bit by this as well, just started using this module this evening:

Error: Could not retrieve catalog from remote server: Error 400 on SERVER: Failed to parse template nginx/vhost/vhost_header.erb:
  Filepath: /etc/puppet/modules/nginx/templates/vhost/vhost_header.erb
  Line: 14
  Detail: undefined method `each_line' for []:Array
 at /etc/puppet/modules/nginx/manifests/resource/vhost.pp:136

@gorsuch
Copy link

gorsuch commented Aug 24, 2013

Also noting that I dropped down to 0.0.3, and all is good.

$ puppet --version
3.2.4

$ ruby -v
ruby 1.9.3p194 (2012-04-20 revision 35410) [x86_64-linux]

@jfryman
Copy link
Contributor

jfryman commented Aug 25, 2013

😡 Sorry about that, all. Thanks @abraham1901 for the fix.

jfryman pushed a commit that referenced this pull request Aug 25, 2013
* Bug fix, remove each_line method
@jfryman jfryman merged commit f8642b3 into voxpupuli:master Aug 25, 2013
@gorsuch
Copy link

gorsuch commented Aug 26, 2013

Thanks guys!

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.

3 participants