Merge pull request #763 from rodjek/issue-759

Restore Ruby 1.8.7 support

Author: rodjek

.rubocop.yml

@@ -62,7 +62,7 @@ Style/MethodCalledOnDoEndBlock:
 Style/RegexpLiteral:
     EnforcedStyle: percent_r
 Style/TrailingCommaInArguments:
-    EnforcedStyleForMultiline: comma
+    EnforcedStyleForMultiline: no_comma
 Style/TrailingCommaInLiteral:
     EnforcedStyleForMultiline: comma
 Style/FormatStringToken:

.travis.yml

@@ -11,6 +11,7 @@ script:
 env:
     - CHECK=ci
 rvm:
+  - 1.8.7
   - 1.9.3
   - 2.0.0
   - 2.1.10

lib/puppet-lint.rb

@@ -8,6 +8,7 @@
 require 'puppet-lint/data'
 require 'puppet-lint/checks'
 require 'puppet-lint/bin'
+require 'puppet-lint/monkeypatches'
 
 class PuppetLint::NoCodeError < StandardError; end
 class PuppetLint::NoFix < StandardError; end

lib/puppet-lint/checks.rb

@@ -68,7 +68,7 @@ def run(fileinfo, data)
 
     @problems
   rescue => e
-    puts <<-END.gsub(%r{^ {6}}, '')
+    $stdout.puts <<-END.gsub(%r{^ {6}}, '')
       Whoops! It looks like puppet-lint has encountered an error that it doesn't
       know how to handle. Please open an issue at https://github.com/rodjek/puppet-lint
       and paste the following output into the issue description.
@@ -80,15 +80,15 @@ def run(fileinfo, data)
     END
 
     if File.readable?(fileinfo)
-      puts [
+      $stdout.puts [
         'file contents:',
         '```',
         File.read(fileinfo),
         '```',
       ].join("\n")
     end
 
-    puts [
+    $stdout.puts [
       'error:',
       '```',
       "#{e.class}: #{e.message}",

lib/puppet-lint/configuration.rb

@@ -50,9 +50,9 @@ def self.add_check(check)
     #
     #   <option>=(value)
     def method_missing(method, *args, &_block)
-      super unless method.to_s =~ %r{^(?<option>\w+)=?$}
+      super unless method.to_s =~ %r{^(\w+)=?$}
 
-      option = Regexp.last_match(:option)
+      option = Regexp.last_match(1)
       add_option(option.to_s) if settings[option].nil?
       settings[option] = args[0] unless args.empty?
     end

lib/puppet-lint/lexer.rb

@@ -170,7 +170,7 @@ def heredoc_queue
     # \v == vertical tab
     # \f == form feed
     # \p{Zs} == ASCII + Unicode non-linebreaking whitespace
-    WHITESPACE_RE = %r{[\t\v\f\p{Zs}]}
+    WHITESPACE_RE = RUBY_VERSION == '1.8.7' ? %r{[\t\v\f ]} : %r{[\t\v\f\p{Zs}]}
 
     # Internal: Access the internal token storage.
     #
@@ -266,7 +266,7 @@ def tokenise(code)
           else
             heredoc_tag = heredoc_queue.shift
             heredoc_name = heredoc_tag[%r{\A"?(.+?)"?(:.+?)?(/.*)?\Z}, 1]
-            str_contents = StringScanner.new(code[i + length..-1]).scan_until(%r{\|?\s*-?\s*#{heredoc_name}})
+            str_contents = StringScanner.new(code[(i + length)..-1]).scan_until(%r{\|?\s*-?\s*#{heredoc_name}})
             interpolate_heredoc(str_contents, heredoc_tag)
             length += str_contents.size
           end
@@ -282,8 +282,8 @@ def tokenise(code)
           unless heredoc_queue.empty?
             heredoc_tag = heredoc_queue.shift
             heredoc_name = heredoc_tag[%r{\A"?(.+?)"?(:.+?)?(/.*)?\Z}, 1]
-            str_contents = StringScanner.new(code[i + length..-1]).scan_until(%r{\|?\s*-?\s*#{heredoc_name}})
-            _ = code[0..i + length].split("\n")
+            str_contents = StringScanner.new(code[(i + length)..-1]).scan_until(%r{\|?\s*-?\s*#{heredoc_name}})
+            _ = code[0..(i + length)].split("\n")
             interpolate_heredoc(str_contents, heredoc_tag)
             length += str_contents.size
           end
@@ -557,8 +557,8 @@ def get_heredoc_segment(string, eos_text, interpolate = true)
 
       str = string.scan_until(regexp)
       begin
-        str =~ %r{\A(?<value>.*?)(?<terminator>[$]+|\|?\s*-?#{Regexp.escape(eos_text)})\Z}m
-        [Regexp.last_match(:value), Regexp.last_match(:terminator)]
+        str =~ %r{\A(.*?)([$]+|\|?\s*-?#{Regexp.escape(eos_text)})\Z}m
+        [Regexp.last_match(1), Regexp.last_match(2)]
       rescue
         [nil, nil]
       end

lib/puppet-lint/monkeypatches.rb

@@ -0,0 +1,51 @@
+begin
+  '%{test}' % { :test => 'replaced' } == 'replaced' # rubocop:disable Style/FormatString
+rescue
+  # monkeypatch String#% into Ruby 1.8.7
+  class String
+    Percent = instance_method('%') unless defined?(Percent)
+
+    def %(*a, &b)
+      a.flatten!
+
+      string = case a.last
+               when Hash
+                 expand(a.pop)
+               else
+                 self
+               end
+
+      if a.empty?
+        string
+      else
+        Percent.bind(string).call(a, &b)
+      end
+    end
+
+    def expand!(vars = {})
+      loop do
+        changed = false
+        vars.each do |var, value|
+          var = var.to_s
+          var.gsub!(%r{[^a-zA-Z0-9_]}, '')
+          changed = gsub!(%r{\%\{#{var}\}}, value.to_s)
+        end
+        break unless changed
+      end
+      self
+    end
+
+    def expand(opts = {})
+      dup.expand!(opts)
+    end
+  end
+end
+
+unless String.respond_to?(:prepend)
+  # monkeypatch String#prepend into Ruby 1.8.7
+  class String
+    def prepend(lead)
+      replace("#{lead}#{self}")
+    end
+  end
+end

lib/puppet-lint/optparser.rb

@@ -44,7 +44,7 @@ def self.build
       opts.on(
         '--error-level LEVEL',
         [:all, :warning, :error],
-        'The level of error to return (warning, error or all).',
+        'The level of error to return (warning, error or all).'
       ) do |el|
         PuppetLint.configuration.error_level = el
       end

lib/puppet-lint/plugins/check_classes/arrow_on_right_operand_line.rb

@@ -12,7 +12,7 @@ def check
         :message =>  'arrow should be on the right operand\'s line',
         :line    => token.line,
         :column  => token.column,
-        :token   => token,
+        :token   => token
       )
     end
   end

lib/puppet-lint/plugins/check_classes/autoloader_layout.rb

@@ -27,7 +27,7 @@ def check
         :error,
         :message => "#{title_token.value} not in autoload module layout",
         :line    => title_token.line,
-        :column  => title_token.column,
+        :column  => title_token.column
       )
     end
   end