cocoapods 1.5.3 → 1.6.0.beta.1

data/lib/cocoapods/user_interface.rb

@@ -62,6 +62,7 @@ module Pod
         self.indentation_level += relative_indentation
         self.title_level += 1
         yield if block_given?
+      ensure
         self.indentation_level -= relative_indentation
         self.title_level -= 1
       end
@@ -83,6 +84,7 @@ module Pod
         self.indentation_level += relative_indentation
         self.title_level += 1
         yield if block_given?
+      ensure
         self.indentation_level -= relative_indentation
         self.title_level -= 1
       end
@@ -114,6 +116,7 @@ module Pod
         self.indentation_level += relative_indentation
         self.title_level += 1
         yield if block_given?
+      ensure
         self.indentation_level -= relative_indentation
         self.title_level -= 1
       end
@@ -140,6 +143,7 @@ module Pod
 
         self.indentation_level += relative_indentation
         yield if block_given?
+      ensure
         self.indentation_level -= relative_indentation
       end
 
@@ -160,6 +164,7 @@ module Pod
         self.indentation_level += 2
         @treat_titles_as_messages = true
         yield if block_given?
+      ensure
         @treat_titles_as_messages = false
         self.indentation_level -= 2
       end

data/lib/cocoapods/validator.rb

@@ -18,6 +18,10 @@ module Pod
     #
     DEFAULT_SWIFT_VERSION = '3.2'.freeze
 
+    # The valid platforms for linting
+    #
+    VALID_PLATFORMS = Platform.all.freeze
+
     # @return [Specification::Linter] the linter instance from CocoaPods
     #         Core.
     #
@@ -31,13 +35,28 @@ module Pod
     # @param  [Array<String>] source_urls
     #         the Source URLs to use in creating a {Podfile}.
     #
-    def initialize(spec_or_path, source_urls)
+    # @param. [Array<String>] platforms
+    #         the platforms to lint.
+    #
+    def initialize(spec_or_path, source_urls, platforms = [])
       @linter = Specification::Linter.new(spec_or_path)
       @source_urls = if @linter.spec && @linter.spec.dependencies.empty? && @linter.spec.recursive_subspecs.all? { |s| s.dependencies.empty? }
                        []
                      else
                        source_urls.map { |url| config.sources_manager.source_with_name_or_url(url) }.map(&:url)
                      end
+
+      @platforms = platforms.map do |platform|
+        result =  case platform.to_s.downcase
+                  # Platform doesn't recognize 'macos' as being the same as 'osx' when initializing
+                  when 'macos' then Platform.macos
+                  else Platform.new(platform, nil)
+                  end
+        unless valid_platform?(result)
+          raise Informative, "Unrecognized platform `#{platform}`. Valid platforms: #{VALID_PLATFORMS.join(', ')}"
+        end
+        result
+      end
     end
 
     #-------------------------------------------------------------------------#
@@ -55,6 +74,28 @@ module Pod
       @linter.file
     end
 
+    # Returns a list of platforms to lint for a given Specification
+    #
+    # @param [Specification] spec
+    #         The specification to lint
+    #
+    # @return [Array<Platform>] platforms to lint for the given specification
+    #
+    def platforms_to_lint(spec)
+      return spec.available_platforms if @platforms.empty?
+
+      # Validate that the platforms specified are actually supported by the spec
+      results = @platforms.map do |platform|
+        matching_platform = spec.available_platforms.find { |p| p.name == platform.name }
+        unless matching_platform
+          raise Informative, "Platform `#{platform}` is not supported by specification `#{spec}`."
+        end
+        matching_platform
+      end.uniq
+
+      results
+    end
+
     # @return [Sandbox::FileAccessor] the file accessor for the spec.
     #
     attr_accessor :file_accessor
@@ -202,6 +243,10 @@ module Pod
     #
     attr_accessor :use_frameworks
 
+    # @return [Boolean] Whether modular headers should be used for the installation.
+    #
+    attr_accessor :use_modular_headers
+
     # @return [Boolean] Whether attributes that affect only public sources
     #         Bool be skipped.
     #
@@ -309,7 +354,9 @@ module Pod
       validate_documentation_url(spec)
       validate_source_url(spec)
 
-      valid = spec.available_platforms.send(fail_fast ? :all? : :each) do |platform|
+      platforms = platforms_to_lint(spec)
+
+      valid = platforms.send(fail_fast ? :all? : :each) do |platform|
         UI.message "\n\n#{spec} - Analyzing on #{platform} platform.".green.reversed
         @consumer = spec.consumer(platform)
         setup_validation_environment
