cocoapods 1.0.0 → 1.1.0

data/lib/cocoapods/resolver.rb

@@ -73,9 +73,10 @@ module Pod
     def specs_by_target
       @specs_by_target ||= {}.tap do |specs_by_target|
         podfile.target_definition_list.each do |target|
+          dependencies = {}
           specs = target.dependencies.map(&:name).flat_map do |name|
             node = @activated.vertex_named(name)
-            valid_dependencies_for_target_from_node(target, node) << node
+            valid_dependencies_for_target_from_node(target, dependencies, node) << node
           end
 
           specs_by_target[target] = specs.
@@ -389,7 +390,6 @@ module Pod
         error.conflicts.each do |name, conflict|
           local_pod_parent = conflict.requirement_trees.flatten.reverse.find(&:local?)
           lockfile_reqs = conflict.requirements[name_for_locking_dependency_source]
-
           if lockfile_reqs && lockfile_reqs.last && lockfile_reqs.last.prerelease? && !conflict.existing
             message = 'Due to the previous naïve CocoaPods resolver, ' \
               "you were using a pre-release version of `#{name}`, " \
@@ -398,7 +398,7 @@ module Pod
               'version requirement to your Podfile ' \
               "(e.g. `pod '#{name}', '#{lockfile_reqs.map(&:requirement).join("', '")}'`) " \
               "or revert to a stable version by running `pod update #{name}`."
-          elsif local_pod_parent && !specifications_for_dependency(conflict.requirement).empty?
+          elsif local_pod_parent && !specifications_for_dependency(conflict.requirement).empty? && !conflict.possibility
             # Conflict was caused by a requirement from a local dependency.
             # Tell user to use `pod update`.
             message << "\n\nIt seems like you've changed the constraints of dependency `#{name}` " \
@@ -472,13 +472,19 @@ module Pod
     #         An array of target-appropriate nodes whose `payload`s are
     #         dependencies for `target`.
     #
-    def valid_dependencies_for_target_from_node(target, node)
-      validate_platform(node.payload, target)
-      dependency_nodes = node.outgoing_edges.select do |edge|
-        edge_is_valid_for_target?(edge, target)
-      end.map(&:destination)
+    def valid_dependencies_for_target_from_node(target, dependencies, node)
+      dependencies[node.name] ||= begin
+        validate_platform(node.payload, target)
+        dependency_nodes = node.outgoing_edges.select do |edge|
+          edge_is_valid_for_target?(edge, target)
+        end.map(&:destination)
+
+        dependency_nodes + dependency_nodes.flat_map do |item|
+          node_result = valid_dependencies_for_target_from_node(target, dependencies, item)
 
-      dependency_nodes + dependency_nodes.flat_map { |n| valid_dependencies_for_target_from_node(target, n) }
+          node_result
+        end
+      end
     end
 
     # Whether the given `edge` should be followed to find dependencies for the

data/lib/cocoapods/sandbox/headers_store.rb

@@ -72,11 +72,11 @@ module Pod
       #
       def add_files(namespace, relative_header_paths)
         relative_header_paths.map do |relative_header_path|
-          add_file(namespace, relative_header_path, relative_header_path.basename)
+          add_file(namespace, relative_header_path)
         end
       end
 
-      # Adds a header to the directory under different name.
+      # Adds a header to the directory.
       #
       # @param  [Pathname] namespace
       #         the path where the header file should be stored relative to the
@@ -86,23 +86,17 @@ module Pod
       #         the path of the header file relative to the Pods project
       #         (`PODS_ROOT` variable of the xcconfigs).
       #
-      # @param  [String] final_name
-      #         the name under which the file should be available in the
-      #         headers directory.
-      #
       # @note   This method does _not_ add the file to the search paths.
       #
       # @return [Pathname]
       #
-      def add_file(namespace, relative_header_path, final_name)
+      def add_file(namespace, relative_header_path)
         namespaced_path = root + namespace
         namespaced_path.mkpath unless File.exist?(namespaced_path)
 
         absolute_source = (sandbox.root + relative_header_path)
         source = absolute_source.relative_path_from(namespaced_path)
-        Dir.chdir(namespaced_path) do
-          FileUtils.ln_sf(source, final_name)
-        end
+        FileUtils.ln_sf(source, namespaced_path)
         namespaced_path + relative_header_path.basename
       end
 

