Issue #22474 has been updated by Matthaus Owens. Status changed from Unreviewed to In Topic Branch Pending Review Branch set to https://github.com/puppetlabs/puppet/pull/1886
Pull Request: <https://github.com/puppetlabs/puppet/pull/1886> ---------------------------------------- Bug #22474: --no-zlib flag doesn't prevent zlib from being required in Puppet https://projects.puppetlabs.com/issues/22474#change-97323 * Author: Matthaus Owens * Status: In Topic Branch Pending Review * Priority: Normal * Assignee: Matthaus Owens * Category: * Target version: * Affected Puppet version: 3.0.0 * Keywords: * Branch: https://github.com/puppetlabs/puppet/pull/1886 ---------------------------------------- Using a ruby without zlib support compiled in, the --no-zlib flag should prevent zlib from being used. It does not accomplish this. <pre>[root@el4-64 puppet-enterprise-3.1.0-rc0-53-g3f89475-el-4-x86_64]# puppet --no-zlib --version --trace --debug /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require': cannot load such file -- zlib (LoadError) from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/forge/repository.rb:2:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/forge.rb:6:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/module_tool.rb:148:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/module_tool/applications.rb:1:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/module.rb:3:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser/files.rb:1:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser/templatewrapper.rb:1:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser/scope.rb:6:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser/methods.rb:2:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser/ast/method_call.rb:2:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser/ast.rb:116:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser/parser.rb:11:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/parser.rb:4:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet.rb:176:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/site_ruby/1.9.1/puppet/util/command_line.rb:12:in `<top (required)>' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /opt/puppet/lib/ruby/1.9.1/rubygems/custom_require.rb:36:in `require' from /usr/local/bin/puppet:3:in `<main>'</pre> Looks like this got changed in 3.0.0 in commit e29afda8. lib/puppet/forge/repository.rb is to blame. Lines 2 and 29 reference zlib directly, and aren't wrapped in use_zlib or has_feature(zlib) calls. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.
