prick 0.27.2 → 0.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d567ea292c77fde8d238f35dad37e284917bb54bdcfa044e9916fd771686c82c
-  data.tar.gz: df8f4285e9d53fa9c907e1c43aa4a587d7b6709cda42540f1e1ba6092caf8bce
+  metadata.gz: '022876c913aadf3a2b42646f514f51c2db481c32db42a5903f2dbce88303d423'
+  data.tar.gz: 1772f43a5a99fe93b97e748488f3c5bfa93fddce768176744ebb27e983cbb475
 SHA512:
-  metadata.gz: aaca4641608807b55e8154db7d79aff8314b88aade591e0d5fd5920b9ada1984f9ecf07a1098d40a1afd3578dc51915b40bea334bd5eb4b36b33e45bb1d45c25
-  data.tar.gz: 2e72149f29ca87b21f407491f3bb40e5bdf1bbda10908b5df25ac4f432d3776b2510e77592ad5cda45c6bb55447091cb53709788869f10bbb6a97403685bc28d
+  metadata.gz: d8c647cc168aee7f0aab47d39d6ce1050331a06b6aa7a04fc8173152618432c377eb7ce42784eb95d2a70134be10bddd0ff3a43298c2e20af8a3649dc8227451
+  data.tar.gz: 84068d08a68d2b1177054faa4d31218bf7bc66cf437bf255a6ff2404c0126e10288e58519e11fe7c80029c7169ff9e3a59be5327c39f8fd112bc0a846519d833

data/lib/builder/parser.rb

@@ -62,11 +62,11 @@ module Prick
                   when "seed"; unit.seed_nodes
                 else
                   raise Error, "Illegal key in #{unit.path}: #{key}"
-                end.concat(Array(value).map { |value| parse_entry(unit, key.to_sym, dir, value) })
+                end.concat(Array(value).map { |value| parse_entry(unit, key.to_sym, dir, value) }.compact)
               end
             }
           else
-            node = parse_entry(unit, :decl, dir, entry)
+            node = parse_entry(unit, :decl, dir, entry) or next
             if node.kind == :fox
               unit.seed_nodes << node
             else
@@ -77,11 +77,11 @@ module Prick
         unit
       end
 
-      # Expand $ENVIRONMENT variable. The function implements a hierarchy of
-      # environments:
+      # Expand $ENVIRONMENT variable in file names. The function implements a
+      # hierarchy of environments:
       #
       #   production
-      #     online
+      #     (online)
       #   development
       #     online
       #     offline
@@ -92,19 +92,26 @@ module Prick
       # If an environment doesn't exist in #dir, then the higher level
       # environments are tried in turn. Eg. if the current environment is
       # 'test', the algorithm expands '$ENVIRONMENT' to 'test', 'offline', and
-      # 'development' until an existing file is found. If no file is found, the
-      # algorithm returns the unexpanded value
+      # 'development' until an existing file is found. If no file is found the
+      # the unexpanded value is returned
+      #
+      # NOTE: This is a hardcoded feature for an internal project. It is
+      #       configurable in the next major version
       #
-      def expand_environment(dir, name)
-        env = Prick.state.environment.to_s
-        file = name
-        if !File.exist?("#{dir}/#{file}") && name =~ /^(.*?)\$ENVIRONMENT/
+      def expand_filename(dir, filename)
+        if File.exist?("#{dir}/#{filename}")
+          return filename
+        elsif filename =~ /\$ENVIRONMENT|\$\{ENVIRONMENT\}/
+          env = Prick.state.environment.to_s
           while true
-            file = expand_variables(name, ENVIRONMENT: env)
-            break if File.exist? "#{dir}/#{file}"
+            file = expand_variables(filename, ENVIRONMENT: env, PWD: Dir.getwd)
+            return file if File.exist? "#{dir}/#{file}"
             case env
-              when "production"; break
-              when "development"; break
+              when "production"
+                file = expand_variables(filename, ENVIRONMENT: "online", PWD: Dir.getwd) or return nil
+                return file if File.exist? "#{dir}/#{file}"
+                return nil
+              when "development"; return nil
               when "online"; env = "development"
               when "offline"; env = "development"
               when "backend"; env = "offline"
@@ -114,43 +121,64 @@ module Prick
               raise Error, "Illegal env: '#{env}'"
             end
           end
+        else
+          return nil
         end
-        return file
       end
 
-      def parse_file_entry(dir, entry)
-        name = expand_environment(dir, entry)
-        name.sub!(/\/$/, "")
-        if name =~ /^(\S+)\s+(.+)$/ # has arguments -> exe
-          file = $1
-          args = $2.split
+      # Expand $ENVIRONMENT variable
+      def expand_string(string)
+        expand_variables(string, ENVIRONMENT: Prick.state.environment.to_s, PWD: Dir.getwd)
+      end
+
+      def parse_file_entry(unit, dir, entry)
+        entry = entry.sub(/\/$/, "")
+        if entry =~ /^(\S+?)(\?)?(?:\s+(.+))?\s*$/
+          filename = $1
+          optional = !$2.nil?
+          args = expand_string($3 || '').split
+          file = expand_filename(dir, filename) 
+          file || optional or raise Error, "Can't find #{filename} in #{dir} from #{unit}"
+          !file.nil? or return nil
         else
-          file = name
+          raise Error, "Not a file name: '#{entry}'"
         end
         path = "#{dir}/#{file}"
-        [path, name, file, args]
+        [path, entry, file, args].flatten
       end
 
+#     def parse_file_entry(dir, entry)
+#       name = expand_environment(dir, entry)
+#       name.sub!(/\/$/, "")
+#       if name =~ /^(\S+)\s+(.+)$/ # has arguments -> exe
+#         file = $1
+#         args = $2.split
+#       else
+#         file = name
+#       end
+#       path = "#{dir}/#{file}"
+#       [path, name, file, args]
+#     end
+
       def parse_entry(unit, phase, dir, entry)
         if entry.is_a?(Hash)
           entry.size == 1 or raise Error, "sql and module are single-line values"
           key, value = entry.first
           case key
             when "sql"
-              InlineNode.new(unit, phase, unit.path, value)
-            when "call" 
-              args = value.split(/\s+/)
-              args.size >= 3 or raise "Illegal number of arguments: #{value}"
-              file, klass, command, args = *args
+              InlineNode.new(unit, phase, unit.path, expand_string(value))
+            when "call"
+              (path, name, file, klass, command, args = parse_file_entry(unit, dir, value)) or return nil
+              klass && command or raise "Illegal number of arguments: #{value}"
               ModuleNode.new(unit, phase, "#{dir}/#{file}", klass, command, args)
             when "exec"
-              path, name, file, args = parse_file_entry(dir, value)
+              (path, name, file, args = parse_file_entry(unit, dir, value)) or return nil
               ExeNode.new(unit, phase, path, args)
           else
             raise Error, "Illegal key: #{key}"
           end
         else
-          path, name, file, args = parse_file_entry(dir, entry)
+          (path, name, file, args = parse_file_entry(unit, dir, entry)) or return nil
           if File.directory? path
             parse_directory(unit, path)
           elsif File.file? path

data/lib/prick/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Prick
-  VERSION = "0.27.2"
+  VERSION = "0.28.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: prick
 version: !ruby/object:Gem::Version
-  version: 0.27.2
+  version: 0.28.0
 platform: ruby
 authors:
 - Claus Rasmussen
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-01-06 00:00:00.000000000 Z
+date: 2024-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: semantic
@@ -231,7 +231,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.18
+rubygems_version: 3.3.7
 signing_key:
 specification_version: 4
 summary: A release control and management system for postgresql