data/lib/cocoapods/sandbox/path_list.rb

@@ -1,3 +1,5 @@
+require 'active_support/multibyte/unicode'
+
 module Pod
   class Sandbox
     # The PathList class is designed to perform multiple glob matches against
@@ -20,7 +22,8 @@ module Pod
       # @param  [Pathname] root The root of the PathList.
       #
       def initialize(root)
-        @root = root
+        root_dir = ActiveSupport::Multibyte::Unicode.normalize(root.to_s)
+        @root = Pathname.new(root_dir)
         @glob_cache = {}
       end
 
@@ -47,15 +50,23 @@ module Pod
         unless root.exist?
           raise Informative, "Attempt to read non existent folder `#{root}`."
         end
-        root_length  = root.to_s.length + 1
         escaped_root = escape_path_for_glob(root)
-        paths = Dir.glob(escaped_root + '**/*', File::FNM_DOTMATCH).sort_by(&:upcase)
-        absolute_dirs  = paths.select { |path| File.directory?(path) }
-        relative_dirs  = absolute_dirs.map  { |p| p[root_length..-1] }
-        absolute_paths = paths.reject { |p| p == "#{root}/." || p == "#{root}/.." }
-        relative_paths = absolute_paths.map { |p| p[root_length..-1] }
-        @files = relative_paths - relative_dirs
-        @dirs  = relative_dirs.map { |d| d.gsub(/\/\.\.?$/, '') }.reject { |d| d == '.' || d == '..' } .uniq
+
+        absolute_paths = Pathname.glob(escaped_root + '**/*', File::FNM_DOTMATCH).lazy
+        dirs_and_files = absolute_paths.reject { |path| path.basename.to_s =~ /^\.\.?$/ }
+        dirs, files = dirs_and_files.partition { |path| File.directory?(path) }
+
+        root_length = root.cleanpath.to_s.length + File::SEPARATOR.length
+        sorted_relative_paths_from_full_paths = lambda do |paths|
+          relative_paths = paths.lazy.map do |path|
+            path_string = path.to_s
+            path_string.slice(root_length, path_string.length - root_length)
+          end
+          relative_paths.sort_by(&:upcase)
+        end
+
+        @dirs = sorted_relative_paths_from_full_paths.call(dirs)
+        @files = sorted_relative_paths_from_full_paths.call(files)
         @glob_cache = {}
       end
 

data/lib/cocoapods/sources_manager.rb

@@ -1,4 +1,5 @@
 require 'cocoapods-core/source'
+require 'set'
 
 module Pod
   class Source
@@ -102,24 +103,22 @@ module Pod
     end
 
     def update_git_repo(show_output = false)
-      ensure_in_repo!
       Config.instance.with_changes(:verbose => show_output) do
-        git!(%w(pull --ff-only))
+        git!(['-C', repo, 'pull', '--ff-only'])
       end
     rescue
-      UI.warn 'CocoaPods was not able to update the ' \
-                "`#{name}` repo. If this is an unexpected issue " \
-                'and persists you can inspect it running ' \
-                '`pod repo update --verbose`'
+      raise Informative, 'CocoaPods was not able to update the ' \
+        "`#{name}` repo. If this is an unexpected issue " \
+        'and persists you can inspect it running ' \
+        '`pod repo update --verbose`'
     end
   end
 
   class MasterSource
     def update_git_repo(show_output = false)
-      ensure_in_repo!
       if repo.join('.git', 'shallow').file?
         UI.info "Performing a deep fetch of the `#{name}` specs repo to improve future performance" do
-          git!(%w(fetch --unshallow))
+          git!(['-C', repo, 'fetch', '--unshallow'])
         end
       end
       super
@@ -141,8 +140,6 @@ module Pod
         "CocoaPods #{latest_cocoapods_version} is available.".green,
         "To update use: `#{install_message}`".green,
         ("[!] This is a test version we'd love you to try.".yellow if rc),
-        ("Until we reach version 1.0 the features of CocoaPods can and will change.\n" \
-         'We strongly recommend that you use the latest version at all times.'.yellow unless rc),
         '',
         'For more information, see https://blog.cocoapods.org ' \
         'and the CHANGELOG for this version at ' \

