pod-builder 3.0.0 → 3.4.0

data/lib/pod_builder/command/sync_podfile.rb

@@ -12,17 +12,17 @@ module PodBuilder
 
         all_buildable_items = Analyze.podfile_items(installer, analyzer)
 
-        Dir.chdir(PodBuilder::project_path)
-
-        previous_podfile_content = File.read("Podfile")
-        Podfile::write_prebuilt(all_buildable_items, analyzer)        
-        updated_podfile_content = File.read("Podfile")
-
-        Licenses::write([], all_buildable_items)
-
-        if previous_podfile_content != updated_podfile_content
-          bundler_prefix = Configuration.use_bundler ? "bundle exec " : ""
-          system("#{bundler_prefix}pod install;")
+        Dir.chdir(PodBuilder::project_path) do
+          previous_podfile_content = File.read("Podfile")
+          Podfile::write_prebuilt(all_buildable_items, analyzer)        
+          updated_podfile_content = File.read("Podfile")
+  
+          Licenses::write([], all_buildable_items)
+  
+          if previous_podfile_content != updated_podfile_content
+            bundler_prefix = Configuration.use_bundler ? "bundle exec " : ""
+            system("#{bundler_prefix}pod install;")
+          end  
         end
         
         puts "\n\n🎉 done!\n".green

data/lib/pod_builder/command/update.rb

@@ -35,7 +35,6 @@ module PodBuilder
         ARGV.clear
         pods_to_update.each { |x| ARGV << x }
 
-        # OPTIONS[:resolve_parent_dependencies] = true
         return PodBuilder::Command::Build.call
       end              
     end

data/lib/pod_builder/configuration.rb

@@ -75,6 +75,7 @@ module PodBuilder
       attr_accessor :build_xcframeworks_all
       attr_accessor :build_xcframeworks_include
       attr_accessor :build_xcframeworks_exclude
+      attr_accessor :post_actions
     end
 
     @build_settings = DEFAULT_BUILD_SETTINGS
@@ -111,6 +112,8 @@ module PodBuilder
     @build_xcframeworks_all = false
     @build_xcframeworks_include = []
     @build_xcframeworks_exclude = []
+
+    @post_actions = {}
     
     def self.check_inited
       raise "\n\nNot inited, run `pod_builder init`\n".red if podbuilder_path.nil?
@@ -229,6 +232,11 @@ module PodBuilder
             Configuration.build_xcframeworks_exclude = value
           end
         end
+        if value = json["post_actions"]
+          if value.is_a?(Hash)
+            Configuration.post_actions = PodBuilder::PostActions.load(value)
+          end
+        end
 
         Configuration.build_settings.freeze
 
@@ -286,9 +294,9 @@ module PodBuilder
         end
       end
       if Configuration.build_xcframeworks_all
-        raise "Invalid PodBuilder.json configuration: 'build_xcframeworks_all' is true and 'build_xcframeworks_include' is not empty" if Configuration.build_xcframeworks_include.count > 0
+        raise "\n\nInvalid PodBuilder.json configuration: 'build_xcframeworks_all' is true and 'build_xcframeworks_include' is not empty\n".red if Configuration.build_xcframeworks_include.count > 0
       else
-        raise "Invalid PodBuilder.json configuration: 'build_xcframeworks_all' is false and 'build_xcframeworks_exclude' is not empty" if Configuration.build_xcframeworks_exclude.count > 0
+        raise "\n\nInvalid PodBuilder.json configuration: 'build_xcframeworks_all' is false and 'build_xcframeworks_exclude' is not empty\n".red if Configuration.build_xcframeworks_exclude.count > 0
       end
     end
     

data/lib/pod_builder/core.rb

@@ -12,6 +12,7 @@ require 'pod_builder/info'
 require 'pod_builder/configuration'
 require 'pod_builder/podspec'
 require 'pod_builder/licenses'
+require 'pod_builder/post_actions'
 
 require 'core_ext/string'
 
@@ -132,7 +133,7 @@ module PodBuilder
       folder_in_home = x.gsub(home, "")
       !folder_in_home.include?("/Pods/") && !x.include?(PodBuilder::basepath("Sources")) && !x.include?(PodBuilder::basepath + "/") 
     }
