dapp 0.22.14 → 0.22.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c2a7b9af7bc533b0ab816bf4b5a6ebc22a3fd596
-  data.tar.gz: 9fe197382aec5851bcf0bd49c615328633d44c6d
+  metadata.gz: ada0cbea74d45cc601b5fc677937182f09402435
+  data.tar.gz: bca95e83b1a81f8955457afa53dc512196f96c87
 SHA512:
-  metadata.gz: f9a7cb1f32ff27e410723ce1b57558ff3ca02395d743c3dcecec1c5d408c2176ba5e557799f31cd8b41919f30c69fcaa29490c164d236636e493e4cc1a119bbd
-  data.tar.gz: bb9784a35025ffdd01cec49abbef022d76400f07772dd034066148d1a8619d91a79b66300a6c3c14cda825759882f6e8b148c73e146d424b961dec7c1eee864a
+  metadata.gz: c29702a13788871b7a6ffd099cf216902268e2bd5c73b2bdea3cb9f5ccf6ac478e7e8eb84e439d458982ec4ced276f01c235b80f870b3802837037b93d211f3b
+  data.tar.gz: 99c367343d1a038ad112e99fe6423f4921409b60fdf9aca06360212b7d6961969c270c0536f4cd74ee4f4e617420a27069c10c1a154a8195ab975953ccf0bbef

data/lib/dapp/dimg/config/directive/dimg/validation.rb

@@ -4,6 +4,8 @@ module Dapp
       module Directive
         class Dimg < Base
           module Validation
+            include Helper::Trivia
+
             def validate!
               directives_validate!
               validate_scratch!
@@ -138,29 +140,13 @@ module Dapp
             end
 
             def validate_artifact!(verifiable_artifact, artifact)
-              verifiable_artifact[:include_paths].each do |verifiable_path|
-                potential_conflicts = artifact[:include_paths].select { |path| path.start_with?(verifiable_path) }
-                validate_artifact_path!(verifiable_artifact, potential_conflicts)
-              end
-
-              if verifiable_artifact[:include_paths].empty?
-                if artifact[:include_paths].empty? || verifiable_artifact[:exclude_paths].empty?
-                  raise ::Dapp::Error::Config, code: :artifact_conflict
-                else
-                  validate_artifact_path!(verifiable_artifact, artifact[:include_paths])
-                end
+              cases = []
+              cases << verifiable_artifact[:include_paths].any? do |verifiable_path|
+                !ignore_path?(verifiable_path, paths: artifact[:include_paths], exclude_paths: artifact[:exclude_paths])
               end
-            end
+              cases << verifiable_artifact[:include_paths].empty? && artifact[:include_paths].empty?
 
-            def validate_artifact_path!(verifiable_artifact, potential_conflicts)
-              raise ::Dapp::Error::Config, code: :artifact_conflict unless begin
-                potential_conflicts.all? do |path|
-                  loop do
-                    break if verifiable_artifact[:exclude_paths].include?(path) || ((path = File.dirname(path)) == '.')
-                  end
-                  verifiable_artifact[:exclude_paths].include?(path)
-                end
-              end
+              raise ::Dapp::Error::Config, code: :artifact_conflict if cases.any?
             end
 
             def _associated_artifacts

data/lib/dapp/dimg/git_repo/base.rb

@@ -3,6 +3,8 @@ module Dapp
     module GitRepo
       # Base class for any Git repo (remote, gitkeeper, etc)
       class Base
+        include Helper::Trivia
+
         attr_reader :name
 
         def initialize(manager, name)
@@ -132,32 +134,6 @@ module Dapp
         def git(**kwargs)
           @git ||= Rugged::Repository.new(path.to_s, **kwargs)
         end
-
-        private
-
-        def ignore_path?(path, paths: [], exclude_paths: [])
-          is_exclude_path = exclude_paths.any? { |p| check_path?(path, p) }
-          is_include_path = begin
-            paths.empty? ||
-              paths.any? do |p|
-                File.fnmatch?(p, path, File::FNM_PATHNAME) ||
-                  File.fnmatch?(File.join(p, '**', '*'), path, File::FNM_PATHNAME)
-              end
-          end
-
-          is_exclude_path || !is_include_path
-        end
-
-        def check_path?(path, format)
-          path_parts = path.split('/')
-          checking_path = nil
-
-          until path_parts.empty?
-            checking_path = [checking_path, path_parts.shift].compact.join('/')
-            return true if File.fnmatch?(format, checking_path, File::FNM_PATHNAME)
-          end
-          false
-        end
       end
     end
   end

data/lib/dapp/helper/trivia.rb

@@ -28,6 +28,41 @@ module Dapp
         Pathname.new(File.join(base.to_s, *path.compact.map(&:to_s)))
       end
 
+      def ignore_path?(path, paths: [], exclude_paths: [])
+        ignore_path_base(path, exclude_paths: exclude_paths) do
+          paths.empty? ||
+            paths.any? do |p|
+              File.fnmatch?(p, path, File::FNM_PATHNAME|File::FNM_DOTMATCH) ||
+                File.fnmatch?(File.join(p, '**', '*'), path, File::FNM_PATHNAME|File::FNM_DOTMATCH)
+            end
+        end
+      end
+
+      def ignore_path_base(path, exclude_paths: [])
+        is_exclude_path = exclude_paths.any? { |p| check_path?(path, p) }
+        is_include_path = yield
+        is_exclude_path || !is_include_path
+      end
+
+      def check_path?(path, format)
+        path_checker(path) { |checking_path| File.fnmatch(format, checking_path, File::FNM_PATHNAME|File::FNM_DOTMATCH) }
+      end
+
+      def check_subpath?(path, format)
+        path_checker(format) { |checking_path| File.fnmatch(checking_path, path, File::FNM_PATHNAME|File::FNM_DOTMATCH) }
+      end
+
+      def path_checker(path)
+        path_parts = path.split('/')
+        checking_path = nil
+
+        until path_parts.empty?
+          checking_path = [checking_path, path_parts.shift].compact.join('/')
+          return true if yield checking_path
+        end
+        false
+      end
+
       def self.class_to_lowercase(class_name = self)
         class_name.to_s.split('::').last.split(/(?=[[:upper:]]|[0-9])/).join('_').downcase.to_s
       end

data/lib/dapp/kube/dapp/command/render.rb

@@ -13,8 +13,8 @@ module Dapp
                       template_relative_path_pattern = Pathname(File.expand_path(template_path_pattern)).subpath_of(path('.helm'))
                       template_relative_path_pattern ||= template_path_pattern
 
-                      File.fnmatch?(template_relative_path_pattern, template_path_without_chart_name, File::FNM_PATHNAME) ||
-                        File.fnmatch?(template_relative_path_pattern, template_path, File::FNM_PATHNAME)
+                      File.fnmatch?(template_relative_path_pattern, template_path_without_chart_name, File::FNM_PATHNAME|File::FNM_DOTMATCH) ||
+                        File.fnmatch?(template_relative_path_pattern, template_path, File::FNM_PATHNAME|File::FNM_DOTMATCH)
                     end
                   end
                 else

data/lib/dapp/version.rb

@@ -1,4 +1,4 @@
 module Dapp
-  VERSION = '0.22.14'.freeze
-  BUILD_CACHE_VERSION = 26
+  VERSION = '0.22.15'.freeze
+  BUILD_CACHE_VERSION = 26.1
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dapp
 version: !ruby/object:Gem::Version
-  version: 0.22.14
+  version: 0.22.15
 platform: ruby
 authors:
 - Dmitry Stolyarov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-25 00:00:00.000000000 Z
+date: 2018-02-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-shellout