data/lib/cocoapods/target/aggregate_target.rb

@@ -7,6 +7,10 @@ module Pod
     #         generated this target.
     attr_reader :target_definition
 
+    # Product types where the product's frameworks must be embedded in a host target
+    #
+    EMBED_FRAMEWORKS_IN_HOST_TARGET_TYPES = [:app_extension, :framework, :messages_extension, :watch_extension, :xpc_service].freeze
+
     # Initialize a new instance
     #
     # @param [TargetDefinition] target_definition @see target_definition
@@ -23,6 +27,22 @@ module Pod
       @xcconfigs = {}
     end
 
+    # @return [Boolean] True if the user_target's pods are
+    #         for an extension and must be embedded in a host,
+    #         target, otherwise false.
+    #
+    def requires_host_target?
+      # If we don't have a user_project, then we can't
+      # glean any info about how this target is going to
+      # be integrated, so return false since we can't know
+      # for sure that this target refers to an extension
+      # target that would require a host target
+      return false if user_project.nil?
+      symbol_types = user_targets.map(&:symbol_type).uniq
+      raise ArgumentError, "Expected single kind of user_target for #{name}. Found #{symbol_types.join(', ')}." unless symbol_types.count == 1
+      EMBED_FRAMEWORKS_IN_HOST_TARGET_TYPES.include?(symbol_types[0])
+    end
+
     # @return [String] the label for the target.
     #
     def label

data/lib/cocoapods/target/pod_target.rb

@@ -47,6 +47,7 @@ module Pod
       @file_accessors = []
       @resource_bundle_targets = []
       @dependent_targets = []
+      @build_config_cache = {}
     end
 
     # @param [Hash{Array => PodTarget}] cache
@@ -65,6 +66,7 @@ module Pod
           target.native_target = native_target
           target.archs = archs
           target.dependent_targets = dependent_targets.flat_map { |pt| pt.scoped(cache) }.select { |pt| pt.target_definitions == [target_definition] }
+          target.host_requires_frameworks = host_requires_frameworks
           cache[cache_key] = target
         end
       end
@@ -80,6 +82,12 @@ module Pod
       end
     end
 
+    # @return [String] the Swift version for the target.
+    #
+    def swift_version
+      target_definitions.map(&:swift_version).compact.uniq.first
+    end
+
     # @note   The deployment target for the pod target is the maximum of all
     #         the deployment targets for the current platform of the target
     #         (or the minimum required to support the current installation
@@ -134,9 +142,12 @@ module Pod
     # A target should not be build if it has no source files.
     #
     def should_build?
-      source_files = file_accessors.flat_map(&:source_files)
-      source_files -= file_accessors.flat_map(&:headers)
-      !source_files.empty?
+      return @should_build if defined? @should_build
+      @should_build = begin
+        source_files = file_accessors.flat_map(&:source_files)
+        source_files -= file_accessors.flat_map(&:headers)
+        !source_files.empty?
+      end
     end
 
     # @return [Array<Specification::Consumer>] the specification consumers for
@@ -149,8 +160,11 @@ module Pod
     # @return [Boolean] Whether the target uses Swift code
     #
     def uses_swift?
-      file_accessors.any? do |file_accessor|
-        file_accessor.source_files.any? { |sf| sf.extname == '.swift' }
+      return @uses_swift if defined? @uses_swift
+      @uses_swift = begin
+        file_accessors.any? do |file_accessor|
+          file_accessor.source_files.any? { |sf| sf.extname == '.swift' }
+        end
       end
     end
 
@@ -188,8 +202,14 @@ module Pod
     #         dependency upon.
     #
     def recursive_dependent_targets
-      targets = dependent_targets + dependent_targets.flat_map(&:recursive_dependent_targets)
-      targets.uniq!
+      targets = dependent_targets.clone
+
+      targets.each do |target|
+        target.dependent_targets.each do |t|
+          targets.push(t) unless t == self || targets.include?(t)
+        end
+      end
+
       targets
     end
 
@@ -203,13 +223,20 @@ module Pod
     #         The name of the build configuration.
     #
     def include_in_build_config?(target_definition, configuration_name)