-    raise "\n\nxcodeproj not found!".red if xcodeprojects.count == 0
+    raise "\n\nxcodeproj not found!\n".red if xcodeprojects.count == 0
     raise "\n\nFound multiple xcodeproj:\n#{xcodeprojects.join("\n")}".red if xcodeprojects.count > 1
 
     @@xcodeproj_path = xcodeprojects.first
@@ -148,7 +149,7 @@ module PodBuilder
       folder_in_home = x.gsub(home, "")
       !folder_in_home.include?("/Pods/") && !x.include?(PodBuilder::basepath("Sources")) && !x.include?(PodBuilder::basepath + "/") && !x.include?(".xcodeproj/")
     }
-    raise "\n\nxcworkspace not found!".red if xcworkspaces.count == 0
+    raise "\n\nxcworkspace not found!\n".red if xcworkspaces.count == 0
     raise "\n\nFound multiple xcworkspaces:\n#{xcworkspaces.join("\n")}".red if xcworkspaces.count > 1
 
     @@xcodeworkspace_path = xcworkspaces.first
@@ -179,7 +180,7 @@ module PodBuilder
 
   def self.system_swift_version
     swift_version = `swiftc --version | grep -o 'swiftlang-.*\s'`.strip()
-    raise "\n\nUnsupported swift compiler version, expecting `swiftlang` keyword in `swiftc --version`".red if swift_version.length == 0
+    raise "\n\nUnsupported swift compiler version, expecting `swiftlang` keyword in `swiftc --version`\n".red if swift_version.length == 0
     return swift_version
   end
 

data/lib/pod_builder/info.rb

@@ -60,7 +60,7 @@ module PodBuilder
         
         return { "repo": "local" }
       else
-        raise "\n\nFailed extracting version from line:\n#{line}\n\n".red
+        raise "\n\nFailed extracting version from line:\n#{line}\n".red
       end
     end
 

data/lib/pod_builder/install.rb

@@ -45,7 +45,7 @@ begin
         elsif defined?(Pod::Target::BuildType) # CocoaPods 1.7, 1.8
           Pod::Target::BuildType.new(linkage: :dynamic, packaging: :framework)
         else
-          raise "\n\nBuildType not found. Open an issue reporting your CocoaPods version".red
+          raise "\n\nBuildType not found. Open an issue reporting your CocoaPods version\n".red
         end
       else
         swz_build_type()
@@ -139,7 +139,7 @@ module PodBuilder
 
   class Install
     # This method will generate prebuilt data by building from "/tmp/pod_builder/Podfile"
-    def self.podfile(podfile_content, podfile_items, build_configuration)
+    def self.podfile(podfile_content, podfile_items, argument_pods, build_configuration)
       puts "Preparing build Podfile".yellow
       
       PodBuilder::safe_rm_rf(Configuration.build_path)
@@ -160,7 +160,7 @@ module PodBuilder
         lock_file = "#{Configuration.build_path}/pod_builder.lock"
         FileUtils.touch(lock_file)
         
-        prebuilt_entries = use_prebuilt_entries_for_unchanged_pods(podfile_path, podfile_items)
+        prebuilt_entries = use_prebuilt_entries_for_unchanged_pods(podfile_path, podfile_items, argument_pods)
         
         install
         
@@ -247,7 +247,7 @@ module PodBuilder
     def self.license_specifiers
       acknowledge_file = "#{Configuration.build_path}/Pods/Target Support Files/Pods-DummyTarget/Pods-DummyTarget-acknowledgements.plist"
       unless File.exist?(acknowledge_file)
-        raise "\n\nLicense file not found".red
+        raise "\n\nLicense file not found\n".red
       end
       
       plist = CFPropertyList::List.new(:file => acknowledge_file)
@@ -280,45 +280,67 @@ module PodBuilder
       return podfile_content
     end
     
-    def self.use_prebuilt_entries_for_unchanged_pods(podfile_path, podfile_items)
+    def self.use_prebuilt_entries_for_unchanged_pods(podfile_path, podfile_items, argument_pods)
       podfile_content = File.read(podfile_path)
       
       replaced_items = []
 