@@ -464,7 +511,7 @@ module Pod
     end
 
     def download_pod
-      podfile = podfile_from_spec(consumer.platform_name, deployment_target, use_frameworks, consumer.spec.test_specs.map(&:name))
+      podfile = podfile_from_spec(consumer.platform_name, deployment_target, use_frameworks, consumer.spec.test_specs.map(&:name), use_modular_headers)
       sandbox = Sandbox.new(config.sandbox_root)
       @installer = Installer.new(sandbox, podfile)
       @installer.use_default_plugins = false
@@ -475,7 +522,8 @@ module Pod
 
     def create_app_project
       app_project = Xcodeproj::Project.new(validation_dir + 'App.xcodeproj')
-      Pod::Generator::AppTargetHelper.add_app_target(app_project, consumer.platform_name, deployment_target)
+      app_target = Pod::Generator::AppTargetHelper.add_app_target(app_project, consumer.platform_name, deployment_target)
+      Pod::Generator::AppTargetHelper.add_swift_version(app_target, swift_version)
       app_project.save
       app_project.recreate_user_schemes
     end
@@ -484,9 +532,9 @@ module Pod
       app_project = Xcodeproj::Project.open(validation_dir + 'App.xcodeproj')
       app_target = app_project.targets.first
       pod_target = @installer.pod_targets.find { |pt| pt.pod_name == spec.root.name }
-      Pod::Generator::AppTargetHelper.add_app_project_import(app_project, app_target, pod_target, consumer.platform_name, use_frameworks)
-      Pod::Generator::AppTargetHelper.add_swift_version(app_target, swift_version)
+      Pod::Generator::AppTargetHelper.add_app_project_import(app_project, app_target, pod_target, consumer.platform_name)
       Pod::Generator::AppTargetHelper.add_xctest_search_paths(app_target) if @installer.pod_targets.any? { |pt| pt.spec_consumers.any? { |c| c.frameworks.include?('XCTest') } }
+      Pod::Generator::AppTargetHelper.add_empty_swift_file(app_project, app_target) if @installer.pod_targets.any?(&:uses_swift?)
       app_project.save
       Xcodeproj::XCScheme.share_scheme(app_project.path, 'App')
       # Share the pods xcscheme only if it exists. For pre-built vendored pods there is no xcscheme generated.
@@ -501,43 +549,33 @@ module Pod
          perform_post_install_actions).each { |m| @installer.send(m) }
 
       deployment_target = spec.subspec_by_name(subspec_name).deployment_target(consumer.platform_name)
-      configure_pod_targets(@installer.aggregate_targets, deployment_target)
+      configure_pod_targets(@installer.aggregate_targets, @installer.target_installation_results, deployment_target)
       @installer.pods_project.save
     end
 
-    def configure_pod_targets(targets, deployment_target)
-      test_only_pod_targets = @installer.pod_targets - targets.flat_map(&:pod_targets)
-      targets.each do |target|
-        target.pod_targets.each do |pod_target|
-          update_pod_target_build_settings(pod_target)
-          if pod_target.uses_swift?
-            pod_target.test_native_targets.each do |test_native_target|
-              test_native_target.build_configuration_list.build_configurations.each do |build_configuration|
-                build_configuration.build_settings['SWIFT_VERSION'] = swift_version
-              end
+    def configure_pod_targets(targets, target_installation_results, deployment_target)
+      target_installation_results.first.values.each do |pod_target_installation_result|
+        pod_target = pod_target_installation_result.target
+        native_target = pod_target_installation_result.native_target
+        native_target.build_configuration_list.build_configurations.each do |build_configuration|
+          (build_configuration.build_settings['OTHER_CFLAGS'] ||= '$(inherited)') << ' -Wincomplete-umbrella'
+          build_configuration.build_settings['SWIFT_VERSION'] = (pod_target.swift_version || swift_version) if pod_target.uses_swift?
+        end
+        pod_target_installation_result.test_specs_by_native_target.each do |test_native_target, test_specs|
+          if pod_target.uses_swift_for_test_spec?(test_specs.first)
+            test_native_target.build_configuration_list.build_configurations.each do |build_configuration|
+              build_configuration.build_settings['SWIFT_VERSION'] = swift_version
             end
           end
         end
+      end
+      targets.each do |target|
         if target.pod_targets.any?(&:uses_swift?) && consumer.platform_name == :ios &&
             (deployment_target.nil? || Version.new(deployment_target).major < 8)
           uses_xctest = target.spec_consumers.any? { |c| (c.frameworks + c.weak_frameworks).include? 'XCTest' }
           error('swift', 'Swift support uses dynamic frameworks and is therefore only supported on iOS > 8.') unless uses_xctest
         end
       end
