pod-builder 1.9.2 → 2.0.0.beta.21

data/exe/pod_builder

@@ -36,15 +36,15 @@ Command:
     + deintegrate                    Deintegrate prebuild folders
     + build                          Build a specific pod declared in the PodBuilder-Podfile
     + build_all                      Build all pods declared in the PodBuilder-Podfile
-    + update                         Rebuild frameworks that are outdated
+    + update                         Rebuild items that are outdated
     + restore_all                    Rebuild all pods declared in the Restore-Podfile
-    + install_sources                Install sources of pods to debug into prebuilt frameworks
-    + update_lldbinit                Generate an lldbinit file with setting target.source-map to debug prebuilt frameworks
+    + install_sources                Install sources of pods to debug into prebuilt items
+    + update_lldbinit                Generate an lldbinit file with setting target.source-map to debug prebuilt items
     + clear_lldbinit                 Clear an lldbinit file from PodBuilder's customizations
     + switch                         Switch between prebuilt/development/standard pod in the Application-Podfile
-    + clean                          Remove prebuild frameworks, dSYMs and source files added by `install_sources` command that are no longer in the PodBuilder-Podfile
+    + clean                          Remove prebuild items, dSYMs and source files added by `install_sources` command that are no longer in the PodBuilder-Podfile
     + sync_podfile                   Update your Application-Podfile with all pods declared in the PodBuilder-Podfile
-    + info                           Print json-formatted informations about prebuilt frameworks
+    + info                           Print json-formatted informations about prebuilt items
 
 Options:
 "
@@ -71,12 +71,15 @@ Options:
         opts.on("-u", "--skip-repo-update", "Skip CocoaPods repo update") do |o|
           OPTIONS[:update_repos] = false
         end
+        opts.on("-f", "--force", "Rebuild items even when no code change is detected") do |o|
+          OPTIONS[:force_rebuild] = false
+        end
         opts.on("-w", "--allow-warnings", "Allow warnings") do |o|
           OPTIONS[:allow_warnings] = o
         end
-        opts.on("-a", "--auto-resolve-deps", "Resolve pod dependencies automatically") do |o|
-          OPTIONS[:auto_resolve_dependencies] = o
-        end
+        # opts.on("-a", "--auto-resolve-deps", "Resolve pod dependencies automatically") do |o|
+        #   OPTIONS[:auto_resolve_dependencies] = o
+        # end
         opts.on("-d", "--debug", "Don't clean build folder") do |o|
           OPTIONS[:debug] = o
         end
@@ -100,6 +103,9 @@ Options:
         opts.on("-u", "--skip-repo-update", "Skip CocoaPods repo update") do |o|
           OPTIONS[:update_repos] = false
         end
+        opts.on("-f", "--force", "Rebuild items even when no code change is detected") do |o|
+          OPTIONS[:force_rebuild] = false
+        end
         opts.on("-w", "--allow-warnings", "Allow warnings") do |o|
           OPTIONS[:allow_warnings] = o
         end
@@ -119,7 +125,7 @@ Usage:
         
     $ pod_builder update [OPTIONS]
 
-      Rebuild frameworks that are outdated
+      Rebuild items that are outdated
         
 Options:
 "        
@@ -129,7 +135,7 @@ Options:
         opts.on("-w", "--allow-warnings", "Allow warnings") do |o|
           OPTIONS[:allow_warnings] = o
         end
-        opts.on("-r", "--dry", "Determine which frameworks need update") do |o|
+        opts.on("-r", "--dry", "Determine which items need to be updated") do |o|
           OPTIONS[:dry_run] = o
         end
         opts.on("-d", "--debug", "Don't clean build folder") do |o|
@@ -224,10 +230,13 @@ Usage:
         
     $ pod_builder clean
 
-      Remove unused framework, dSYM and source folders.
+      Remove unused prebuild data, dSYM and source folders.
         
 Options:
