lino 1.2.0.pre.1 → 1.2.0.pre.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f400d67549dca909bceb3ca4596ae999a1c9f29cee17ef113e445f4a481163f2
-  data.tar.gz: 10252b5eadc145a150fcffcc121988c98567da08dc1c51f8d261c58d9387e026
+  metadata.gz: 638e1715f758c7cf7021e6fc6e6222fd87313073b1041bd0f685405a7eb37d54
+  data.tar.gz: c2e681493e5d8d0470013c90f9df879b6dec19bb97ac81ba4ade7b3c4d2f5c23
 SHA512:
-  metadata.gz: e6dc5a7ee780de4de6f3576751dca285816bfe21ded3329804e0983466ea4db4d406793672f63d7b45e46f3445e310b396e081283038d2a655275f61bbe65568
-  data.tar.gz: 19c1769b9fce8be5e49be853d6974ac704853b205a6979ffb836a5acd189191640b00428a625e3b5ae62f3dadde8131567405ca5564a1b6da42706a0a14c8d11
+  metadata.gz: 3defa65d401ed798c06399ef7a68ea1dad267b0982db7d177e4d3a527202fcccbea1a7f322358ecec9ee7de849d65840d5acb77faa7a4e8b91e943ea50026a0b
+  data.tar.gz: 61dea9a9cafc9ba91df99690192e0e165e55cefeaaffade628ec7e48418cad393681d1b505533b38552246bb524f94e1ec134917c219261e1a049f8bba341fc2

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    lino (1.2.0.pre.1)
+    lino (1.2.0.pre.2)
       hamster (~> 3.0)
       open4 (~> 1.3)
 

data/lib/lino/command_line_builder.rb

@@ -8,7 +8,7 @@ module Lino
     include Lino::Utilities
 
     class <<self
-      def for_command command
+      def for_command(command)
         CommandLineBuilder.new(command: command)
       end
     end
@@ -19,13 +19,15 @@ module Lino
         switches: [],
         arguments: [],
         environment_variables: [],
-        option_separator: ' ')
+        option_separator: ' ',
+        option_quoting: nil)
       @command = command
       @subcommands = Hamster::Vector.new(subcommands)
       @switches = Hamster::Vector.new(switches)
       @arguments = Hamster::Vector.new(arguments)
       @environment_variables = Hamster::Vector.new(environment_variables)
       @option_separator = option_separator
+      @option_quoting = option_quoting
     end
 
     def with_subcommand(subcommand, &block)
@@ -33,14 +35,22 @@ module Lino
           SubcommandBuilder.for_subcommand(subcommand))))
     end
 
-    def with_option(switch, value, separator: nil)
-      with(switches: @switches.add({components: [switch, value], separator: separator}))
+    def with_option(switch, value, separator: nil, quoting: nil)
+      with(switches: @switches.add({
+          components: [switch, value],
+          separator: separator,
+          quoting: quoting
+      }))
     end
 
     def with_option_separator(option_separator)
       with(option_separator: option_separator)
     end
 
+    def with_option_quoting(character)
+      with(option_quoting: character)
+    end
+
     def with_flag(flag)
       with(switches: @switches.add({components: [flag]}))
     end
@@ -55,10 +65,15 @@ module Lino
 
     def build
       components = [
-          map_and_join(@environment_variables) { |var| "#{var[0]}=\"#{var[1]}\"" },
+          map_and_join(@environment_variables) { |var|
+            "#{var[0]}=\"#{var[1]}\""
+          },
           @command,
-          map_and_join(@switches, &join_with(@option_separator)),
-          map_and_join(@subcommands) { |sub| sub.build(@option_separator)},
+          map_and_join(@switches,
+              &(quote_with(@option_quoting) >> join_with(@option_separator))),
+          map_and_join(@subcommands) { |sub|
+            sub.build(@option_separator, @option_quoting)
+          },
           map_and_join(@arguments, &join_with(' '))
       ]
 
@@ -82,7 +97,8 @@ module Lino
           switches: @switches,
           arguments: @arguments,
           environment_variables: @environment_variables,
-          option_separator: @option_separator
+          option_separator: @option_separator,
+          option_quoting: @option_quoting
       }
     end
   end

data/lib/lino/subcommand_builder.rb

@@ -18,18 +18,23 @@ module Lino
       @switches = Hamster::Vector.new(switches)
     end
 
-    def with_option(switch, value, separator: nil)
-      with(switches: @switches.add({components: [switch, value], separator: separator}))
+    def with_option(switch, value, separator: nil, quoting: nil)
+      with(switches: @switches.add({
+          components: [switch, value],
+          separator: separator,
+          quoting: quoting
+      }))
     end
 
     def with_flag(flag)
       with(switches: @switches.add({components: [flag]}))
     end
 
-    def build(option_separator)
+    def build(option_separator, option_quoting)
       components = [
           @subcommand,
-          map_and_join(@switches, &join_with(option_separator))
+          map_and_join(@switches,
+              &(quote_with(option_quoting) >> join_with(option_separator)))
       ]
 
       components

data/lib/lino/utilities.rb

@@ -1,11 +1,29 @@
+require 'pp'
+
 module Lino
   module Utilities
     def map_and_join(collection, &block)
       collection.map { |item| block.call(item) }.join(' ')
     end
 
-    def join_with(separator)
-      lambda { |item| item[:components].join(item[:separator] || separator) }
+    def join_with(global_separator)
+      lambda do |item|
+        item[:components].join(item[:separator] || global_separator)
+      end
+    end
+
+    def quote_with(global_character)
+      lambda do |item|
+        character = item[:quoting] || global_character
+        components = item[:components]
+        switch = components[0]
+        value = components[1]
+
+        item.merge(
+            components: (components.count > 1) ?
+                [switch, "#{character}#{value}#{character}"] :
+                components)
+      end
     end
   end
 end

data/lib/lino/version.rb

@@ -1,3 +1,3 @@
 module Lino
-  VERSION = '1.2.0.pre.1'
+  VERSION = '1.2.0.pre.2'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lino
 version: !ruby/object:Gem::Version
-  version: 1.2.0.pre.1
+  version: 1.2.0.pre.2
 platform: ruby
 authors:
 - Toby Clemson