-      # Wire up Swift version to pod targets used only by tests.
-      test_only_pod_targets.each do |test_pod_target|
-        update_pod_target_build_settings(test_pod_target)
-      end
-    end
-
-    def update_pod_target_build_settings(pod_target)
-      native_target = pod_target.native_target
-      unless native_target.nil?
-        native_target.build_configuration_list.build_configurations.each do |build_configuration|
-          (build_configuration.build_settings['OTHER_CFLAGS'] ||= '$(inherited)') << ' -Wincomplete-umbrella'
-          build_configuration.build_settings['SWIFT_VERSION'] = (pod_target.swift_version || swift_version) if pod_target.uses_swift?
-        end
-      end
     end
 
     def validate_vendored_dynamic_frameworks
@@ -597,7 +635,7 @@ module Pod
         UI.message "\nTesting with `xcodebuild`.\n".yellow do
           pod_target = @installer.pod_targets.find { |pt| pt.pod_name == spec.root.name }
           consumer.spec.test_specs.each do |test_spec|
-            scheme = pod_target.native_target_for_spec(test_spec)
+            scheme = @installer.target_installation_results.first[pod_target.name].native_target_for_spec(test_spec)
             output = xcodebuild('test', scheme, 'Debug')
             parsed_output = parse_xcodebuild_output(output)
             translate_output_to_linter_messages(parsed_output)
@@ -624,10 +662,8 @@ module Pod
       FILE_PATTERNS.each do |attr_name|
         if respond_to?("_validate_#{attr_name}", true)
           send("_validate_#{attr_name}")
-        end
-
-        if !file_accessor.spec_consumer.send(attr_name).empty? && file_accessor.send(attr_name).empty?
-          error('file patterns', "The `#{attr_name}` pattern did not match any file.")
+        else
+          validate_nonempty_patterns(attr_name, :error)
         end
       end
 
@@ -638,12 +674,26 @@ module Pod
       end
     end
 
+    # Validates that the file patterns in `attr_name` match at least 1 file.
+    #
+    # @param [String,Symbol] attr_name the name of the attribute to check (ex. :public_header_files)
+    #
+    # @param [String,Symbol] message_type the type of message to send if the patterns are empty (ex. :error)
+    #
+    def validate_nonempty_patterns(attr_name, message_type)
+      return unless !file_accessor.spec_consumer.send(attr_name).empty? && file_accessor.send(attr_name).empty?
+
+      add_result(message_type, 'file patterns', "The `#{attr_name}` pattern did not match any file.")
+    end
+
     def _validate_private_header_files
       _validate_header_files(:private_header_files)
+      validate_nonempty_patterns(:private_header_files, :warning)
     end
 
     def _validate_public_header_files
       _validate_header_files(:public_header_files)
+      validate_nonempty_patterns(:public_header_files, :warning)
     end
 
     def _validate_license
@@ -801,27 +851,30 @@ module Pod
     # @note   The generated podfile takes into account whether the linter is
     #         in local mode.
     #
-    def podfile_from_spec(platform_name, deployment_target, use_frameworks = true, test_spec_names = [])
+    def podfile_from_spec(platform_name, deployment_target, use_frameworks = true, test_spec_names = [], use_modular_headers = false)
       name     = subspec_name || spec.name
       podspec  = file.realpath
       local    = local?
       urls     = source_urls
       Pod::Podfile.new do
         install! 'cocoapods', :deterministic_uuids => false
+        # By default inhibit warnings for all pods, except the one being validated.
+        inhibit_all_warnings!
         urls.each { |u| source(u) }
         target 'App' do
           use_frameworks!(use_frameworks)
+          use_modular_headers! if use_modular_headers
           platform(platform_name, deployment_target)
           if local
-            pod name, :path => podspec.dirname.to_s
+            pod name, :path => podspec.dirname.to_s, :inhibit_warnings => false
           else
-            pod name, :podspec => podspec.to_s
+            pod name, :podspec => podspec.to_s, :inhibit_warnings => false
           end
           test_spec_names.each do |test_spec_name|
             if local
-              pod test_spec_name, :path => podspec.dirname.to_s
+              pod test_spec_name, :path => podspec.dirname.to_s, :inhibit_warnings => false
             else
-              pod test_spec_name, :podspec => podspec.to_s
+              pod test_spec_name, :podspec => podspec.to_s, :inhibit_warnings => false
             end
           end
         end