-"           
+"         
+        opts.on("-u", "--skip-repo-update", "Skip CocoaPods repo update") do |o|
+          OPTIONS[:update_repos] = false
+        end
       end,
       :call => [        
         PodBuilder::Command::Clean
@@ -241,7 +250,7 @@ Usage:
         
     $ pod_builder install_sources
 
-      Install source of prebuilt pods to be able to step into and debug framework's code.
+      Install source of prebuilt pods to be able to step into and debug prebuilt's code.
 
 "         
       end,
@@ -258,9 +267,9 @@ Usage:
     $ pod_builder update_lldbinit <LLDBINIT_PATH> <PATH>
 
       Update LLDBINIT_PATH setting target.source-map in order to be able to step into 
-      and debug prebuilt framework's code.
+      and debug prebuilt prebuilt's code.
 
-      Specify the PATH of the folder containing the source code of prebuilt frameworks. PATH can be relative to PodBuilder's Framework folder.
+      Specify the PATH of the folder containing the source code of prebuilt items. PATH can be relative to PodBuilder's Prebuilt folder.
 
 "         
       end,
@@ -289,9 +298,9 @@ Usage:
         opts.banner = "
 Usage:
         
-    $ pod_builder switch PODNAME [OPTIONS]
+    $ pod_builder switch [OPTIONS] PODNAME[s]
 
-      Switch integration between prebuilt/development/default pod version.
+      Switch integration between prebuilt/development/default pod version. Multiple space separated pods can be passed
         
 Options:
 "        
@@ -304,6 +313,9 @@ Options:
         opts.on("-s", "--default", "Default version specified in PodBuilder-Podfile") do |o|
           OPTIONS[:switch_mode] = "default"
         end
+        opts.on("-a", "--all", "Include dependencies") do |o|
+          OPTIONS[:switch_all] = true
+        end
       end,
       :call => [        
         PodBuilder::Command::Switch
@@ -321,7 +333,12 @@ Usage:
       You may want to run this command when you add a new pod to the PodBuilder-Podfile 
       and you want to integrate it in the project without rebuilding it.
         
-"      end,
+Options:
+"       
+        opts.on("-u", "--skip-repo-update", "Skip CocoaPods repo update") do |o|
+          OPTIONS[:update_repos] = false
+        end
+      end,
       :call => [        
         PodBuilder::Command::SyncPodfile
       ]
@@ -357,9 +374,11 @@ Usage:
   else
     PodBuilder::Configuration.load
 
+    PodBuilder::add_lockfile
+
     subcommand[:opts].order!    
     subcommand[:call].each do |k|
-      if (ret = k.call(OPTIONS)) && ret == -1
+      if (ret = k.call) && ret == -1
         puts subcommand[:opts].help
       end
     end
@@ -374,8 +393,6 @@ begin
     raise "\n\nFor safety do not run this as root\n".red
   end
 
-  PodBuilder::add_lock_file
-  valid_lock_file = true
   command_ret = parse_commandline
 rescue Exception => e
   error = e.to_s
@@ -392,14 +409,8 @@ rescue Exception => e
 ensure
   if command_ret == 0
     PodBuilder::clean_basepath
-    
-    if !OPTIONS.has_key?(:debug)
-      PodBuilder::safe_rm_rf(PodBuilder::Configuration.build_path)
-    end    
-  end
-  if valid_lock_file
-    PodBuilder::remove_lock_file
   end
+  PodBuilder::remove_lockfile
 
   exit(command_ret)
 end

data/lib/pod_builder/analyze.rb

@@ -1,4 +1,6 @@
-require 'pod_builder/cocoapods/specification'
+require 'rubygems/specification'
+require 'pod_builder/rome/pre_install.rb'
+require 'pod_builder/rome/post_install.rb'
 
 module PodBuilder
   class Analyze
@@ -6,6 +8,14 @@ module PodBuilder
     #
     def self.installer_at(path, repo_update = false)
       CLAide::Command::PluginManager.load_plugins("cocoapods")
+      
+      # Manually load inline podbuilder-rome plugin
+      pluginspec = Gem::Specification.new("podbuilder-rome", PodBuilder::VERSION)
+      pluginspec.activate
+
+      if !CLAide::Command::PluginManager.loaded_plugins["cocoapods"].map(&:name).include?(pluginspec.name)
+        CLAide::Command::PluginManager.loaded_plugins["cocoapods"].push(pluginspec)
+      end
 
       current_dir = Dir.pwd
       Dir.chdir(path)
@@ -40,15 +50,30 @@ module PodBuilder
       checkout_options.merge!(analyzer.sandbox.checkout_sources)
       
       all_specs = analysis_result.specifications
+      
+      supported_platforms = analyzer.instance_variable_get("@result").targets.map { |t| t.platform.safe_string_name.downcase }
+      all_podfile_items = all_specs.map { |spec| PodfileItem.new(spec, all_specs, checkout_options, supported_platforms) }
 
-      all_podfile_specs = all_specs.select { |x| all_podfile_pods.map(&:name).include?(x.name) }
+      names = analyzer.explicit_pods().map(&:name)
 
-      deps_names = all_podfile_specs.map { |x| x.recursive_dep_names(all_specs) }.flatten.uniq
+      podfile_pods = []
+      last_count = -1 
+      while podfile_pods.count != last_count do
+        last_count = podfile_pods.count
 
-      all_podfile_specs += all_specs.select { |x| deps_names.include?(x.name) }
-      all_podfile_specs.uniq!
-      
-      return all_podfile_specs.map { |spec| PodfileItem.new(spec, all_specs, checkout_options) }
+        updated_names = []
+        names.each do |name|
+          if pod = all_podfile_items.detect { |t| t.name == name }
+            podfile_pods.push(pod)
+            updated_names += pod.dependency_names
+          end
+        end
+        
+        names = updated_names.uniq
+        podfile_pods.uniq!
+      end
+
+      return podfile_pods.sort_by(&:name)
     end
   end
 end

data/lib/pod_builder/analyzer.rb

@@ -0,0 +1,16 @@
+require 'cocoapods/installer/analyzer.rb'
+
+module Pod
+  class Installer
+    class Analyzer
+        def explicit_pods
+            pods = []
+            podfile.root_target_definitions[0].children.each do |children|
+                pods += children.dependencies
+            end
+
+            pods.flatten.uniq.sort
+        end
+    end
+  end
+end

data/lib/pod_builder/command/build.rb

@@ -3,7 +3,7 @@ require 'pod_builder/core'
 module PodBuilder
   module Command
     class Build
-      def self.call(options)          
+      def self.call 
         Configuration.check_inited
         PodBuilder::prepare_basepath
 
@@ -13,20 +13,28 @@ module PodBuilder
           return -1
         end
 
-        raise "\n\nPlease rename your Xcode installation path removing spaces, current `#{`xcode-select -p`.strip()}`\n" if `xcode-select -p`.strip().include?(" ")
+        raise "\n\nPlease rename your Xcode installation path removing spaces, current `#{`xcode-select -p`.strip()}`\n".red if `xcode-select -p`.strip().include?(" ")
 
         Podfile.sanity_check()
         check_not_building_subspecs(argument_pods)
 
-        install_update_repo = options.fetch(:update_repos, true)
+        puts "Loading Podfile".yellow
+
+        install_update_repo = OPTIONS.fetch(:update_repos, true)
         installer, analyzer = Analyze.installer_at(PodBuilder::basepath, install_update_repo)
 
         all_buildable_items = Analyze.podfile_items(installer, analyzer)
         prebuilt_items = all_buildable_items.select { |x| x.is_prebuilt }
         buildable_items = all_buildable_items - prebuilt_items
 
-        if argument_pods.first == "*"
-          argument_pods = buildable_items.map(&:root_name)
+        build_all = argument_pods.first == "*"
+        if build_all
+          argument_pods = all_buildable_items.map(&:root_name).uniq
+        else
+          argument_pods = Podfile::resolve_pod_names(argument_pods, all_buildable_items)
+          deps = all_buildable_items.select { |t| argument_pods.include?(t.root_name) }.map(&:dependency_names).flatten.map { |t| t.split("/").first }
+          argument_pods += deps
+          argument_pods.uniq!
         end
 
         available_argument_pods = argument_pods.select { |x| all_buildable_items.map(&:root_name).include?(x) }     
@@ -41,10 +49,10 @@ module PodBuilder
 
         restore_file_error = Podfile.restore_file_sanity_check
   
-        check_splitted_subspecs_are_static(all_buildable_items, options)
+        check_splitted_subspecs_are_static(all_buildable_items)
         check_pods_exists(argument_pods, all_buildable_items)
 
-        pods_to_build = resolve_pods_to_build(argument_pods, buildable_items, options)
+        pods_to_build = resolve_pods_to_build(argument_pods, buildable_items)
         buildable_items -= pods_to_build
 
         # We need to split pods to build in 3 groups
@@ -52,7 +60,7 @@ module PodBuilder
         # 2. pods to build in release
         # 3. pods to build in debug
 
-        check_not_building_development_pods(pods_to_build, options)
+        check_not_building_development_pods(pods_to_build)
 
         pods_to_build_subspecs = pods_to_build.select { |x| x.is_subspec && Configuration.subspecs_to_split.include?(x.name) }
 
@@ -73,34 +81,37 @@ module PodBuilder
         licenses = []
         
         podfiles_items.select { |x| x.count > 0 }.each do |podfile_items|
-          podfile_items = add_dependencies(podfile_items, all_buildable_items)
-          podfile_content = Podfile.from_podfile_items(podfile_items, analyzer)
+          build_configuration = podfile_items.map(&:build_configuration).uniq.first
           
-          Install.podfile(podfile_content, podfile_items, podfile_items.first.build_configuration)
-
-          licenses += license_specifiers
+          podfile_items = podfile_items.map { |t| t.recursive_dependencies(all_buildable_items) }.flatten.uniq
+          podfile_content = Podfile.from_podfile_items(podfile_items, analyzer, build_configuration)
+          
+          licenses += Install.podfile(podfile_content, podfile_items, podfile_items.first.build_configuration)
           
           # remove lockfile which gets unexplicably created
           FileUtils.rm_f(PodBuilder::basepath("Podfile.lock"))
         end
 
-        clean_frameworks_folder(all_buildable_items)
+        Clean::prebuilt_items(all_buildable_items)
 
         Licenses::write(licenses, all_buildable_items)
 
-        GenerateLFS::call(nil)
+        GenerateLFS::call()
         Podspec::generate(all_buildable_items, analyzer)
 
         builded_pods = podfiles_items.flatten
-        builded_pods_and_deps = add_dependencies(builded_pods, all_buildable_items).select { |x| !x.is_prebuilt }
+        
+        builded_pods_and_deps = podfiles_items.flatten.map { |t| t.recursive_dependencies(all_buildable_items) }.flatten.uniq
+        builded_pods_and_deps.select! { |x| !x.is_prebuilt }
+        
         Podfile::write_restorable(builded_pods_and_deps + prebuilt_pods_to_install, all_buildable_items, analyzer)     
-        if !options.has_key?(:skip_prebuild_update)   
+        if !OPTIONS.has_key?(:skip_prebuild_update)   
           Podfile::write_prebuilt(all_buildable_items, analyzer)
         end
 
         Podfile::install
 
-        sanity_checks(options)
+        sanity_checks
 
         if (restore_file_error = restore_file_error) && Configuration.restore_enabled
           puts "\n\n⚠️ Podfile.restore was found invalid and was overwritten. Error:\n #{restore_file_error}".red
@@ -112,88 +123,6 @@ module PodBuilder
 
       private
 
-      def self.license_specifiers
-        acknowledge_files = Dir.glob("#{PodBuilder::Configuration.build_path}/Pods/**/*acknowledgements.plist")
-        raise "Too many ackwnoledge files found" if acknowledge_files.count > 1
-
-        if acknowledge_file = acknowledge_files.first
-          plist = CFPropertyList::List.new(:file => acknowledge_file)
-          data = CFPropertyList.native_types(plist.value)
-          
-          return data["PreferenceSpecifiers"]
-        end
-
-        return []
-      end
-
-      def self.add_dependencies(pods, buildable_items)
-        pods.dup.each do |pod|
-          build_configuration = pods.first.build_configuration
-
-          dependencies = pod.dependencies(buildable_items).select { |x| !pods.include?(x) && !pod.has_common_spec(x.name) }
-          dependencies.each { |x| x.build_configuration = build_configuration }
-          pods = dependencies + pods # dependencies should come first
-        end
-        
-        return pods
-      end
-
-      def self.expected_common_dependencies(pods_to_build, buildable_items, options)
-        warned_expected_pod_list = []
-        expected_pod_list = []
-        errors = []
-
-        pods_to_build.each do |pod_to_build|
-          pod_to_build.dependency_names.each do |dependency|
-            unless buildable_items.detect { |x| x.root_name == dependency || x.name == dependency } != nil
-              next
-            end
-
-            buildable_items.each do |buildable_pod|
-              unless !pod_to_build.dependency_names.include?(buildable_pod.name)
-                next
-              end
-
-              if buildable_pod.dependency_names.include?(dependency) && !buildable_pod.has_subspec(dependency) && !buildable_pod.has_common_spec(dependency) then
-                expected_pod_list += pods_to_build.map(&:root_name) + [buildable_pod.root_name]
-                expected_pod_list.uniq!
-
-                expected_list = expected_pod_list.join(" ")
-                if !warned_expected_pod_list.include?(expected_list)
-                  errors.push("Can't build #{pod_to_build.name} because it has common dependencies (#{dependency}) with #{buildable_pod.name}.\n\nUse `pod_builder build #{expected_list}` instead or use `pod_builder build -a #{pod_to_build.name}` to automatically resolve missing dependencies\n\n")
-                  errors.uniq!
-                  warned_expected_pod_list.push(expected_list)
-
-                  if options.has_key?(:auto_resolve_dependencies)
-                    puts "`#{pod_to_build.name}` has the following dependencies:\n`#{pod_to_build.dependency_names.join("`, `")}`\nWhich are in common with `#{buildable_pod.name}` and requires it to be recompiled\n\n".yellow
-                  end
-                end
-              end
-            end
-          end
-        end
-
-        return expected_pod_list, errors
-      end
-
-      def self.expected_parent_dependencies(pods_to_build, buildable_items, options)
-        expected_pod_list = []
-        errors = []
-
-        buildable_items_dependencies = buildable_items.map(&:dependency_names).flatten.uniq
-        pods_to_build.each do |pod_to_build|
-          if buildable_items_dependencies.include?(pod_to_build.name)
-            parent = buildable_items.detect { |x| x.dependency_names.include?(pod_to_build.name) }
-
-            expected_pod_list += (pods_to_build + [parent]).map(&:root_name)
-            expected_pod_list.uniq!
-            errors.push("Can't build #{pod_to_build.name} because it is a dependency of #{parent.name}.\n\nUse `pod_builder build #{expected_pod_list.join(" ")}` instead\n\n")
-          end
-        end
-
-        return expected_pod_list, errors
-      end
-
       def self.check_not_building_subspecs(pods_to_build)
         pods_to_build.each do |pod_to_build|
           if pod_to_build.include?("/")
@@ -203,7 +132,7 @@ module PodBuilder
       end
 
       def self.check_pods_exists(pods, buildable_items)
-        raise "Empty Podfile?" if buildable_items.nil?
+        raise "\n\nEmpty Podfile?".red if buildable_items.nil?
 
         buildable_items = buildable_items.map(&:root_name)
         pods.each do |pod|
@@ -211,7 +140,7 @@ module PodBuilder
         end
       end
 
-      def self.check_splitted_subspecs_are_static(all_buildable_items, options)
+      def self.check_splitted_subspecs_are_static(all_buildable_items)
         non_static_subspecs = all_buildable_items.select { |x| x.is_subspec && x.is_static == false }
         non_static_subspecs_names = non_static_subspecs.map(&:name)
 
@@ -221,11 +150,11 @@ module PodBuilder
           return
         end
 
-        warn_message = "The following pods `#{invalid_subspecs.join(" ")}` are non static frameworks which are being splitted over different targets. Beware that this is an unsafe setup as per https://github.com/CocoaPods/CocoaPods/issues/5708 and https://github.com/CocoaPods/CocoaPods/issues/5643\n\nYou can ignore this error by passing the `--allow-warnings` flag to the build command\n"
-        if options[:allow_warnings]
+        warn_message = "The following pods `#{invalid_subspecs.join(" ")}` are non static binaries which are being splitted over different targets. Beware that this is an unsafe setup as per https://github.com/CocoaPods/CocoaPods/issues/5708 and https://github.com/CocoaPods/CocoaPods/issues/5643\n\nYou can ignore this error by passing the `--allow-warnings` flag to the build command\n"
+        if OPTIONS[:allow_warnings]
           puts "\n\n⚠️  #{warn_message}".yellow
         else
-          raise "\n\n🚨️  #{warn_message}".yellow
+          raise "\n\n🚨️  #{warn_message}".red
         end
       end
 
@@ -239,14 +168,14 @@ module PodBuilder
           pods_with_unaligned_build_configuration = pods_with_common_deps.select { |x| x.build_configuration != pod.build_configuration }
           pods_with_unaligned_build_configuration.map!(&:name)
 
-          raise "Dependencies of `#{pod.name}` don't have the same build configuration (#{pod.build_configuration}) of `#{pods_with_unaligned_build_configuration.join(",")}`'s dependencies" if pods_with_unaligned_build_configuration.count > 0
+          raise "\n\nDependencies of `#{pod.name}` don't have the same build configuration (#{pod.build_configuration}) of `#{pods_with_unaligned_build_configuration.join(",")}`'s dependencies".red if pods_with_unaligned_build_configuration.count > 0
         end
       end
 
-      def self.check_not_building_development_pods(pods, options)
-        if (development_pods = pods.select { |x| x.is_development_pod }) && development_pods.count > 0 && (options[:allow_warnings].nil?  && Configuration.allow_building_development_pods == false)
+      def self.check_not_building_development_pods(pods)
+        if (development_pods = pods.select { |x| x.is_development_pod }) && development_pods.count > 0 && (OPTIONS[:allow_warnings].nil?  && Configuration.allow_building_development_pods == false)
           pod_names = development_pods.map(&:name).join(", ")
-          raise "The following pods are in development mode: `#{pod_names}`, won't proceed building.\n\nYou can ignore this error by passing the `--allow-warnings` flag to the build command\n"
+          raise "\n\nThe following pods are in development mode: `#{pod_names}`, won't proceed building.\n\nYou can ignore this error by passing the `--allow-warnings` flag to the build command\n".red
         end
       end
 
@@ -259,7 +188,7 @@ module PodBuilder
         return buildable_subspecs - pods_to_build
       end
 
-      def self.sanity_checks(options)
+      def self.sanity_checks
         lines = File.read(PodBuilder::project_path("Podfile")).split("\n")
         stripped_lines = lines.map { |x| Podfile.strip_line(x) }.select { |x| !x.start_with?("#")}
 
@@ -267,7 +196,7 @@ module PodBuilder
 
         if !expected_stripped.all? { |x| stripped_lines.include?(x) }
           warn_message = "PodBuilder's post install actions missing from application Podfile!\n"
-          if options[:allow_warnings]
+          if OPTIONS[:allow_warnings]
             puts "\n\n⚠️  #{warn_message}".yellow
           else
             raise "\n\n🚨️  #{warn_message}".red
@@ -275,60 +204,14 @@ module PodBuilder
         end
       end
 
-      def self.resolve_pods_to_build(argument_pods, buildable_items, options)
+      def self.resolve_pods_to_build(argument_pods, buildable_items)
         pods_to_build = []
         
-        fns = [method(:expected_common_dependencies), method(:expected_parent_dependencies)]
-        fns.each do |fn|
-          loop do
-            pods_to_build = buildable_items.select { |x| argument_pods.include?(x.root_name) }
-            pods_to_build += other_subspecs(pods_to_build, buildable_items)
-            tmp_buildable_items = buildable_items - pods_to_build
-
-            expected_pods, errors = fn.call(pods_to_build, tmp_buildable_items, options)
-            if expected_pods.count > 0
-              if !options.has_key?(:auto_resolve_dependencies) && expected_pods.count > 0
-                raise "\n\n#{errors.join("\n")}".red
-              else
-                argument_pods = expected_pods.uniq
-              end  
-            end
-            
-            if !options.has_key?(:auto_resolve_dependencies) || expected_pods.count == 0
-              break
-            end
-          end  
-        end
+        pods_to_build = buildable_items.select { |x| argument_pods.include?(x.root_name) }
+        pods_to_build += other_subspecs(pods_to_build, buildable_items)
 
         return pods_to_build
-      end
-
-      def self.clean_frameworks_folder(buildable_items)
-        puts "Cleaning framework folder".yellow
-
-        expected_frameworks = buildable_items.map { |x| "#{x.module_name}.framework" }
-        expected_frameworks += buildable_items.map { |x| x.vendored_items }.flatten.map { |x| File.basename(x) }
-        expected_frameworks.uniq!
-
-        existing_frameworks = Dir.glob("#{PodBuilder::basepath("Rome")}/*.framework")
-
-        existing_frameworks.each do |existing_framework|
-          existing_framework_name = File.basename(existing_framework)
-          if !expected_frameworks.include?(existing_framework_name)
-            puts "Cleanining up `#{existing_framework_name}`, no longer found among dependencies".blue
-            FileUtils.rm_rf(existing_framework)
-          end
-        end
-
-        existing_dsyms = Dir.glob("#{PodBuilder::basepath("dSYM")}/**/*.dSYM")
-        existing_dsyms.each do |existing_dsym|
-          existing_dsym_name = File.basename(existing_dsym)
-          if !expected_frameworks.include?(existing_dsym_name.gsub(".dSYM", ""))
-            puts "Cleanining up `#{existing_dsym_name}`, no longer found among dependencies".blue
-            FileUtils.rm_rf(existing_dsym)
-          end
-        end
-      end
+      end      
     end
   end
 end