-      if OPTIONS.has_key?(:force_rebuild)
-        podfile_content.gsub!("%%%prebuilt_root_paths%%%", "{}")
-      else
-        download # Copy files under #{Configuration.build_path}/Pods so that we can determine build folder hashes
+      download # Copy files under #{Configuration.build_path}/Pods so that we can determine build folder hashes
 
-        gitignored_files = PodBuilder::gitignoredfiles
+      gitignored_files = PodBuilder::gitignoredfiles
 
-        prebuilt_root_paths = Hash.new
+      prebuilt_root_paths = Hash.new
 
-        # Replace prebuilt entries in Podfile for Pods that have no changes in source code which will avoid rebuilding them
-        items = podfile_items.group_by { |t| t.root_name }.map { |k, v| v.first } # Return one podfile_item per root_name
-        items.each do |item|
-          podspec_path = item.prebuilt_podspec_path
-          if last_build_folder_hash = build_folder_hash_in_prebuilt_info_file(item)
-            if last_build_folder_hash == build_folder_hash(item, gitignored_files)
-              puts "No changes detected to '#{item.root_name}', will skip rebuild".blue
+      puts "Optimizing build".yellow
+      puts "Build strategy".blue
 
-              replaced_items.push(item)
+      prebuild_log = lambda { |item, reason|
+        if item.is_prebuilt
+          puts "#{item.root_name} is prebuilt"
+        else
+          puts "#{item.root_name} from source #{reason}".blue
+        end
+      }
 