+      key = [target_definition.label, configuration_name]
+      if @build_config_cache.key?(key)
+        return @build_config_cache[key]
+      end
+
       whitelists = target_definition_dependencies(target_definition).map do |dependency|
         target_definition.pod_whitelisted_for_configuration?(dependency.name, configuration_name)
       end.uniq
 
       if whitelists.empty?
+        @build_config_cache[key] = true
         return true
       elsif whitelists.count == 1
+        @build_config_cache[key] = whitelists.first
         whitelists.first
       else
         raise Informative, "The subspecs of `#{pod_name}` are linked to " \
@@ -224,13 +251,16 @@ module Pod
     # @return [Bool]
     #
     def inhibit_warnings?
+      return @inhibit_warnings if defined? @inhibit_warnings
       whitelists = target_definitions.map do |target_definition|
         target_definition.inhibits_warnings_for_pod?(root_spec.name)
       end.uniq
 
       if whitelists.empty?
+        @inhibit_warnings = false
         return false
       elsif whitelists.count == 1
+        @inhibit_warnings = whitelists.first
         whitelists.first
       else
         UI.warn "The pod `#{pod_name}` is linked to different targets " \

data/lib/cocoapods/user_interface/error_report.rb

@@ -2,6 +2,7 @@
 
 require 'rbconfig'
 require 'cgi'
+require 'gh_inspector'
 
 module Pod
   module UserInterface
@@ -112,6 +113,12 @@ EOS
 EOS
         end
 
+        def search_for_exceptions(exception)
+          inspector = GhInspector::Inspector.new 'cocoapods', 'cocoapods'
+          message_delegate = UserInterface::InspectorReporter.new
+          inspector.search_exception exception, message_delegate
+        end
+
         private
 
         def `(other)

data/lib/cocoapods/user_interface/inspector_reporter.rb

@@ -0,0 +1,109 @@
+require 'uri'
+
+module Pod
+  module UserInterface
+    # Redirects GH-issues delegate callbacks to CocoaPods UI methods.
+    #
+    class InspectorReporter
+      # Called just as the investigation has begun.
+      # Lets the user know that it's looking for an issue.
+      #
+      # @param [query] String unused
+      #
+      # @param [GhInspector::Inspector] inspector
+      #        The current inspector
+      #
+      # @return [void]
+      #
+      def inspector_started_query(_, inspector)
+        UI.puts "Looking for related issues on #{inspector.repo_owner}/#{inspector.repo_name}..."
+      end
+
+      # Called once the inspector has recieved a report with more than one issue,
+      # showing the top 3 issues, and offering a link to see more.
+      #
+      # @param [GhInspector::InspectionReport] report
+      #        Report a list of the issues
+      #
+      # @param [GhInspector::Inspector] inspector
+      #        The current inspector
+      #
+      # @return [void]
+      #
+      def inspector_successfully_recieved_report(report, _)
+        report.issues[0..2].each { |issue| print_issue_full(issue) }
+
+        if report.issues.count > 3
+          UI.puts "and #{report.total_results - 3} more at:"
+          UI.puts report.url
+        end
+      end
+
+      # Called once the report has been recieved, but when there are no issues found.
+      #
+      # @param [GhInspector::InspectionReport] report
+      #        An empty report
+      #
+      # @param [GhInspector::Inspector] inspector
+      #        The current inspector
+      #
+      # @return [void]
+      #
+      def inspector_recieved_empty_report(_, inspector)
+        UI.puts 'Found no similar issues. To create a new issue, please visit:'
+        UI.puts "https://github.com/#{inspector.repo_owner}/#{inspector.repo_name}/issues/new"
+      end
+
+      # Called when there have been networking issues in creating the report.
+      #
+      # @param [Error] error
+      #        The error returned during networking
+      #
+      # @param [String] query
+      #        The original search query
+      #
+      # @param [GhInspector::Inspector] inspector
+      #        The current inspector
+      #
+      # @return [void]
+      #
+      def inspector_could_not_create_report(error, query, inspector)
+        safe_query = URI.escape query
+        UI.puts 'Could not access the GitHub API, you may have better luck via the website.'
+        UI.puts "https://github.com/#{inspector.repo_owner}/#{inspector.repo_name}/search?q=#{safe_query}&type=Issues&utf8=✓"
+        UI.puts "Error: #{error.name}"
+      end
+
+      private
+
+      def print_issue_full(issue)
+        safe_url = URI.escape issue.html_url
+        UI.puts " - #{issue.title}"
+        UI.puts "   #{safe_url} [#{issue.state}] [#{issue.comments} comment#{issue.comments == 1 ? '' : 's'}]"
+        UI.puts "   #{pretty_date(issue.updated_at)}"
+        UI.puts ''
+      end
+
+      # Taken from http://stackoverflow.com/questions/195740/how-do-you-do-relative-time-in-rails
+      def pretty_date(date_string)
+        date = Time.parse(date_string)
+        a = (Time.now - date).to_i
+
+        case a
+        when 0 then 'just now'
+        when 1 then 'a second ago'
+        when 2..59 then a.to_s + ' seconds ago'
+        when 60..119 then 'a minute ago' # 120 = 2 minutes
+        when 120..3540 then (a / 60).to_i.to_s + ' minutes ago'
+        when 3541..7100 then 'an hour ago' # 3600 = 1 hour
+        when 7101..82_800 then ((a + 99) / 3600).to_i.to_s + ' hours ago'
+        when 82_801..172_000 then 'a day ago' # 86400 = 1 day
+        when 172_001..518_400 then ((a + 800) / (60 * 60 * 24)).to_i.to_s + ' days ago'
+        when 518_400..1_036_800 then 'a week ago'
+        when 1_036_801..4_147_204 then ((a + 180_000) / (60 * 60 * 24 * 7)).to_i.to_s + ' weeks ago'
+        else date.strftime('%d %b %Y')
+        end
+      end
+    end
+  end
+end

data/lib/cocoapods/user_interface.rb

@@ -1,4 +1,5 @@
 require 'cocoapods/user_interface/error_report'
+require 'cocoapods/user_interface/inspector_reporter'
 
 module Pod
   # Provides support for UI output. It provides support for nested sections of
@@ -7,7 +8,7 @@ module Pod
   module UserInterface
     require 'colored'
 
-    @title_colors      =  %w( yellow green    )
+    @title_colors      =  %w( yellow green )
     @title_level       =  0
     @indentation_level =  2
     @treat_titles_as_messages = false
@@ -117,9 +118,6 @@ module Pod
         self.title_level -= 1
       end
 
-      # def title(title, verbose_prefix = '', relative_indentation = 2)
-      # end
-
       # Prints a verbose message taking an optional verbose prefix and
       # a relative indentation valid for the UI action in the passed
       # block.
@@ -187,7 +185,11 @@ module Pod
         if pathname
           from_path = config.podfile_path.dirname if config.podfile_path
           from_path ||= Pathname.pwd
-          path = Pathname(pathname).relative_path_from(from_path)
+          path = begin
+                   Pathname(pathname).relative_path_from(from_path)
+                 rescue
+                   pathname
+                 end
           "`#{path}`"
         else
           ''

