between_meals 0.0.6 → 0.0.11

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA256:
+  metadata.gz: e58d518baa0534eecc6de15ac0fb0f7f09a6069f6fb870a60c55adf2cf2f8404
+  data.tar.gz: 44aaa21c488bea122a228466f0c30f5d0c9eab3ff778b3f60242cccaf2020ee6
+SHA512:
+  metadata.gz: 7a44420814d8e34888bea5be482a7d8b43d30979c0c8369cb11e0b13ff4cddc36e0a3da29a955d70917ba9b878fa75213640b718658c0f4b6fabbd199e002f8a
+  data.tar.gz: 12dece7d9a249c883bef1eb0c6460977998fbc7b7e6ba90ecaf5bb7912d374df72b919bd78b1aece7548e1063664e04b6d80007ad5c68baaf2a587060cc6a0aa

data/README.md

@@ -1,13 +1,14 @@
 # Between Meals
 
-[![Build Status](https://travis-ci.org/facebook/between-meals.svg)](http://travis-ci.org/facebook/between-meals)
+[![TravisCI](https://travis-ci.org/facebook/between-meals.svg)](http://travis-ci.org/facebook/between-meals)
+[![CircleCI](https://circleci.com/gh/facebook/between-meals.svg?style=svg)](https://circleci.com/gh/facebook/between-meals)
 
 ## Intro
 Ohai!
 
-Between Meals is the library for calculating what Chef objects where modified
+Between Meals is the library for calculating what Chef objects were modified
 between two revisions in a version control system. It is also the library
-that that backs Taste Tester and Grocery Delivery.
+that backs Taste Tester and Grocery Delivery.
 
 It currently supports SVN, GIT and HG, but plugins can easily be written for
 other source control systems.
@@ -22,3 +23,6 @@ functions.
 * Mixlib::ShellOut
 * Rugged
 
+## License
+
+See the `LICENSE` file in this repo.

data/lib/between_meals/changes/change.rb

@@ -37,15 +37,11 @@ module BetweenMeals
       end
 
       def self.info(msg)
-        if @@logger
-          @@logger.info(msg)
-        end
+        @@logger&.info(msg)
       end
 
       def self.debug(msg)
-        if @@logger
-          @@logger.debug(msg)
-        end
+        @@logger&.debug(msg)
       end
 
       def info(msg)
@@ -58,3 +54,4 @@ module BetweenMeals
     end
   end
 end
+# rubocop:enable ClassVars

data/lib/between_meals/changes/cookbook.rb

@@ -14,50 +14,111 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# rubocop:disable ClassVars
 module BetweenMeals
   module Changes
     # Changeset aware cookbook
     class Cookbook < Change
-      def self.meaningful_cookbook_file?(path, cookbook_dirs)
-        cookbook_dirs.each do |dir|
-          re = %r{^#{dir}/([^/]+)/.*}
-          m = path.match(re)
-          debug("[cookbook] #{path} meaningful? [#{re}]: #{m}")
-          return true if m
-        end
-        false
+      def self.meaningful_cookbook_file?(path)
+        !explode_path(path).nil?
       end
 
-      def self.explode_path(path, cookbook_dirs)
-        cookbook_dirs.each do |dir|
+      def self.explode_path(path)
+        @cookbook_dirs.each do |dir|
           re = %r{^#{dir}/([^/]+)/.*}
           debug("[cookbook] Matching #{path} against ^#{re}")
           m = path.match(re)
           next unless m
+
           info("Cookbook is #{m[1]}")
           return {
             :cookbook_dir => dir,
-            :name => m[1] }
+            :name => m[1],
+          }
         end
         nil
       end
 
+      def self.map_symlinks(files)
+        # For each symlink get the source path, if any files have changed under
+        # the source path, fake them as coming from the symlink path. This
+        # allows the normal cookbook logic to just work.
+        symlinks = {}
+        @cookbook_dirs.each do |dir|
+          dir = File.join(@repo_dir, dir)
+          # Find symlinks in each cookbook_dir
+          links = Dir.foreach(dir).select do |d|
+            File.symlink?(File.join(dir, d))
+          end
+          links.each do |link|
+            link = File.join(dir, link)
+            next if symlinks[link]
+
+            source = File.realpath(link)
+            repo = File.join(@repo_dir, '/')
+            # maps absolute symlink path to relative source and link paths
+            symlinks[link] = {
+              'source' => source.gsub(repo, ''),
+              'link' => link.gsub(repo, ''),
+            }
+          end
+        end
+
+        # Create the file hash expected for each file that is a link or coming
+        # from a linked directory but fake the source path as a symlink path.
+        # Hacky but works :)
+        links_to_append = []
+        symlinks.each_value do |lrp| # link_abs_path, link_relative_path
+          files.each do |f|
+            # a symlink will never have trailing '/', add one.
+            f[:path] += '/' if f[:path] == lrp['link']
+
+            # If a metadata file in the path of a symlink target directory has a
+            # deleted status, check if a metadata file exists in the symlink
+            # source directory. If so, mark it as modified to prevent deletion.
+            symlink_source_dir = File.join(@repo_dir, lrp['source'])
+            if (f[:path] == File.join(lrp['link'], 'metadata.rb') ||
+                f[:path] == File.join(lrp['link'], 'metadata.json')) &&
+                f[:status] == :deleted &&
+                (File.file?(File.join(symlink_source_dir, 'metadata.rb')) ||
+                 File.file?(File.join(symlink_source_dir, 'metadata.json')))
+              f[:status] = :modified
+            end
+
+            next unless f[:path].start_with?(lrp['source'])
+
+            # This make a deep dup of the file hash
+            l = Marshal.load(Marshal.dump(f))
+            l[:path].gsub!(lrp['source'], lrp['link'])
+            links_to_append << l
+          end
+        end
+        links_to_append
+      end
+
       def initialize(files, cookbook_dirs)
         @files = files
-        @name = self.class.explode_path(
-          files.sample[:path],
-          cookbook_dirs
-        )[:name]
-        # if metadata.rb is being deleted
+        @cookbook_dirs = cookbook_dirs
+        @name = self.class.explode_path(files.sample[:path])[:name]
+        # if metadata.(json|rb) is being deleted and we aren't also
+        # adding/modifying one of those two,
         #   cookbook is marked for deletion
         # otherwise it was modified
         #   and will be re-uploaded
         if files.
            select { |x| x[:status] == :deleted }.
-           map { |x| x[:path].match(%{.*metadata\.rb$}) }.
+           map do |x|
+             x[:path].match(
+               %{^(#{cookbook_dirs.join('|')})/[^/]+/metadata\.(rb|json)$},
+             )
+           end.
            compact.
-           any?
+           any? &&
+           files.reject { |x| x[:status] == :deleted }.
+           map do |x|
+             x[:path].match(
+               %{^(#{cookbook_dirs.join('|')})/[^/]+/metadata\.(rb|json)$},
+             )
+           end.none?
           @status = :deleted
         else
           @status = :modified
@@ -66,16 +127,22 @@ module BetweenMeals
 
       # Given a list of changed files
       # create a list of Cookbook objects
-      def self.find(list, cookbook_dirs, logger)
+      def self.find(list, cookbook_dirs, logger, repo, track_symlinks = false)
+        # rubocop:disable ClassVars
         @@logger = logger
+        # rubocop:enable ClassVars
         return [] if list.nil? || list.empty?
+
         # rubocop:disable MultilineBlockChain
+        @repo_dir = File.realpath(repo.repo_path)
+        @cookbook_dirs = cookbook_dirs
+        list += map_symlinks(list) if track_symlinks
         list.
           group_by do |x|
           # Group by prefix of cookbok_dir + cookbook_name
           # so that we treat deletes and modifications across
           # two locations separately
-          g = self.explode_path(x[:path], cookbook_dirs)
+          g = self.explode_path(x[:path])
           g[:cookbook_dir] + '/' + g[:name] if g
         end.
           map do |_, change|
@@ -83,10 +150,10 @@ module BetweenMeals
           # Changes to OWNERS or other stuff that might end up
           # in [core, other, secure] dirs are ignored
           is_cookbook = change.select do |c|
-            self.meaningful_cookbook_file?(c[:path], cookbook_dirs)
+            self.meaningful_cookbook_file?(c[:path])
           end.any?
           if is_cookbook
-            BetweenMeals::Changes::Cookbook.new(change, cookbook_dirs)
+            BetweenMeals::Changes::Cookbook.new(change, @cookbook_dirs)
           end
         end.compact
         # rubocop:enable MultilineBlockChain

data/lib/between_meals/changes/databag.rb

@@ -14,7 +14,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# rubocop:disable ClassVars
 module BetweenMeals
   module Changes
     # Changeset aware databag
@@ -37,8 +36,11 @@ module BetweenMeals
       end
 
       def self.find(list, databag_dir, logger)
+        # rubocop:disable ClassVars
         @@logger = logger
+        # rubocop:enable ClassVars
         return [] if list.nil? || list.empty?
+
         list.
           select { |x| self.name_from_path(x[:path], databag_dir) }.
           map do |x|

data/lib/between_meals/changes/role.rb

@@ -14,13 +14,12 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# rubocop:disable ClassVars
 module BetweenMeals
   module Changes
     # Changeset aware role
     class Role < Change
       def self.name_from_path(path, role_dir)
-        re = %r{^#{role_dir}/(.+)\.rb}
+        re = %r{^#{role_dir}/(.+)\.(?:rb|json)}
         debug("[role] Matching #{path} against #{re}")
         m = path.match(re)
         if m
@@ -38,8 +37,11 @@ module BetweenMeals
       # Given a list of changed files
       # create a list of Role objects
       def self.find(list, role_dir, logger)
+        # rubocop:disable ClassVars
         @@logger = logger
+        # rubocop:enable ClassVars
         return [] if list.nil? || list.empty?
+
         list.
           select { |x| self.name_from_path(x[:path], role_dir) }.
           map do |x|

data/lib/between_meals/changeset.rb

@@ -27,15 +27,19 @@ module BetweenMeals
   # Basically, you always want to use BetweenMeals::Changes through this
   # helper class.
   class Changeset
-    class ReferenceError < Exception
+    class ReferenceError < RuntimeError
     end
-
-    def initialize(logger, repo, start_ref, end_ref, locations)
+    # rubocop:disable Metrics/ParameterLists
+    def initialize(
+      logger, repo, start_ref, end_ref, locations, track_symlinks = false
+    )
+      # rubocop:enable Metrics/ParameterLists
       @logger = logger
       @repo = repo
       @cookbook_dirs = locations[:cookbook_dirs].dup
       @role_dir = locations[:role_dir]
       @databag_dir = locations[:databag_dir]
+      @track_symlinks = track_symlinks
       # Figure out which files changed if refs provided
       # or return all files (full upload) otherwise
       if start_ref
@@ -49,7 +53,9 @@ module BetweenMeals
     end
 
     def cookbooks
-      BetweenMeals::Changes::Cookbook.find(@files, @cookbook_dirs, @logger)
+      BetweenMeals::Changes::Cookbook.find(
+        @files, @cookbook_dirs, @logger, @repo, @track_symlinks
+      )
     end
 
     def roles

data/lib/between_meals/cmd.rb

@@ -27,17 +27,26 @@ module BetweenMeals
     end
 
     def cmd(params, cwd = nil)
-      unless cwd
-        cwd = File.expand_path(@cwd)
-      end
+      cwd ||= File.expand_path(@cwd)
       cmd = "#{@bin} #{params}"
       @logger.info("Running \"#{cmd}\"")
       c = Mixlib::ShellOut.new(
         cmd,
-        :cwd => cwd
+        :cwd => cwd,
+        :env => {
+          # macOS needs /usr/local/bin as hg cannot be installed in /bin or
+          # /usr/bin
+          'PATH' => '/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/bin',
+        },
       )
       c.run_command
-      c.error!
+      if c.error?
+        # Let's make sure the error goes to the logs
+        @logger.error("#{@bin} failed: #{c.format_for_exception}")
+        # if our logger is STDOUT, we'll double log when we throw
+        # the exception, but that's OK
+        c.error!
+      end
       c
     end
   end

data/lib/between_meals/knife.rb

@@ -26,6 +26,8 @@ module BetweenMeals
   class Knife
     include BetweenMeals::Util
 
+    attr_accessor :cookbook_dirs
+
     def initialize(opts = {})
       @logger = opts[:logger] || nil
       @user = opts[:user] || ENV['USER']
@@ -37,11 +39,20 @@ module BetweenMeals
       @config = opts[:config] ||
                 "#{@home}/.chef/knife-#{@user}-taste-tester.rb"
       @knife = opts[:bin] || 'knife'
+      @knife_verb_option = ''
+      unless @logger.nil?
+        if @logger.level == Logger::DEBUG
+          @knife_verb_option = '-VV'
+        elsif @logger.level == Logger::INFO
+          @knife_verb_option = '-V'
+        end
+      end
       @berks = opts[:berks_bin] || 'berks'
       @berks_config = opts[:berks_config]
       @pem = opts[:pem] ||
              "#{@home}/.chef/#{@user}-taste-tester.pem"
       @role_dir = opts[:role_dir]
+      @role_type = opts[:role_type] || 'rb'
       @cookbook_dirs = opts[:cookbook_dirs]
       @databag_dir = opts[:databag_dir]
       @checksum_dir = opts[:checksum_dir]
@@ -51,30 +62,34 @@ module BetweenMeals
 
     def role_upload_all
       if File.exists?(@role_dir)
-        roles = File.join(@role_dir, '*.rb')
-        exec!("#{@knife} role from file #{roles} -c #{@config}", @logger)
+        roles = File.join(@role_dir, "*.#{@role_type}")
+        exec!("#{@knife} role from file #{roles} #{@knife_verb_option} " +
+          "-c #{@config}", @logger)
       end
     end
 
     def role_upload(roles)
       if roles.any?
-        roles = roles.map { |x| File.join(@role_dir, "#{x.name}.rb") }.join(' ')
-        exec!("#{@knife} role from file #{roles} -c #{@config}", @logger)
+        roles = roles.map do |x|
+          File.join(@role_dir, "#{x.name}.#{@role_type}")
+        end.join(' ')
+        exec!("#{@knife} role from file #{roles} #{@knife_verb_option} " +
+          "-c #{@config}", @logger)
       end
     end
 
     def role_delete(roles)
       if roles.any?
         roles.each do |role|
-          exec!(
-            "#{@knife} role delete #{role.name} --yes -c #{@config}", @logger
-          )
+          exec!("#{@knife} role delete #{role.name} #{@knife_verb_option} " +
+            "--yes -c #{@config}", @logger)
         end
       end
     end
 
     def cookbook_upload_all
-      exec!("#{@knife} cookbook upload -a -c #{@config}", @logger)
+      exec!("#{@knife} cookbook upload -a #{@knife_verb_option} " +
+        "-c #{@config}", @logger)
     end
 
     def berks_cookbook_upload_all
@@ -94,7 +109,8 @@ module BetweenMeals
     def cookbook_upload(cookbooks)
       if cookbooks.any?
         cookbooks = cookbooks.map(&:name).join(' ')
-        exec!("#{@knife} cookbook upload #{cookbooks} -c #{@config}", @logger)
+        exec!("#{@knife} cookbook upload #{cookbooks} #{@knife_verb_option} " +
+          "-c #{@config}", @logger)
       end
     end
 
@@ -108,6 +124,7 @@ module BetweenMeals
         @cookbook_dirs.each do |path|
           cookbooks.each do |cb|
             next unless File.exists?("#{path}/#{cb}")
+
             @logger.warn("Running berkshelf on cookbook: #{cb}")
             exec!("cd #{path}/#{cb} && #{@berks} install #{berks_config} && " +
               "#{@berks} upload #{berks_config}", @logger)
@@ -120,7 +137,8 @@ module BetweenMeals
       if cookbooks.any?
         cookbooks.each do |cookbook|
           exec!("#{@knife} cookbook delete #{cookbook.name}" +
-                  " --purge -a --yes -c #{@config}", @logger)
+                  " --purge -a --yes #{@knife_verb_option} -c #{@config}",
+                @logger)
         end
       end
     end
@@ -143,8 +161,9 @@ module BetweenMeals
             File.join(@databag_dir, dbname, "#{x.item}.json")
           end.join(' ')
           exec!(
-            "#{@knife} data bag from file #{dbname} #{dbitems} -c #{@config}",
-            @logger
+            "#{@knife} data bag from file #{dbname} #{dbitems} " +
+            "#{@knife_verb_option} -c #{@config}",
+            @logger,
           )
         end
       end
@@ -155,7 +174,7 @@ module BetweenMeals
         databags.group_by(&:name).each do |dbname, dbs|
           dbs.each do |db|
             exec!("#{@knife} data bag delete #{dbname} #{db.item}" +
-                    " --yes -c #{@config}", @logger)
+                    " --yes #{@knife_verb_option} -c #{@config}", @logger)
           end
           delete_databag_if_empty(dbname)
         end
@@ -180,8 +199,11 @@ BLOCK
         cfg << "  \"#{dir}\",\n"
       end
       cfg << "]\n"
-      unless File.directory?(File.dirname(@config))
-        Dir.mkdir(File.dirname(@config), 0755)
+      begin
+        Dir.mkdir(File.dirname(@config), 0o755)
+      rescue Errno::EEXIST
+        # not an error if it's already there.
+        nil
       end
       if !File.exists?(@config) ||
          ::Digest::MD5.hexdigest(cfg) !=
@@ -220,7 +242,14 @@ zvgEHqbS0/QkJGOZ+UifPRanTDuGYQkPdHHOER4UghbM+Kz5rZbBicJ3bCyNOsah
 IAMAEpsWX2s2A6phgMCx7kH6wMmoZn3hb7Thh9+PfR8Jtp2/7k+ibCeF4gEWUCs5
 6wX4GR84dwyhG80yd4TP8Qo=
 -----END PRIVATE KEY-----
-    BLOCK
+      BLOCK
+
+      begin
+        Dir.mkdir(File.dirname(@pem), 0o755)
+      rescue Errno::EEXIST
+        # not an error if it's already there.
+        nil
+      end
 
       unless File.exists?(@pem)
         @logger.info("Generating #{@pem}")
@@ -232,22 +261,25 @@ IAMAEpsWX2s2A6phgMCx7kH6wMmoZn3hb7Thh9+PfR8Jtp2/7k+ibCeF4gEWUCs5
 
     def create_databag_if_missing(databag)
       s = Mixlib::ShellOut.new("#{@knife} data bag list" +
-                               " --format json -c #{@config}").run_command
+                               " --format json #{@knife_verb_option} " +
+                               "-c #{@config}").run_command
       s.error!
-      db = JSON.load(s.stdout)
+      db = JSON.parse(s.stdout)
       unless db.include?(databag)
-        exec!("#{@knife} data bag create #{databag} -c #{@config}", @logger)
+        exec!("#{@knife} data bag create #{databag} #{@knife_verb_option} " +
+          "-c #{@config}", @logger)
       end
     end
 
     def delete_databag_if_empty(databag)
       s = Mixlib::ShellOut.new("#{@knife} data bag show #{databag}" +
-                               " --format json -c #{@config}").run_command
+                               " --format json #{@knife_verb_option} " +
+                               "-c #{@config}").run_command
       s.error!
-      db = JSON.load(s.stdout)
+      db = JSON.parse(s.stdout)
       if db.empty?
-        exec!("#{@knife} data bag delete #{databag} --yes -c #{@config}",
-              @logger)
+        exec!("#{@knife} data bag delete #{databag} --yes " +
+          "#{@knife_verb_option} -c #{@config}", @logger)
       end
     end
   end

data/lib/between_meals/repo.rb

@@ -27,7 +27,7 @@ module BetweenMeals
       @repo = nil
       @bin = nil
       setup
-    rescue
+    rescue StandardError
       @logger.warn("Unable to read repo from #{File.expand_path(repo_path)}")
       exit(1)
     end
@@ -54,7 +54,7 @@ module BetweenMeals
               logger.info("Repo found to be #{klass.to_s.split('::').last}")
               return r
             end
-          rescue
+          rescue StandardError
             logger.debug("Skipping #{klass}")
           end
         end

data/lib/between_meals/repo/git.rb

@@ -29,7 +29,7 @@ module BetweenMeals
         if File.exists?(File.expand_path(@repo_path))
           begin
             @repo = Rugged::Repository.new(File.expand_path(@repo_path))
-          rescue
+          rescue StandardError
             @repo = nil
           end
         else
@@ -60,7 +60,7 @@ module BetweenMeals
           {
             :message => msg,
             :update_ref => 'HEAD',
-          }
+          },
         )
       end
 
@@ -86,10 +86,10 @@ module BetweenMeals
         stdout = @cmd.diff(start_ref, end_ref).stdout
         begin
           parse_status(stdout).compact
-        rescue => e
+        rescue StandardError => e
           # We've seen some weird non-reproducible failures here
           @logger.error(
-            'Something went wrong. Please please report this output.'
+            'Something went wrong. Please report this output.',
           )
           @logger.error(e)
           stdout.lines.each do |line|
@@ -112,8 +112,9 @@ module BetweenMeals
         if @cmd.merge_base(rev, master).stdout.strip == rev
           return true
         end
+
         return false
-      rescue
+      rescue StandardError
         return false
       end
 
@@ -153,62 +154,63 @@ module BetweenMeals
 
         # rubocop:disable MultilineBlockChain
         changes.lines.map do |line|
-          case line
-          when /^A\s+(\S+)$/
+          parts = line.chomp.split("\t")
+          case parts[0]
+          when 'A'
             # A path
             {
               :status => :modified,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^C(?:\d*)\s+(\S+)\s+(\S+)/
+          when /^C(?:\d*)/
             # C<numbers> path1 path2
             {
               :status => :modified,
-              :path => Regexp.last_match(2)
+              :path => parts[2],
             }
-          when /^D\s+(\S+)$/
+          when 'D'
             # D path
             {
               :status => :deleted,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^M(?:\d*)\s+(\S+)$/
+          when /^M(?:\d*)/
             # M<numbers> path
             {
               :status => :modified,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^R(?:\d*)\s+(\S+)\s+(\S+)/
+          when /^R(?:\d*)/
             # R<numbers> path1 path2
             [
               {
                 :status => :deleted,
-                :path => Regexp.last_match(1)
+                :path => parts[1],
               },
               {
                 :status => :modified,
-                :path => Regexp.last_match(2)
-              }
+                :path => parts[2],
+              },
             ]
-          when /^T\s+(\S+)$/
+          when 'T'
             # T path
             [
               {
                 :status => :deleted,
-                :path => Regexp.last_match(1)
+                :path => parts[1],
               },
               {
                 :status => :modified,
-                :path => Regexp.last_match(1)
-              }
+                :path => parts[1],
+              },
             ]
           else
-            fail 'No match'
+            fail 'Failed to parse repo diff line.'
           end
         end.flatten.map do |x|
           {
             :status => x[:status],
-            :path => x[:path].sub("#{@repo_path}/", '')
+            :path => x[:path].sub("#{@repo_path}/", ''),
           }
         end
         # rubocop:enable MultilineBlockChain

data/lib/between_meals/repo/hg.rb

@@ -50,10 +50,10 @@ module BetweenMeals
         stdout = @cmd.status(start_ref, end_ref).stdout
         begin
           parse_status(stdout).compact
-        rescue => e
+        rescue StandardError => e
           # We've seen some weird non-reproducible failures here
           @logger.error(
-            'Something went wrong. Please please report this output.'
+            'Something went wrong. Please report this output.',
           )
           @logger.error(e)
           stdout.lines.each do |line|
@@ -65,9 +65,9 @@ module BetweenMeals
 
       def update
         @cmd.pull.stdout
-      rescue
+      rescue StandardError => e
         @logger.error('Something went wrong with hg!')
-        @logger.error(cmd.stdout)
+        @logger.error(e)
         raise
       end
 
@@ -83,7 +83,7 @@ module BetweenMeals
           :time => Time.parse(@cmd.log('date|isodate', 'master').stdout),
           :rev => @cmd.log('node', 'master').stdout,
         }]
-      rescue
+      rescue StandardError
         [{
           :time => nil,
           :rev => nil,
@@ -103,7 +103,7 @@ module BetweenMeals
 
       def last_msg
         @cmd.log('desc').stdout
-      rescue
+      rescue StandardError
         nil
       end
 
@@ -115,13 +115,13 @@ module BetweenMeals
 
       def email
         username[2]
-      rescue
+      rescue StandardError
         nil
       end
 
       def name
         username[1]
-      rescue
+      rescue StandardError
         nil
       end
 
@@ -143,7 +143,7 @@ module BetweenMeals
       def valid_ref?(ref)
         @cmd.rev(ref)
         return true
-      rescue
+      rescue StandardError
         raise Changeset::ReferenceError
       end
 
@@ -165,49 +165,48 @@ module BetweenMeals
         #  I = ignored
         #    = origin of the previous file (with --copies)
 
-        # rubocop:disable MultilineBlockChain
         changes.lines.map do |line|
-          case line
-          when /^A (\S+)$/
+          parts = line.chomp.split(nil, 2)
+          case parts[0]
+          when 'A'
             {
               :status => :added,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^C (\S+)$/
+          when 'C'
             {
               :status => :clean,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^R (\S+)$/
+          when 'R'
             {
               :status => :deleted,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^M (\S+)$/
+          when 'M'
             {
               :status => :modified,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^! (\S+)$/
+          when '!'
             {
               :status => :missing,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^\? (\S+)$/
+          when '?'
             {
               :status => :untracked,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
-          when /^I (\S+)$/
+          when 'I'
             {
               :status => :ignored,
-              :path => Regexp.last_match(1)
+              :path => parts[1],
             }
           else
-            fail 'No match'
+            fail 'Failed to parse repo diff line.'
           end
         end
-        # rubocop:enable MultilineBlockChain
       end
     end
   end

data/lib/between_meals/repo/hg/cmd.rb

@@ -50,7 +50,7 @@ module BetweenMeals
           begin
             f.write(msg)
             f.flush
-            cmd("commit --amend -l #{f.path}")
+            cmd("commit --amend --exclude '**' -l #{f.path}")
           ensure
             f.close
             f.unlink

data/lib/between_meals/repo/svn.rb

@@ -64,9 +64,9 @@ module BetweenMeals
 
         begin
           parse_status(changes).compact
-        rescue => e
+        rescue StandardError => e
           @logger.error(
-            'Something went wrong. Please please report this output.'
+            'Something went wrong. Please report this output.',
           )
           @logger.error(e)
           stdout.lines.each do |line|
@@ -88,12 +88,11 @@ module BetweenMeals
         end
       end
 
-      def upstream?
-      end
+      def upstream?; end
 
       def valid_ref?(ref)
         @cmd.info_r(ref, @repo_path)
-      rescue
+      rescue StandardError
         raise Changeset::ReferenceError
       end
 
@@ -103,13 +102,13 @@ module BetweenMeals
         # http://svnbook.red-bean.com/en/1.0/re26.html
         changes.lines.map do |line|
           case line
-          when (/^([\w ])\w?\s+(\S+)$/)
+          when /^([\w ])\w?\s+(.+)$/
             {
               :status => Regexp.last_match(1) == 'D' ? :deleted : :modified,
               :path => Regexp.last_match(2).sub("#{@repo_path}/", ''),
             }
           else
-            fail 'No match'
+            fail 'Failed to parse repo diff line.'
           end
         end
       end

data/lib/between_meals/util.rb

@@ -15,13 +15,15 @@
 # limitations under the License.
 
 require 'colorize'
+require 'net/http'
+require 'openssl'
 require 'socket'
 require 'timeout'
 
 module BetweenMeals
   # A set of simple utility functions used throughout BetweenMeals
   #
-  # Feel freeo to use... note that if you pass in a logger once
+  # Feel free to use... note that if you pass in a logger once
   # you don't need to again, but be safe and always pass one in. :)
 
   # Util classes need class vars :)
@@ -36,28 +38,29 @@ module BetweenMeals
       info("Executed in #{format('%.2f', Time.now - t0)}s")
     end
 
-    def exec!(command, logger = nil)
+    def exec!(command, logger = nil, stream = nil)
       @@logger = logger if logger
-      c = execute(command)
+      c = execute(command, stream)
       c.error!
       return c.status.exitstatus, c.stdout
     end
 
-    def exec(command, logger = nil)
+    def exec(command, logger = nil, stream = nil)
       @@logger = logger if logger
-      c = execute(command)
+
+      c = execute(command, stream)
       return c.status.exitstatus, c.stdout
     end
 
     private
 
     def info(msg)
-      @@logger.info(msg) if @@logger
+      @@logger&.info(msg)
     end
 
-    def execute(command)
+    def execute(command, stream)
       info("Running: #{command}")
-      c = Mixlib::ShellOut.new(command)
+      c = Mixlib::ShellOut.new(command, :live_stream => stream)
       c.run_command
       c.stdout.lines.each do |line|
         info("STDOUT: #{line.strip}")
@@ -69,20 +72,43 @@ module BetweenMeals
     end
 
     def port_open?(port)
-      begin
-        Timeout.timeout(1) do
-          begin
-            s = TCPSocket.new('localhost', port)
-            s.close
-            return true
-          rescue Errno::ECONNREFUSED, Errno::EHOSTUNREACH
-            return false
+      ips = Socket.ip_address_list
+      ips.map!(&:ip_address)
+      ips.each do |ip|
+        begin
+          Timeout.timeout(1) do
+            begin
+              s = TCPSocket.new(ip, port)
+              s.close
+              return true
+            rescue Errno::ECONNREFUSED, Errno::EHOSTUNREACH
+              next
+            end
+          end
+        rescue Timeout::Error
+          next
+        end
+      end
+      return false
+    end
+
+    def chef_zero_running?(port, use_ssl)
+      Timeout.timeout(1) do
+        begin
+          http = Net::HTTP.new('localhost', port)
+          if use_ssl
+            http.use_ssl = true
+            http.verify_mode = OpenSSL::SSL::VERIFY_NONE
           end
+          res = http.get('/')
+          return res['Server'] == 'chef-zero'
+        rescue StandardError
+          return false
         end
-      rescue Timeout::Error
-        return false
       end
+    rescue Timeout::Error
       return false
     end
   end
 end
+# rubocop:enable ClassVars

metadata

@@ -1,208 +1,175 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: between_meals
-version: !ruby/object:Gem::Version 
-  hash: 19
-  prerelease: 
-  segments: 
-  - 0
-  - 0
-  - 6
-  version: 0.0.6
+version: !ruby/object:Gem::Version
+  version: 0.0.11
 platform: ruby
-authors: 
+authors:
 - Phil Dibowitz
 - Marcin Sawicki
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2015-11-26 00:00:00 Z
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2020-05-28 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: colorize
-  prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: json
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  type: :runtime
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
   name: mixlib-shellout
-  prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id003
-- !ruby/object:Gem::Dependency 
-  name: rugged
   prerelease: false
-  requirement: &id004 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rugged
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :runtime
-  version_requirements: *id004
-- !ruby/object:Gem::Dependency 
-  name: rspec-core
   prerelease: false
-  requirement: &id005 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-core
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id005
-- !ruby/object:Gem::Dependency 
-  name: rspec-expectations
   prerelease: false
-  requirement: &id006 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-expectations
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id006
-- !ruby/object:Gem::Dependency 
-  name: rspec-mocks
   prerelease: false
-  requirement: &id007 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-mocks
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: '0'
   type: :development
-  version_requirements: *id007
-- !ruby/object:Gem::Dependency 
-  name: rubocop
   prerelease: false
-  requirement: &id008 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  type: :development
-  version_requirements: *id008
-- !ruby/object:Gem::Dependency 
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
   name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
   prerelease: false
-  requirement: &id009 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.49.1
   type: :development
-  version_requirements: *id009
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.49.1
 description: Library for calculation Chef differences between revisions
 email: 
 executables: []
-
 extensions: []
-
-extra_rdoc_files: 
+extra_rdoc_files:
 - README.md
 - LICENSE
-files: 
-- README.md
+files:
 - LICENSE
-- lib/between_meals/knife.rb
-- lib/between_meals/cmd.rb
-- lib/between_meals/util.rb
-- lib/between_meals/repo.rb
-- lib/between_meals/changeset.rb
+- README.md
 - lib/between_meals/changes/change.rb
-- lib/between_meals/changes/role.rb
 - lib/between_meals/changes/cookbook.rb
 - lib/between_meals/changes/databag.rb
-- lib/between_meals/repo/svn/cmd.rb
-- lib/between_meals/repo/hg/cmd.rb
-- lib/between_meals/repo/git/cmd.rb
-- lib/between_meals/repo/svn.rb
+- lib/between_meals/changes/role.rb
+- lib/between_meals/changeset.rb
+- lib/between_meals/cmd.rb
+- lib/between_meals/knife.rb
+- lib/between_meals/repo.rb
 - lib/between_meals/repo/git.rb
+- lib/between_meals/repo/git/cmd.rb
 - lib/between_meals/repo/hg.rb
+- lib/between_meals/repo/hg/cmd.rb
+- lib/between_meals/repo/svn.rb
+- lib/between_meals/repo/svn/cmd.rb
+- lib/between_meals/util.rb
 homepage: https://github.com/facebook/between-meals
-licenses: 
-- Apache
+licenses:
+- Apache-2.0
+metadata: {}
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.8.5
+rubygems_version: 2.7.6.2
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Between Meals
 test_files: []
-