@@ -889,6 +942,45 @@ module Pod
       Executable.execute_command('xcodebuild', command, raise_on_failure)
     end
 
+    # Whether the platform with the specified name is valid
+    #
+    # @param  [Platform] platform
+    #         The platform to check
+    #
+    # @return [Bool] True if the platform is valid
+    #
+    def valid_platform?(platform)
+      VALID_PLATFORMS.any? { |p| p.name == platform.name }
+    end
+
+    # Whether the platform is supported by the specification
+    #
+    # @param  [Platform] platform
+    #         The platform to check
+    #
+    # @param  [Specification] specification
+    #         The specification which must support the provided platform
+    #
+    # @return [Bool] Whether the platform is supported by the specification
+    #
+    def supported_platform?(platform, spec)
+      available_platforms = spec.available_platforms
+
+      available_platforms.any? { |p| p.name == platform.name }
+    end
+
+    # Whether the provided name matches the platform
+    #
+    # @param  [Platform] platform
+    #         The platform
+    #
+    # @param  [String] name
+    #         The name to check against the provided platform
+    #
+    def platform_name_match?(platform, name)
+      [platform.name, platform.string_name].any? { |n| n.casecmp(name) == 0 }
+    end
+
     #-------------------------------------------------------------------------#
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cocoapods
 version: !ruby/object:Gem::Version
-  version: 1.5.3
+  version: 1.6.0.beta.1
 platform: ruby
 authors:
 - Eloy Duran
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-05-24 00:00:00.000000000 Z
+date: 2018-08-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: cocoapods-core
@@ -19,14 +19,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.5.3
+        version: 1.6.0.beta.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.5.3
+        version: 1.6.0.beta.1
 - !ruby/object:Gem::Dependency
   name: claide
   requirement: !ruby/object:Gem::Requirement
@@ -73,7 +73,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: 1.2.1
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -83,7 +83,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: 1.2.1
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -153,7 +153,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.3.0
+        version: 1.3.1
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -163,7 +163,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.3.0
+        version: 1.3.1
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -193,21 +193,21 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.5
+        version: 0.6.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.6.5
+        version: 0.6.6
 - !ruby/object:Gem::Dependency
   name: xcodeproj
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.5.7
+        version: 1.6.0
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -217,7 +217,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.5.7
+        version: 1.6.0
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -317,14 +317,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: bacon
   requirement: !ruby/object:Gem::Requirement
@@ -454,10 +454,6 @@ files:
 - lib/cocoapods/generator/module_map.rb
 - lib/cocoapods/generator/prefix_header.rb
 - lib/cocoapods/generator/umbrella_header.rb
-- lib/cocoapods/generator/xcconfig.rb
-- lib/cocoapods/generator/xcconfig/aggregate_xcconfig.rb
-- lib/cocoapods/generator/xcconfig/pod_xcconfig.rb
-- lib/cocoapods/generator/xcconfig/xcconfig_helper.rb
 - lib/cocoapods/hooks_manager.rb
 - lib/cocoapods/installer.rb
 - lib/cocoapods/installer/analyzer.rb
@@ -483,10 +479,13 @@ files:
 - lib/cocoapods/installer/xcode.rb
 - lib/cocoapods/installer/xcode/pods_project_generator.rb
 - lib/cocoapods/installer/xcode/pods_project_generator/aggregate_target_installer.rb
+- lib/cocoapods/installer/xcode/pods_project_generator/app_host_installer.rb
 - lib/cocoapods/installer/xcode/pods_project_generator/file_references_installer.rb
 - lib/cocoapods/installer/xcode/pods_project_generator/pod_target_installer.rb
 - lib/cocoapods/installer/xcode/pods_project_generator/pod_target_integrator.rb
+- lib/cocoapods/installer/xcode/pods_project_generator/target_installation_result.rb
 - lib/cocoapods/installer/xcode/pods_project_generator/target_installer.rb
+- lib/cocoapods/installer/xcode/pods_project_generator/target_installer_helper.rb
 - lib/cocoapods/installer/xcode/target_validator.rb
 - lib/cocoapods/open-uri.rb
 - lib/cocoapods/project.rb
@@ -501,6 +500,7 @@ files:
 - lib/cocoapods/sources_manager.rb
 - lib/cocoapods/target.rb
 - lib/cocoapods/target/aggregate_target.rb
+- lib/cocoapods/target/build_settings.rb
 - lib/cocoapods/target/pod_target.rb
 - lib/cocoapods/user_interface.rb
 - lib/cocoapods/user_interface/error_report.rb