data/lib/cocoapods/validator.rb

@@ -1,8 +1,6 @@
 require 'active_support/core_ext/array'
 require 'active_support/core_ext/string/inflections'
 
-autoload :Fourflusher, 'fourflusher'
-
 module Pod
   # Validates a Specification.
   #
@@ -144,7 +142,15 @@ module Pod
         reasons << 'all results apply only to public specs, but you can use ' \
                    '`--private` to ignore them if linting the specification for a private pod'
       end
-      reasons.to_sentence
+      if dot_swift_version.nil?
+        reasons.to_sentence + ".\n[!] The validator for Swift projects uses " \
+          'Swift 3.0 by default, if you are using a different version of ' \
+          'swift you can use a `.swift-version` file to set the version for ' \
+          "your Pod. For example to use Swift 2.3, run: \n" \
+          '    `echo "2.3" > .swift-version`'
+      else
+        reasons.to_sentence
+      end
     end
 
     #-------------------------------------------------------------------------#
@@ -239,6 +245,30 @@ module Pod
       Pathname(Dir.tmpdir) + 'CocoaPods/Lint'
     end
 
+    # @return [String] the SWIFT_VERSION to use for validation.
+    #
+    def swift_version
+      @swift_version ||= dot_swift_version || '3.0'
+    end
+
+    # Set the SWIFT_VERSION that should be used to validate the pod.
+    #
+    attr_writer :swift_version
+
+    # @return [String] the SWIFT_VERSION in the .swift-version file or nil.
+    #
+    def dot_swift_version
+      swift_version_path = file.dirname + '.swift-version'
+      swift_version_path.read if swift_version_path.exist?
+    end
+
+    # @return [String] A string representing the Swift version used during linting
+    #                  or nil, if Swift was not used.
+    #
+    def used_swift_version
+      swift_version if @installer.pod_targets.any?(&:uses_swift?)
+    end
+
     #-------------------------------------------------------------------------#
 
     private