-              podfile_items.select { |t| t.root_name == item.root_name }.each do |replace_item|
-                replace_regex = "pod '#{Regexp.quote(replace_item.name)}', .*"
-                replace_line_found = podfile_content =~ /#{replace_regex}/i
-                raise "\n\nFailed finding pod entry for '#{replace_item.name}'".red unless replace_line_found
-                podfile_content.gsub!(/#{replace_regex}/, replace_item.prebuilt_entry(true, true))
+      # Replace prebuilt entries in Podfile for Pods that have no changes in source code which will avoid rebuilding them
+      items = podfile_items.group_by { |t| t.root_name }.map { |k, v| v.first }.sort_by { |t| t.root_name } # Return one podfile_item per root_name
+      if OPTIONS.has_key?(:force_rebuild)
+        rebuild_pods = items.select { |t| argument_pods.include?(t.root_name) }
 
-                prebuilt_root_paths[replace_item.root_name] = PodBuilder::prebuiltpath
-              end
-            end
-          end
+        rebuild_pods.each do |item|
+          prebuild_log.call(item, "")
+        end
+
+        items -= rebuild_pods
+      end
+      
+      items.each do |item|
+        podspec_path = item.prebuilt_podspec_path
+        unless last_build_folder_hash = build_folder_hash_in_prebuilt_info_file(item)
+          prebuild_log.call(item, "(folder hash missing)")
+          next
         end
+        
+        if last_build_folder_hash == build_folder_hash(item, gitignored_files)
+          puts "#{item.root_name} reuse PodBuilder cache"
+
+          replaced_items.push(item)
 
-        podfile_content.gsub!("%%%prebuilt_root_paths%%%", prebuilt_root_paths.to_s)
+          podfile_items.select { |t| t.root_name == item.root_name }.each do |replace_item|
+            replace_regex = "pod '#{Regexp.quote(replace_item.name)}', .*"
+            replace_line_found = podfile_content =~ /#{replace_regex}/i
+            raise "\n\nFailed finding pod entry for '#{replace_item.name}'\n".red unless replace_line_found
+            podfile_content.gsub!(/#{replace_regex}/, replace_item.prebuilt_entry(true, true))
+
+            prebuilt_root_paths[replace_item.root_name] = PodBuilder::prebuiltpath
+          end
+        else
+          prebuild_log.call(item, "(folder hash mismatch)")
+        end
       end
 
+      podfile_content.gsub!("%%%prebuilt_root_paths%%%", prebuilt_root_paths.to_s)
+
       File.write(podfile_path, podfile_content)
 
       return replaced_items
@@ -402,6 +424,13 @@ module PodBuilder
           end
         end
 
+        # Cleanup unneeded files (see https://github.com/bazelbuild/rules_apple/pull/1113)
+        ignore_files = Dir.glob(["#{source_path}/**/Modules/**/*.swiftmodule/*.swiftdoc", "#{source_path}/**/Modules/**/*.swiftmodule/**/*.swiftsourceinfo"])
+        ignore_files.each { |t| PodBuilder::safe_rm_rf(t) }
+
+        project_folder = Dir.glob("#{source_path}/**/Modules/**/*.swiftmodule/Project")
+        project_folder.select { |t| File.directory?(t) && Dir.empty?(t) }.each { |t| PodBuilder::safe_rm_rf(t) }
+
         unless Dir.glob("#{source_path}/**/*").select { |t| File.file?(t) }.empty?
           destination_folder = PodBuilder::prebuiltpath(root_name)
           FileUtils.mkdir_p(destination_folder)
@@ -425,11 +454,9 @@ module PodBuilder
     end
     
     def self.init_git(path)
-      current_dir = Dir.pwd
-      
-      Dir.chdir(path)
-      system("git init")
-      Dir.chdir(current_dir)
+      Dir.chdir(path) do 
+        system("git init")
+      end
     end
     
     def self.build_folder_hash_in_prebuilt_info_file(podfile_item)
@@ -457,15 +484,16 @@ module PodBuilder
           unless File.file?(path)
             next
           end
-          
+
           path = File.expand_path(path)
           rel_path = path.gsub(rootpath, "")[1..]
-          unless exclude_files.include?(rel_path)
+
+          unless exclude_files.any? { |t| rel_path.start_with?(t) }
             file_hashes.push(Digest::MD5.hexdigest(File.read(path)))
           end
         end
         
-        return Digest::MD5.hexdigest(file_hashes.join)
+        return Digest::MD5.hexdigest(file_hashes.sort.join)
       else
         # Pod folder might be under .gitignore
         item_path = "#{Configuration.build_path}/Pods/#{podfile_item.root_name}"

data/lib/pod_builder/licenses.rb

@@ -12,19 +12,19 @@
       
         if current_licenses.count > 0 
           licenses_header = current_licenses.shift
-          raise "\n\nUnexpected license found in header".red if licenses_header.has_key?("License")
+          raise "\n\nUnexpected license found in header\n".red if licenses_header.has_key?("License")
         end
         if current_licenses.count > 0 
           license_footer = current_licenses.pop
-          raise "\n\nUnexpected license found in footer".red if license_footer.has_key?("License")
+          raise "\n\nUnexpected license found in footer\n".red if license_footer.has_key?("License")
         end
       end
 
       if licenses.count > 0
         licenses_header = licenses.shift
-        raise "\n\nUnexpected license found in header".red if licenses_header.has_key?("License")
+        raise "\n\nUnexpected license found in header\n".red if licenses_header.has_key?("License")
         license_footer = licenses.pop
-        raise "\n\nUnexpected license found in footer".red if license_footer.has_key?("License")
+        raise "\n\nUnexpected license found in footer\n".red if license_footer.has_key?("License")
 
         lincenses_titles = licenses.map { |x| x["Title"] }
         current_licenses.select! { |x| !lincenses_titles.include?(x["Title"]) }

data/lib/pod_builder/podfile.rb

@@ -6,7 +6,7 @@ module PodBuilder
     PRE_INSTALL_ACTIONS = ["Pod::Installer::Xcode::TargetValidator.send(:define_method, :verify_no_duplicate_framework_and_library_names) {}", "require 'pod_builder/podfile/pre_actions_swizzles'"].freeze
 
     def self.from_podfile_items(items, analyzer, build_configuration, install_using_frameworks, build_catalyst, build_xcframeworks)
-      raise "\n\nno items".red unless items.count > 0
+      raise "\n\nno items\n".red unless items.count > 0
 
       sources = analyzer.sources
       
@@ -34,7 +34,7 @@ module PodBuilder
       items.each do |item|
         build_settings = Configuration.build_settings.dup
 
-        item_build_settings = Configuration.build_settings_overrides[item.name] || {}
+        item_build_settings = Configuration.build_settings_overrides[item.name].dup || {}
         
         # These settings need to be set as is to properly build frameworks
         build_settings["SWIFT_COMPILATION_MODE"] = "wholemodule"
@@ -46,10 +46,6 @@ module PodBuilder
 
         build_settings["IPHONEOS_DEPLOYMENT_TARGET"] = platform.deployment_target.version # Fix compilation warnings on Xcode 12
 
-        # Don't store .pcm info in binary, see https://forums.swift.org/t/swift-behavior-of-gmodules-and-dsyms/23211/3
-        build_settings["CLANG_ENABLE_MODULE_DEBUGGING"] = "NO"
-        build_settings["OTHER_SWIFT_FLAGS"] = "$(inherited) -Xfrontend -no-clang-module-breadcrumbs"
-
         # Ignore deprecation warnings
         build_settings["GCC_WARN_ABOUT_DEPRECATED_FUNCTIONS"] = "NO"
 
@@ -70,10 +66,20 @@ module PodBuilder
           build_settings["BITCODE_GENERATION_MODE"] = "bitcode"
         end
 
+        # Don't store .pcm info in binary, see https://forums.swift.org/t/swift-behavior-of-gmodules-and-dsyms/23211/3
+        build_settings["CLANG_ENABLE_MODULE_DEBUGGING"] = "NO"
+        other_swift_flags_override = " $(inherited) -Xfrontend -no-clang-module-breadcrumbs"
+
         item_build_settings.each do |k, v|
-          build_settings[k] = v
+          # Do not allow to override above settings which are mandatory for a correct compilation
+          if build_settings[k].nil?
+            build_settings[k] = v
+          end
         end
 
+        # All the below settings should be merged with global (Configuration.build_settings) or per pod build_settings (Configuration.build_settings_overrides)
+        build_settings["OTHER_SWIFT_FLAGS"] = build_settings.fetch("OTHER_SWIFT_FLAGS", "") + other_swift_flags_override        
+
         podfile_build_settings += "set_build_settings(\"#{item.root_name}\", #{build_settings.to_s}, installer)\n  "
 
         dependency_names = item.dependency_names.map { |x|
@@ -354,7 +360,7 @@ module PodBuilder
 
         if stripped_line.match(/(pod')(.*?)(')/) != nil
           starting_def_found = stripped_line.start_with?("def") && (line.match("\s*def\s") != nil)
-          raise "\n\nUnsupported single line def/pod. `def` and `pod` shouldn't be on the same line, please modify the following line:\n#{line}".red if starting_def_found
+          raise "\n\nUnsupported single line def/pod. `def` and `pod` shouldn't be on the same line, please modify the following line:\n#{line}\n".red if starting_def_found
         end
       end
     end
@@ -396,9 +402,9 @@ module PodBuilder
       if target_settings.count == 1
         return target_settings.first
       elsif target_settings.count > 1
-        raise "\n\n'use_frameworks!' should be declared only once at Podfile root level (not nested in targets)".red
+        raise "\n\n'use_frameworks!' should be declared only once at Podfile root level (not nested in targets)\n".red
       else
-        raise "\n\nFailed detecting use_frameworks!"
+        raise "\n\nFailed detecting use_frameworks!\n".red
       end
 
       return true
@@ -443,13 +449,13 @@ module PodBuilder
     def self.project_swift_version(analyzer)
       swift_versions = analyzer.instance_variable_get("@result").targets.map { |x| x.target_definition.swift_version }.compact.uniq
 
-      raise "\n\nFound different Swift versions in targets. Expecting one, got `#{swift_versions}`".red if swift_versions.count > 1
+      raise "\n\nFound different Swift versions in targets. Expecting one, got `#{swift_versions}`\n".red if swift_versions.count > 1
 
       return swift_versions.first || PodBuilder::system_swift_version
     end
 
     def self.podfile_items_at(podfile_path, include_prebuilt = false)
-      raise "\n\nExpecting basepath folder!".red if !File.exist?(PodBuilder::basepath("Podfile"))
+      raise "\n\nExpecting basepath folder!\n".red if !File.exist?(PodBuilder::basepath("Podfile"))
 
       if File.basename(podfile_path) != "Podfile"
         File.rename(PodBuilder::basepath("Podfile"), PodBuilder::basepath("Podfile.tmp"))
@@ -607,12 +613,12 @@ module PodBuilder
       base = File.expand_path(File.join(PodBuilder::project_path, ".."))
       bin_js = Dir.glob("#{base}/node_modules/@react-native-community/cli/build/bin.js")
 
-      raise "\n\nReact native cli bin_js not found! Did you run yarn install?".red unless bin_js.count == 1
+      raise "\n\nReact native cli bin_js not found! Did you run yarn install?\n".red unless bin_js.count == 1
       bin_js = bin_js.first
 
       config_dest_path = PodBuilder::basepath("rn_config.json")
 
-      raise "\n\nFailed generating react native configuration file".red unless system("node '#{bin_js}' config > #{config_dest_path}")
+      raise "\n\nFailed generating react native configuration file\n".red unless system("node '#{bin_js}' config > #{config_dest_path}")
       
       content = File.read(config_dest_path)
 
@@ -624,7 +630,7 @@ module PodBuilder
         json["project"]["ios"]["sourceDir"] = "./"
         json["project"]["ios"]["podfile"] = "./"  
       rescue => exception
-        raise "\n\nFailed updating react native configuration json".red
+        raise "\n\nFailed updating react native configuration json\n".red
       end
       
       File.write(config_dest_path, JSON.pretty_generate(json))

data/lib/pod_builder/podfile/pre_actions_swizzles.rb

@@ -24,7 +24,7 @@ class Pod::Generator::EmbedFrameworksScript
   alias_method :swz_initialize, :initialize
   
   def initialize(*args)
-    raise "Unsupported CocoaPods version" if (args.count == 0 || args.count > 2)
+    raise "\n\nUnsupported CocoaPods version\n".red if (args.count == 0 || args.count > 2)
     
     frameworks_by_config = args[0]
     frameworks_by_config.keys.each do |key|

data/lib/pod_builder/podfile_item.rb

@@ -138,6 +138,10 @@ module PodBuilder
     # @return [Bool] Should build as xcframework
     #
     attr_accessor :build_xcframework
+
+    # @return [Bool] True if it's a pod that doesn't provide source code (is already shipped as a prebuilt pod)
+    #
+    attr_accessor :is_prebuilt
     
     # Initialize a new instance
     #
@@ -180,14 +184,26 @@ module PodBuilder
       @weak_frameworks = []
       @libraries = []
 
-      @frameworks += extract_array(spec, "framework")
-      @frameworks += extract_array(spec, "frameworks")
+      @frameworks += extract_array(spec.attributes_hash, "framework")
+      @frameworks += extract_array(spec.attributes_hash, "frameworks")
+      supported_platforms.each do |platform|        
+        @frameworks += extract_array(spec.attributes_hash[platform], "framework")
+        @frameworks += extract_array(spec.attributes_hash[platform], "frameworks")
+      end       
       
-      @weak_frameworks += extract_array(spec, "weak_framework")
-      @weak_frameworks += extract_array(spec, "weak_frameworks")  
+      @weak_frameworks += extract_array(spec.attributes_hash, "weak_framework")
+      @weak_frameworks += extract_array(spec.attributes_hash, "weak_frameworks")  
+      supported_platforms.each do |platform|        
+        @weak_frameworks += extract_array(spec.attributes_hash[platform], "weak_framework")
+        @weak_frameworks += extract_array(spec.attributes_hash[platform], "weak_frameworks")
+      end
 
-      @libraries += extract_array(spec, "library")
-      @libraries += extract_array(spec, "libraries")  
+      @libraries += extract_array(spec.attributes_hash, "library")
+      @libraries += extract_array(spec.attributes_hash, "libraries")  
+      supported_platforms.each do |platform|        
+        @libraries += extract_array(spec.attributes_hash[platform], "library")
+        @libraries += extract_array(spec.attributes_hash[platform], "libraries")
+      end
 
       @header_dir = spec.attributes_hash["header_dir"]
 
@@ -197,7 +213,7 @@ module PodBuilder
       @swift_version = spec.root.swift_version&.to_s
       @module_name = spec.root.module_name
 
-      @default_subspecs = extract_array(spec, "default_subspecs")
+      @default_subspecs = extract_array(spec.attributes_hash, "default_subspecs")
       if default_subspec = spec.attributes_hash["default_subspec"]
         @default_subspecs.push(default_subspec)        
       end
@@ -246,6 +262,8 @@ module PodBuilder
         build_as_xcframework = Configuration.build_xcframeworks_include.include?(@root_name)
       end
       @build_xcframework = build_as_xcframework
+
+      @is_prebuilt = extract_is_prebuilt(spec, all_specs, checkout_options, supported_platforms)
     end
 
     def pod_specification(all_poditems, parent_spec = nil)
@@ -335,33 +353,6 @@ module PodBuilder
       return deps
     end
 
-    # @return [Bool] True if it's a pod that doesn't provide source code (is already shipped as a prebuilt pod)
-    #    
-    def is_prebuilt
-      if Configuration.force_prebuild_pods.include?(@root_name) || Configuration.force_prebuild_pods.include?(@name)
-        return false
-      end
-
-      # We treat pods to skip like prebuilt ones
-      if Configuration.skip_pods.include?(@root_name) || Configuration.skip_pods.include?(@name)
-        return true
-      end
-
-      # Podspecs aren't always properly written (source_file key is often used instead of header_files)
-      # Therefore it can become tricky to understand which pods are already precompiled by boxing a .framework or .a
-      embedded_as_vendored = vendored_frameworks.map { |x| File.basename(x) }.include?("#{module_name}.framework")
-      embedded_as_static_lib = vendored_libraries.map { |x| File.basename(x) }.include?("lib#{module_name}.a")
-      
-      only_headers = (source_files.count > 0 && @source_files.all? { |x| x.end_with?(".h") })
-      no_sources = (@source_files.count == 0 || only_headers) && (@vendored_frameworks + @vendored_libraries).count > 0
-
-      if !no_sources && !only_headers
-        return false
-      else
-        return (no_sources || only_headers || embedded_as_static_lib || embedded_as_vendored)
-      end
-    end
-
     # @return [Bool] True if it's a subspec
     #
     def is_subspec
@@ -472,6 +463,44 @@ module PodBuilder
 
     private
 
+    # @return [Bool] True if it's a pod that doesn't provide source code (is already shipped as a prebuilt pod)
+    #    
+    def extract_is_prebuilt(spec, all_specs, checkout_options, supported_platforms)
+      if Configuration.force_prebuild_pods.include?(@root_name) || Configuration.force_prebuild_pods.include?(@name)
+        return false
+      end
+
+      # We treat pods to skip like prebuilt ones
+      if Configuration.skip_pods.include?(@root_name) || Configuration.skip_pods.include?(@name)
+        return true
+      end
+
+      if default_subspecs != nil && default_subspecs.count > 0
+        default_subspecs.each do |default_subspec_name|
+          if (default_spec = all_specs.detect { |t| t.name == "#{root_name}/#{default_subspec_name}" })
+            default_item = PodfileItem.new(default_spec, all_specs, checkout_options, supported_platforms)
+            if default_item.is_prebuilt
+              return true
+            end
+          end
+        end
+      end
+
+      # Podspecs aren't always properly written (source_file key is often used instead of header_files)
+      # Therefore it can become tricky to understand which pods are already precompiled by boxing a .framework or .a
+      embedded_as_vendored = vendored_frameworks.map { |x| File.basename(x) }.include?("#{module_name}.framework")
+      embedded_as_static_lib = vendored_libraries.map { |x| File.basename(x) }.include?("lib#{module_name}.a")
+      
+      only_headers = (source_files.count > 0 && @source_files.all? { |x| x.end_with?(".h") })
+      no_sources = (@source_files.count == 0 || only_headers) && (@vendored_frameworks + @vendored_libraries).count > 0
+
+      if !no_sources && !only_headers
+        return false
+      else
+        return (no_sources || only_headers || embedded_as_static_lib || embedded_as_vendored)
+      end
+    end
+
     def extract_vendored_frameworks(spec, all_specs)
       items = []
 
@@ -499,8 +528,12 @@ module PodBuilder
       return items.flatten.uniq.compact
     end
 
-    def extract_array(spec, key)
-      element = spec.attributes_hash.fetch(key, [])
+    def extract_array(dict, key)
+      if dict.nil?
+        return []
+      end
+
+      element = dict.fetch(key, [])
       if element.instance_of? String
         element = [element]
       end