squared 0.7.2 → 0.7.3

data/lib/squared/workspace/project/support/optionpartition.rb

@@ -49,10 +49,10 @@ module Squared
               ret.map! do |val|
                 next val if opt?(val)
 
-                if quote || val.is_a?(Pathname)
-                  shell_quote(val, force: force, double: double)
-                elsif escape
+                if !(pa = val.is_a?(Pathname)) && escape
                   shell_escape(val, quote: quote, double: double)
+                elsif quote || pa
+                  shell_quote(val, force: force, double: double)
                 else
                   val
                 end
@@ -153,7 +153,7 @@ module Squared
 
             def pattern?(val)
               val.start_with?('\A', '^') || val.end_with?('\z', '$') ||
-              val.match?(/[.)][*+?]|\(\?:|\\[dsw\d]|\[.+\]|\{\d+,?\d*\}/i)
+              val.match?(/[.)][*+?]|\(\?[:=!><]|\\[dsw\d]|\[.+\]|\{\d+,?\d*\}/i)
             end
 
             private
@@ -362,7 +362,7 @@ module Squared
                   push opt
                   skip = true if args
                 end
-                skip = true if first&.any? { |s| s.is_a?(Regexp) ? opt.match?(s) : !opt.include?(s) }
+                skip = true if first&.any? { |pat| pat.is_a?(Regexp) ? opt.match?(pat) : !opt.include?(pat) }
               end
             end
             @values = @values.empty? ? /\A\s+\z/ : /\A(#{@values.join('|')})#{sep}(.+)\z/m
@@ -388,7 +388,8 @@ module Squared
             self
           end
 
-          def append_any(*args, escape: false, quote: true, **kwargs)
+          def append_any(*args, escape: false, **kwargs)
+            quote = kwargs.fetch(:quote, true)
             (args.empty? ? extras : args.flatten(1)).each do |val|
               if block_given?
                 temp = val
@@ -396,7 +397,7 @@ module Squared
                 if val.is_a?(Array)
                   found << temp
                   k, v, q = val
-                  add_option(k, v, escape: escape, quote: quote, double: q == '"', merge: q == true, **kwargs)
+                  add_option(k, v, escape: escape, double: q == '"', merge: q == true, **kwargs)
                   next
                 end
               end
@@ -404,10 +405,10 @@ module Squared
 
               if exist?(val)
                 add_path(val, **kwargs)
-              elsif quote
-                add_quote(val, **kwargs)
               elsif escape
                 add shell_escape(val, **kwargs)
+              elsif quote
+                add_quote(val, **kwargs)
               else
                 add val
               end
@@ -521,7 +522,7 @@ module Squared
 
           def add_path(*args, option: nil, force: true, double: false, **kwargs)
             if args.empty?
-              list = select { |val| val.is_a?(String) }
+              list = grep(String)
               found.concat(list)
               list.map! { |val| path + val } if path
               append(list, force: force, **kwargs)
@@ -537,6 +538,7 @@ module Squared
           end
 
           def add_quote(*args, **kwargs)
+            kwargs.delete(:quote)
             merge(args.compact.map { |s| s == '--' || OptionPartition.opt?(s) ? s : shell_quote(s, **kwargs) })
             self
           end
@@ -546,8 +548,8 @@ module Squared
             self
           end
 
-          def add_first(fallback = nil, prefix: nil, path: false, escape: false, quote: false, reverse: false,
-                        expect: false, **kwargs)
+          def add_first(fallback = nil, prefix: nil, path: false, escape: false, reverse: false, expect: false,
+                        **kwargs)
             val = (reverse ? pop : shift) || fallback
             if val
               temp = val
@@ -555,10 +557,10 @@ module Squared
               unless block_given? && !(val = yield val).is_a?(String)
                 if path
                   add_path(val, **kwargs)
-                elsif quote
-                  add_quote(val, **kwargs)
                 elsif escape
                   add shell_escape(val, **kwargs)
+                elsif kwargs[:quote]
+                  add_quote(val, **kwargs)
                 else
                   add val
                 end
@@ -669,7 +671,7 @@ module Squared
           end
 
           def exist?(*args, add: false, first: false, last: false, glob: false)
-            return with_glob?(File.join(*args), glob) unless args.empty?
+            return !args.first.nil? && with_glob?(File.join(*args), glob) unless args.empty?
 
             if first || last
               return false unless (val = first ? self.first : self.last)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: squared
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.7.3
 platform: ruby
 authors:
 - An Pham