@@ -388,10 +418,26 @@ module Pod
 
       source_file = write_app_import_source_file(pod_target)
       source_file_ref = app_project.new_group('App', 'App').new_file(source_file)
-      app_project.targets.first.add_file_references([source_file_ref])
+      app_target = app_project.targets.first
+      app_target.add_file_references([source_file_ref])
+      add_swift_version(app_target)
+      add_xctest(app_target) if @installer.pod_targets.any? { |pt| pt.spec_consumers.any? { |c| c.frameworks.include?('XCTest') } }
       app_project.save
     end
 
+    def add_swift_version(app_target)
+      app_target.build_configurations.each do |configuration|
+        configuration.build_settings['SWIFT_VERSION'] = swift_version
+      end
+    end
+
+    def add_xctest(app_target)
+      app_target.build_configurations.each do |configuration|
+        search_paths = configuration.build_settings['FRAMEWORK_SEARCH_PATHS'] ||= '$(inherited)'
+        search_paths << ' "$(PLATFORM_DIR)/Developer/Library/Frameworks"'
+      end
+    end
+
     def write_app_import_source_file(pod_target)
       language = pod_target.uses_swift? ? :swift : :objc
 
@@ -427,7 +473,7 @@ module Pod
     # for all available platforms with xcodebuild.
     #
     def install_pod
-      %i(verify_no_duplicate_framework_names
+      %i(verify_no_duplicate_framework_and_library_names
          verify_no_static_framework_transitive_dependencies
          verify_framework_usage generate_pods_project integrate_user_project
          perform_post_install_actions).each { |m| @installer.send(m) }
@@ -438,6 +484,7 @@ module Pod
           next unless native_target = pod_target.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'] = swift_version if pod_target.uses_swift?
           end
         end
         if target.pod_targets.any?(&:uses_swift?) && consumer.platform_name == :ios &&
@@ -575,7 +622,7 @@ module Pod
       end
       non_source_files = header_files - file_accessor.source_files
       unless non_source_files.empty?
-        error(attr_name, 'The pattern includes header files that are not listed' \
+        error(attr_name, 'The pattern includes header files that are not listed ' \
           "in source_files (#{non_source_files.join(', ')}).")
       end
     end
@@ -710,20 +757,21 @@ module Pod
     #         returns its output (both STDOUT and STDERR).
     #
     def xcodebuild
-      command = %w(clean build -workspace App.xcworkspace -scheme App -configuration Release)
+      require 'fourflusher'
+      command = ['clean', 'build', '-workspace', File.join(validation_dir, 'App.xcworkspace'), '-scheme', 'App', '-configuration', 'Release']
       case consumer.platform_name
       when :ios
         command += %w(CODE_SIGN_IDENTITY=- -sdk iphonesimulator)
-        command += Fourflusher::SimControl.new.destination('iPhone 4s', deployment_target)
+        command += Fourflusher::SimControl.new.destination(:oldest, 'iOS', deployment_target)
       when :watchos
         command += %w(CODE_SIGN_IDENTITY=- -sdk watchsimulator)
-        command += Fourflusher::SimControl.new.destination('Apple Watch - 38mm', deployment_target)
+        command += Fourflusher::SimControl.new.destination(:oldest, 'watchOS', deployment_target)
       when :tvos
         command += %w(CODE_SIGN_IDENTITY=- -sdk appletvsimulator)
-        command += Fourflusher::SimControl.new.destination('Apple TV 1080p', deployment_target)
+        command += Fourflusher::SimControl.new.destination(:oldest, 'tvOS', deployment_target)
       end
 
-      output, status = Dir.chdir(validation_dir) { _xcodebuild(command) }
+      output, status = _xcodebuild(command)
 
       unless status.success?
         message = 'Returned an unsuccessful exit code.'