dapp 0.22.12 → 0.22.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9a58092038bf62ac6fa962e05416cfccc54e9824
-  data.tar.gz: 0ce6299538e16dd4c228a6d4e0851cf9af53c606
+  metadata.gz: e36c3fb64502f0633441843f5c691c9c865532d6
+  data.tar.gz: 3844c3e44040b7faf4a79dd9e67316da9f62f0be
 SHA512:
-  metadata.gz: e8261d17e445b75598da981e71d5f750a9ed41e060c3b0d7fa3a55129acee2d5359ea50d5f66948f39dd2642950123f82f00ff07e2b474e66f4027b82ff00b5d
-  data.tar.gz: eed6248d4a9c00b131dda6abcecd1fd4a4000bbafe1d437583e3e564e76cfd055154fff65258cdde4bacba82e3ea7ee26bbd802a2245636dc68c4873751e950e
+  metadata.gz: b84b16c368ca5c727004a516d6fb3d499eb7da3f4499748bea228b1d018b225e6417bee653f8d8fdf6f3267790276b05020e55818f5be7ba3e16f6f44d61a98d
+  data.tar.gz: ce8ba096db826404d1056b4e0c3ffc30f87e695015a95d8803a7c5eb536e0b69737fe892cbb126a25c0a6dea0eab7828134a4ca1a8fa6540e593a4930526c494

data/config/en/net_status.yml

@@ -81,7 +81,7 @@ en:
       stage_artifact_double_associate: "Can't use `%{stage}` stage for artifact; already used in `%{conflict_stage}` stage!"
       stage_artifact_not_supported_associated_stage: "Bad artifact stage `%{stage}`!"
       git_artifact_remote_branch_with_commit: "Remote git repo: use `commit` or `branch` directive!"
-      artifact_conflict: "Conflict between artifacts paths!"
+      artifact_conflict: "Conflict between artifacts!\n\n%{dappfile_context}"
       scratch_unsupported_directive: "Scratch dimg has unsupported directive `%{directive}`!"
       scratch_artifact_required: "Scratch dimg without artifacts!"
       scratch_artifact_associated: "Scratch artifact can't be associated: not expected `before`/`after` attribute!"

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

@@ -5,13 +5,6 @@ module Dapp
         class ArtifactDimg < Dimg
           def validate_scratch!
           end
-
-          def validate_artifacts_artifacts!
-          end
-
-          def validated_artifacts
-            _git_artifact._local + _git_artifact._remote
-          end
         end
       end
     end

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

@@ -136,6 +136,7 @@ module Dapp
 
             def artifacts_after_parsing!
               _artifacts_auto_excluding!
+              _artifact.map(&:_config).each(&:artifacts_after_parsing!)
             end
 
             protected

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

@@ -71,12 +71,39 @@ module Dapp
                 verifiable_artifact = artifacts.shift
                 artifacts.select { |a| a[:to] == verifiable_artifact[:to] }.each do |artifact|
                   next if verifiable_artifact[:index] == artifact[:index]
-                  validate_artifact!(verifiable_artifact, artifact)
-                  validate_artifact!(artifact, verifiable_artifact)
+                  begin
+                    validate_artifact!(verifiable_artifact, artifact)
+                    validate_artifact!(artifact, verifiable_artifact)
+                  rescue ::Dapp::Error::Config => e
+                    conflict_between_artifacts!(artifact, verifiable_artifact) if e.net_status[:code] == :artifact_conflict
+                    raise
+                  end
                 end
               end
             end
 
+            def conflict_between_artifacts!(*formatted_artifacts)
+              artifacts = formatted_artifacts.flatten.map { |formatted_artifact| formatted_artifact[:related_artifact] }
+              dappfile_context = artifacts.map do |artifact|
+                artifact_directive = []
+                artifact_directive << begin
+                  if artifact.is_a? Artifact::Export
+                    "artifact.export('#{artifact._cwd}') do"
+                  else
+                    "git#{"('#{artifact._url}')" if artifact.respond_to?(:_url)}.add('#{artifact._cwd}') do"
+                  end
+                end
+                [:include_paths, :exclude_paths].each do |directive|
+                  next if (paths = artifact.send("_#{directive}")).empty?
+                  artifact_directive << "  #{directive} '#{paths.join("', '")}'"
+                end
+                artifact_directive << "  to '#{artifact._to}'"
+                artifact_directive << 'end'
+                artifact_directive.join("\n")
+              end.join("\n\n")
+              raise ::Dapp::Error::Config, code: :artifact_conflict, data: { dappfile_context: dappfile_context }
+            end
+
             def validate_artifact_format(artifacts)
               artifacts.map do |a|
                 path_format = proc { |path| File.expand_path(File.join('/', path, '/'))[1..-1] }
@@ -104,7 +131,8 @@ module Dapp
                   index: artifacts.index(a),
                   to: to,
                   include_paths: include_paths,
-                  exclude_paths: exclude_paths
+                  exclude_paths: exclude_paths,
+                  related_artifact: a
                 }
               end
             end
@@ -113,17 +141,26 @@ module Dapp
               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.empty? && verifiable_artifact[:exclude_paths].empty? && raise(::Dapp::Error::Config, code: :artifact_conflict)
-              validate_artifact_path!(verifiable_artifact, artifact[:include_paths]) if verifiable_artifact[:include_paths].empty?
+              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
+              end
             end
 
             def validate_artifact_path!(verifiable_artifact, potential_conflicts)
-              potential_conflicts.all? do |path|
-                loop do
-                  break if verifiable_artifact[:exclude_paths].include?(path) || ((path = File.dirname(path)) == '.')
+              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
-                verifiable_artifact[:exclude_paths].include?(path)
-              end.tap { |res| res || raise(::Dapp::Error::Config, code: :artifact_conflict) }
+              end
             end
 
             def _associated_artifacts

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

@@ -20,7 +20,7 @@ module Dapp
           end
 
           def commit(value)
-            sub_directive_eval { @_commit = value }
+            sub_directive_eval { @_commit = value.to_s }
           end
 
           def _export
@@ -42,11 +42,11 @@ module Dapp
             end
 
             def branch(value)
-              sub_directive_eval { @_branch = value }
+              sub_directive_eval { @_branch = value.to_s }
             end
 
             def commit(value)
-              sub_directive_eval { @_commit = value }
+              sub_directive_eval { @_commit = value.to_s }
             end
 
             def validate!

data/lib/dapp/version.rb

@@ -1,4 +1,4 @@
 module Dapp
-  VERSION = '0.22.12'.freeze
+  VERSION = '0.22.13'.freeze
   BUILD_CACHE_VERSION = 26
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dapp
 version: !ruby/object:Gem::Version
-  version: 0.22.12
+  version: 0.22.13
 platform: ruby
 authors:
 - Dmitry Stolyarov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-15 00:00:00.000000000 Z
+date: 2018-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-shellout
@@ -719,7 +719,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 2.5.0
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: Build docker packaged apps using chef or shell