pod-builder 2.2.1 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f3d87c061e79af94ca1274b3d60765a3ba850c8a0d2b443fbd810dbb0585d0e0
-  data.tar.gz: d2db90c765d618eada490386fa07ffe412d69afefae36aef4f0a183e43019d71
+  metadata.gz: '09cbb24dc2fb49946058f0459990c762afd54e93dfe280289203348162272611'
+  data.tar.gz: 87fc03146c5a8a7fbe2adcc895e8a45c8b24b517b00c557236245902df543267
 SHA512:
-  metadata.gz: 5a6664929a3eb5d564fe8e83345bddf9a746abda7c88adc8600e2fece2b4d3b9ce22d864ae4d363bd0e5fd5fa80bb0b083cbb648ec2cdb04c3f0c01c72a237aa
-  data.tar.gz: ff51be1741b388ba51cbb494203d5c1d6c9f593926ab1b5d0a86940f271dab9423c5f43b7866668007d7393a2d44c94a671205a1a61bfc5dfee2fa06b0b3a04f
+  metadata.gz: 221ccda81fbd9b41d4439ebaa6a8ea3776e4c1d91ea3b24d268fa54203e67e757160ad0e42595527e0e610845ec1faf546aa9d480463f808aae28c94df4710e4
+  data.tar.gz: ef65cce139a54a6ae93e5262e480a3608e9b268a51cf413da58e314d29568c6981f5b198dc1061d93d102ec112935dc890d7e5b009fabd0cc4cc88cce52e9650

data/README.md

@@ -216,7 +216,7 @@ As an example here we're setting `module_name` in Google's Advertising SDK:
 }
 ```
 
-### `skip_pods`
+#### `skip_pods`
 
 You may want to skip some pods to be prebuilt, you can do that as follows:
 
@@ -229,7 +229,7 @@ You may want to skip some pods to be prebuilt, you can do that as follows:
 ```
 
 
-### `force_prebuild_pods`
+#### `force_prebuild_pods`
 
 You may want to force some pods to be prebuilt, this might be the case for prebuilt ones (pods with a single vendored .framework) which are dependencies of othere pods
 
@@ -244,7 +244,9 @@ You may want to force some pods to be prebuilt, this might be the case for prebu
 
 #### `build_settings`
 
-Xcode build settings to use. You can override the default values which are:
+These settings allow you to specify the build settings to use when compiling prebuilt items. These values won't be reflected in your project once the pod has been built.
+
+You can override the default values which are:
 
 ```json
 {
@@ -280,9 +282,17 @@ Like `build_settings` but per pod. Pod name can also refer to subspec.
 
 Specify which build system to use to compile frameworks. Either `Legacy` (standard build system) or `Latest` (new build system). Default value: `Latest`.
 
-#### `build_xcframeworks`
+#### `build_xcframeworks_all`
+
+Specify if PodBuilder should build all pods as .xcframeworks. Will enable `library_evolution_support`. Default value: false
+
+#### `build_xcframeworks_exclude`
+
+Specify which pods PodBuilder should NOT be built as .xcframeworks when `build_xcframeworks_all` is true. Default value: []
+
+#### `build_xcframeworks_include`
 
-Specify if PodBuilder will build .xcframeworks. Will enable `library_evolution_support`. Default value: false
+Specify which pods PodBuilder should be built as .xcframeworks. Will enable `library_evolution_support`. Default value: []
 
 #### `library_evolution_support`
 
@@ -327,11 +337,8 @@ The podspec of the Pod you're trying to build doesn't specify the swift_version
 
 ```json
 "spec_overrides": {
-    "Google-Mobile-Ads-SDK": {
-      "module_name": "GoogleMobileAds"
-    },
     "PodWithError": {
-      "swift_version": "5.0"
+      "swift_version": "5.3"
     }
 }
 ```
@@ -357,7 +364,7 @@ Please open an issue here. You may also add the name of the pod to the [`skip_po
 
 # Git LFS
 
-We high encourage to use PodBuilder in combination with Git LFS. Tacking PodBuilder/Prebuilt/**/*.framework/* and PodBuilder/Prebuilt/**/*.a and PodBuilder/dSYM/**/* will ensure that your repo size stays under control with all the benefits of having prebuilt dependencies ready to use.
+We high encourage to use PodBuilder in combination with Git LFS. Tacking `PodBuilder/Prebuilt/**/*.framework/*` and `PodBuilder/Prebuilt/**/*.a` and `PodBuilder/dSYM/**/*` will ensure that your repo size stays under control with all the benefits of having prebuilt dependencies ready to use.
 
 
 # Try it out!

data/exe/pod_builder

@@ -76,9 +76,6 @@ Options:
         opts.on("-w", "--allow-warnings", "Allow warnings") do |o|
           OPTIONS[:allow_warnings] = o
         end
-        opts.on("-r", "--parent-deps", "Include all pods that depend on the specified <PODNAME...>") do |o|
-          OPTIONS[:resolve_parent_dependencies] = true
-        end
         opts.on("-s", "--no-stdin", "Never request interaction with sdtin (e.g. in CI environment)") do |o|
           OPTIONS[:no_stdin_available] = o
         end

data/lib/pod_builder/command/build.rb

@@ -32,8 +32,6 @@ module PodBuilder
           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
 
@@ -43,8 +41,6 @@ module PodBuilder
         }
         argument_pods = available_argument_pods.uniq
         
-        prebuilt_pods_to_install = prebuilt_items.select { |x| argument_pods.include?(x.root_name) }
-
         Podfile.restore_podfile_clean(all_buildable_items)
 
         restore_file_error = Podfile.restore_file_sanity_check
@@ -54,28 +50,53 @@ module PodBuilder
         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
+        argument_pods += pods_to_build.map(&:root_name)
+        argument_pods.uniq!
+
+        # We need to split pods to build in 4 groups
         # 1. pods to build in release
         # 2. pods to build in debug
+        # 3. pods to build in release as xcframeworks
+        # 4. pods to build in debug as xcframeworks
 
         check_not_building_development_pods(pods_to_build)
 
+        # We need to recursively add dependencies to properly split pods in groups.
+        # Example:        
+        # 1. PodA has a dep to PodB
+        # 2. PodB is marked to be built as xcframework
+        # 3. We rebuild PodA only (pods_to_build contains only PodA)
+        # 4. We need to add dependencies recursively so that PodB is is added to pods_to_build_release_xcframework
+        pods_to_build = pods_to_build.map { |t| t.recursive_dependencies(all_buildable_items) }.flatten.uniq
+
         pods_to_build_debug = pods_to_build.select { |x| x.build_configuration == "debug" }
         pods_to_build_release = pods_to_build - pods_to_build_debug
 
+        pods_to_build_debug_xcframework = pods_to_build_debug.select { |x| x.build_xcframework }
+        pods_to_build_debug -= pods_to_build_debug_xcframework
+
+        pods_to_build_release_xcframework = pods_to_build_release.select { |x| x.build_xcframework }
+        pods_to_build_release -= pods_to_build_release_xcframework
+
         check_dependencies_build_configurations(all_buildable_items)
 
-        podfiles_items = [pods_to_build_debug] + [pods_to_build_release]
+        # When building mixed framwork/xcframeworks pods xcframeworks should be built last 
+        # so that the .xcframework overwrite the .framwork if the same pod needs to be built
+        # in both ways. 
+        # For example we might have configured to build onlt PodA as xcframework, another pod
+        # PodB has a dependency to PodA. When Building PodB, PodA gets rebuilt as .framework
+        # but then PodA gets rebuilt again as .xcframework overwriting the .framework.
+        podfiles_items = [pods_to_build_debug] + [pods_to_build_release] + [pods_to_build_debug_xcframework] + [pods_to_build_release_xcframework]
 
         install_using_frameworks = Podfile::install_using_frameworks(analyzer)
         if Configuration.react_native_project
           if install_using_frameworks
-            raise "\n\nOnly static library packaging currently supported for react native projects. Please remove 'use_frameworks!' in #{PodBuilder::basepath("Podfile")}".red 
+            raise "\n\nOnly static library packaging currently supported for react native projects. Please remove 'use_frameworks!' in #{PodBuilder::basepath("Podfile")}\n".red 
           end  
           prepare_defines_modules_override(all_buildable_items)
         else
           unless install_using_frameworks
-            raise "\n\nOnly framework packaging currently supported. Please add 'use_frameworks!' at root level (not nested in targets) in #{PodBuilder::basepath("Podfile")}".red
+            raise "\n\nOnly framework packaging currently supported. Please add 'use_frameworks!' at root level (not nested in targets) in #{PodBuilder::basepath("Podfile")}\n".red
           end  
         end
         
@@ -84,11 +105,17 @@ module PodBuilder
         install_result = InstallResult.new
         podfiles_items.reject { |x| x.empty? }.each do |podfile_items|
           build_configuration = podfile_items.map(&:build_configuration).uniq.first
-          
+
+          # We need to recursively find dependencies again because some of the required dependencies might have been moved to a separate group
+          # Example:
+          # 1. PodA has a dep to PodB
+          # 2. PodB is marked to be built as xcframework -> PodB will be added to pods_to_build_release_xcframework and won't be present in
+          # pods_to_build_release and therefore build will fail
           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, install_using_frameworks, build_catalyst, Configuration.build_xcframeworks)
           
-          install_result += Install.podfile(podfile_content, podfile_items, podfile_items.first.build_configuration)          
+          podfile_content = Podfile.from_podfile_items(podfile_items, analyzer, build_configuration, install_using_frameworks, build_catalyst, podfile_items.first.build_xcframework)
+          
+          install_result += Install.podfile(podfile_content, podfile_items, argument_pods, podfile_items.first.build_configuration)          
           
           FileUtils.rm_f(PodBuilder::basepath("Podfile.lock"))
         end
@@ -106,6 +133,7 @@ module PodBuilder
         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 }
         
+        prebuilt_pods_to_install = prebuilt_items.select { |x| argument_pods.include?(x.root_name) }
         Podfile::write_restorable(builded_pods_and_deps + prebuilt_pods_to_install, all_buildable_items, analyzer)     
         if !OPTIONS.has_key?(:skip_prebuild_update)   
           Podfile::write_prebuilt(all_buildable_items, analyzer)
@@ -129,7 +157,7 @@ module PodBuilder
         build_settings = installer.analysis_result.targets.map { |t| t.user_project.root_object.targets.map { |u| u.build_configuration_list.build_configurations.map { |v| v.build_settings } } }.flatten
         build_catalyst = build_settings.detect { |t| t["SUPPORTS_MACCATALYST"] == "YES" } != nil 
         
-        puts "\nTo support Catalyst you should enable 'build_xcframeworks' in PodBuilder.json\n".red if build_catalyst && !Configuration.build_xcframeworks
+        puts "\nTo support Catalyst you should enable 'build_xcframeworks' in PodBuilder.json\n".red if build_catalyst && !Configuration.build_xcframeworks_all
 
         return build_catalyst
       end
@@ -151,11 +179,11 @@ module PodBuilder
       end
 
       def self.check_pods_exists(pods, buildable_items)
-        raise "\n\nEmpty Podfile?".red if buildable_items.nil?
+        raise "\n\nEmpty Podfile?\n".red if buildable_items.nil?
 
         buildable_items = buildable_items.map(&:root_name)
         pods.each do |pod|
-          raise "\n\nPod `#{pod}` wasn't found in Podfile.\n\nFound:\n#{buildable_items.join("\n")}\n\n".red if !buildable_items.include?(pod)
+          raise "\n\nPod `#{pod}` wasn't found in Podfile.\n\nFound:\n#{buildable_items.join("\n")}\n".red if !buildable_items.include?(pod)
         end
       end
 
@@ -169,7 +197,7 @@ 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 "\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
+          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\n".red if pods_with_unaligned_build_configuration.count > 0
         end
       end
 
@@ -200,7 +228,7 @@ module PodBuilder
           if OPTIONS[:allow_warnings]
             puts "\n\n#{warn_message}".yellow
           else
-            raise "\n\n#{warn_message}".red
+            raise "\n\n#{warn_message}\n".red
           end
         end
       end
@@ -211,15 +239,17 @@ module PodBuilder
         pods_to_build = buildable_items.select { |x| argument_pods.include?(x.root_name) }
         pods_to_build += other_subspecs(pods_to_build, buildable_items)
 
-        if OPTIONS[:resolve_parent_dependencies]
-          dependencies = []
-          buildable_items.each do |pod|
-            if !(pod.dependencies(buildable_items) & pods_to_build).empty?
-              dependencies.push(pod)
-            end
+        # Build all pods that depend on the those that were explictly passed by the user
+        dependencies = []
+        buildable_items.each do |pod|
+          if !(pod.dependencies(buildable_items) & pods_to_build).empty?
+            dependencies.push(pod)
           end
-          pods_to_build += dependencies
         end
+        log = dependencies.reject { |t| pods_to_build.map(&:root_name).include?(t.root_name) }.map(&:root_name)
+        puts "Adding inverse dependencies: #{log.join(", ")}".blue
+
+        pods_to_build += dependencies
 
         return pods_to_build.uniq
       end      

data/lib/pod_builder/command/generate_lldbinit.rb

@@ -19,7 +19,7 @@ module PodBuilder
           end
           source_path = File.expand_path(source_path)  
           
-          raise "\n\nSpecified path does not exists" unless File.directory?(source_path)
+          raise "\n\nSpecified path does not exists\n".red unless File.directory?(source_path)
         end
         
         base_path = PodBuilder::basepath

data/lib/pod_builder/command/init.rb

@@ -122,11 +122,11 @@ module PodBuilder
         # React-Core.podspec
         file = "React-Core.podspec"
         paths = Dir.glob("#{PodBuilder::git_rootpath}/node_modules/**/#{file}")
-        raise "Unexpected number of #{file} found" if paths.count != 1
+        raise "\n\nUnexpected number of #{file} found\n".red if paths.count != 1
 
         content = File.read(paths[0])
         expected_header_search_path_prefix = "s.pod_target_xcconfig    = { \"HEADER_SEARCH_PATHS\" => \""
-        raise "Expected header search path entry not found" unless content.include?(expected_header_search_path_prefix)
+        raise "\n\nExpected header search path entry not found\n".red unless content.include?(expected_header_search_path_prefix)
 
         content.sub!(expected_header_search_path_prefix, "#{expected_header_search_path_prefix}\\\"$(PODS_ROOT)/Headers/Public/Flipper-Folly\\\" ")
         File.write(paths[0], content)        
@@ -134,11 +134,11 @@ module PodBuilder
         # React-CoreModules.podspec
         file = "React-CoreModules.podspec"
         paths = Dir.glob("#{PodBuilder::git_rootpath}/node_modules/**/#{file}")
-        raise "Unexpected number of #{file} found" if paths.count != 1
+        raise "\n\nUnexpected number of #{file} found\n".red if paths.count != 1
 
         content = File.read(paths[0])
         expected_header_search_path_prefix = "\"HEADER_SEARCH_PATHS\" => \""
-        raise "Expected header search path entry not found" unless content.include?(expected_header_search_path_prefix)
+        raise "\n\nExpected header search path entry not found\n".red unless content.include?(expected_header_search_path_prefix)
 
         content.sub!(expected_header_search_path_prefix, "#{expected_header_search_path_prefix}\\\"$(PODS_ROOT)/Headers/Public/Flipper-Folly\\\" ")
         File.write(paths[0], content)

data/lib/pod_builder/command/switch.rb

@@ -13,19 +13,23 @@ module PodBuilder
           return -1
         end
 
+        pods_not_found = []
         pod_names_to_switch = []
         argument_pods.each do |pod|
           pod_name_to_switch = pod
           pod_name_to_switch = Podfile::resolve_pod_names_from_podfile([pod_name_to_switch]).first
-          raise "\n\nDid not find pod '#{pod}'".red if pod_name_to_switch.nil?
-          
-          check_not_building_subspec(pod_name_to_switch)  
 
-          pod_names_to_switch.push(pod_name_to_switch)
+          if pod_name_to_switch.nil?
+            raise "\n\n'#{pod}' not found in PodBuilder's Podfile.\n\nYou might need to explictly add:\n\n    pod '#{pod}'\n\nto #{PodBuilder::basepath("Podfile")}\n".red
+          else
+            check_not_building_subspec(pod_name_to_switch)  
+
+            pod_names_to_switch.push(pod_name_to_switch)  
+          end          
         end
 
         if OPTIONS[:resolve_parent_dependencies] == true
-          install_update_repo = OPTIONS.fetch(:update_repos, true)
+          install_update_repo = OPTIONS.fetch(:update_repos, false)
           installer, analyzer = Analyze.installer_at(PodBuilder::basepath, install_update_repo)
   
           all_buildable_items = Analyze.podfile_items(installer, analyzer)
@@ -55,7 +59,7 @@ module PodBuilder
 
             podspec_content = File.read(podspec_path)
 
-            regex = "p\\d\\.dependency '(.*)'"
+            regex = "p\\d\\.dependency ['|\"](.*)['|\"]"
 
             podspec_content.each_line do |line|
               matches = line.match(/#{regex}/)
@@ -107,15 +111,15 @@ module PodBuilder
                 next
               end
     
-              matches = line.gsub("\"", "'").match(/pod '(.*?)',(.*)/)
-              if matches&.size == 3
+              matches = line.gsub("\"", "'").match(/pod '(.*?)'/)
+              if matches&.size == 2
                 if matches[1].split("/").first == pod_name_to_switch
                   default_entries[current_section] = line
                 end  
               end
             end
     
-            raise "\n\n'#{pod_name_to_switch}' not found in #{podfile_path}".red if default_entries.keys.count == 0
+            raise "\n\n'#{pod_name_to_switch}' not found in PodBuilder's Podfile.\n\nYou might need to explictly add:\n\n    pod '#{pod_name_to_switch}'\n\nto #{podfile_path}\n".red if default_entries.keys.count == 0
           end
 
           if development_path.nil? 
@@ -137,8 +141,8 @@ module PodBuilder
               current_section = line.split(" ")[1]
             end
 
-            matches = line.gsub("\"", "'").match(/pod '(.*?)',(.*)/)
-            if matches&.size == 3
+            matches = line.gsub("\"", "'").match(/pod '(.*?)'/)
+            if matches&.size == 2
               if matches[1].split("/").first == pod_name_to_switch
                 case OPTIONS[:switch_mode]
                 when "prebuilt"
@@ -166,6 +170,14 @@ module PodBuilder
                     if line.include?("# pb<") && marker = line.split("# pb<").last
                       default_line = default_line.chomp("\n") + " # pb<#{marker}"
                     end
+                    if (path_match = default_line.match(/:path => '(.*?)'/)) && path_match&.size == 2
+                      original_path = path_match[1]
+                      if !is_absolute_path(original_path)
+                        updated_path = Pathname.new(PodBuilder::basepath(original_path)).relative_path_from(Pathname.new(PodBuilder::project_path)).to_s
+                        default_line.gsub!(":path => '#{original_path}'", ":path => '#{updated_path}'")
+                      end
+                    end
+
                     lines.append(default_line)
                     next
                   elsif
@@ -190,7 +202,11 @@ module PodBuilder
         return 0
       end
       
-      private     
+      private 
+      
+      def self.is_absolute_path(path)
+        return ["~", "/"].any? { |t| path.start_with?(t) }
+      end  
 
       def self.find_podspec(podname)
         unless Configuration.development_pods_paths.count > 0
@@ -216,7 +232,7 @@ module PodBuilder
             
       def self.check_not_building_subspec(pod_to_switch)
         if pod_to_switch.include?("/")
-          raise "\n\nCan't switch subspec #{pod_to_switch} refer to podspec name.\n\nUse `pod_builder switch #{pod_to_switch.split("/").first}` instead\n\n".red
+          raise "\n\nCan't switch subspec #{pod_to_switch} refer to podspec name.\n\nUse `pod_builder switch #{pod_to_switch.split("/").first}` instead\n".red
         end
       end
     end

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

@@ -39,19 +39,10 @@ module PodBuilder
         "ENABLE_BITCODE": "NO"
       }
     }.freeze
-    DEFAULT_SKIP_PODS = ["GoogleMaps", "React-RCTFabric", "React-Core", "React-CoreModules"] # Not including React-RCTNetwork might loose some debug warnings
-
-    DEFAULT_FORCE_PREBUILD_PODS = []
-    DEFAULT_BUILD_SYSTEM = "Latest".freeze # either Latest (New build system) or Legacy (Standard build system)
-    DEFAULT_LIBRARY_EVOLUTION_SUPPORT = false
-    DEFAULT_PLATFORMS = ["iphoneos", "iphonesimulator", "appletvos", "appletvsimulator"].freeze
-    DEFAULT_BUILD_USING_REPO_PATHS = false
-    DEFAULT_BUILD_XCFRAMEWORKS = false
     
     private_constant :DEFAULT_BUILD_SETTINGS
     private_constant :DEFAULT_BUILD_SETTINGS_OVERRIDES
-    private_constant :DEFAULT_BUILD_SYSTEM
-    private_constant :DEFAULT_LIBRARY_EVOLUTION_SUPPORT
+    private_constant :DEFAULT_SPEC_OVERRIDE
     
     class <<self      
       attr_accessor :allow_building_development_pods
@@ -81,19 +72,22 @@ module PodBuilder
       attr_accessor :build_using_repo_paths
       attr_accessor :react_native_project
       attr_accessor :lldbinit_name
-      attr_accessor :build_xcframeworks
+      attr_accessor :build_xcframeworks_all
+      attr_accessor :build_xcframeworks_include
+      attr_accessor :build_xcframeworks_exclude
     end
-    
-    @allow_building_development_pods = false
+
     @build_settings = DEFAULT_BUILD_SETTINGS
     @build_settings_overrides = DEFAULT_BUILD_SETTINGS_OVERRIDES
-    @build_system = DEFAULT_BUILD_SYSTEM
-    @library_evolution_support = DEFAULT_LIBRARY_EVOLUTION_SUPPORT
-    @base_path = "PodBuilder" # Not nice. This value is used only for initial initization. Once config is loaded it will be an absolute path. FIXME
     @spec_overrides = DEFAULT_SPEC_OVERRIDE
+
+    @allow_building_development_pods = false
+    @build_system = "Latest".freeze # either Latest (New build system) or Legacy (Standard build system)
+    @library_evolution_support = false
+    @base_path = "PodBuilder" # Not nice. This value is used only for initial initization. Once config is loaded it will be an absolute path. FIXME
     @skip_licenses = []
-    @skip_pods = DEFAULT_SKIP_PODS
-    @force_prebuild_pods = DEFAULT_FORCE_PREBUILD_PODS
+    @skip_pods = ["GoogleMaps", "React-RCTFabric", "React-Core", "React-CoreModules"] # Not including React-RCTNetwork might loose some debug warnings
+    @force_prebuild_pods = []
     @license_filename = "Pods-acknowledgements"
     @development_pods_paths = []
     @build_base_path = "/tmp/pod_builder".freeze
@@ -110,11 +104,13 @@ module PodBuilder
     @use_bundler = false
     @deterministic_build = false
 
-    @supported_platforms = DEFAULT_PLATFORMS
-    @build_using_repo_paths = DEFAULT_BUILD_USING_REPO_PATHS
+    @supported_platforms = ["iphoneos", "iphonesimulator", "appletvos", "appletvsimulator"].freeze
+    @build_using_repo_paths = false
     @react_native_project = false
 
-    @build_xcframeworks = DEFAULT_BUILD_XCFRAMEWORKS
+    @build_xcframeworks_all = false
+    @build_xcframeworks_include = []
+    @build_xcframeworks_exclude = []
     
     def self.check_inited
       raise "\n\nNot inited, run `pod_builder init`\n".red if podbuilder_path.nil?
@@ -218,12 +214,22 @@ module PodBuilder
             Configuration.react_native_project = value
           end
         end
-        if value = json["build_xcframeworks"]
+        if value = json["build_xcframeworks_all"]
           if [TrueClass, FalseClass].include?(value.class)
-            Configuration.build_xcframeworks = value
+            Configuration.build_xcframeworks_all = value
           end
         end
-        
+        if value = json["build_xcframeworks_include"]
+          if value.is_a?(Array)
+            Configuration.build_xcframeworks_include = value
+          end
+        end
+        if value = json["build_xcframeworks_exclude"]
+          if value.is_a?(Array)
+            Configuration.build_xcframeworks_exclude = value
+          end
+        end
+
         Configuration.build_settings.freeze
 
         sanity_check()
@@ -279,6 +285,11 @@ module PodBuilder
           puts "PodBuilder.json contains '#{pod}' both in `force_prebuild_pods` and `skip_pods`. Will force prebuilding.".yellow
         end
       end
+      if Configuration.build_xcframeworks_all
+        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 "\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
     
     def self.config_path

data/lib/pod_builder/core.rb

@@ -132,7 +132,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 +148,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 +179,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,44 +280,66 @@ 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
 
-        podfile_content.gsub!("%%%prebuilt_root_paths%%%", prebuilt_root_paths.to_s)
+        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_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)
 
@@ -379,6 +401,7 @@ module PodBuilder
       pod_names.each do |pod_name|   
         root_name = pod_name.split("/").first
 
+        # Remove existing files
         items_to_delete = Dir.glob("#{PodBuilder::prebuiltpath(root_name)}/**/*")
         items_to_delete.each { |t| PodBuilder::safe_rm_rf(t) }
       end
@@ -393,6 +416,14 @@ module PodBuilder
           next
         end
 
+        if podfile_item = podfile_items.detect { |t| t.root_name == pod_name }
+          if Dir.glob("#{source_path}/**/Modules/**/*.swiftmodule/*.swiftinterface").count > 0
+            # We can safely remove .swiftmodule if .swiftinterface exists
+            swiftmodule_files = Dir.glob("#{source_path}/**/Modules/**/*.swiftmodule/*.swiftmodule")
+            swiftmodule_files.each { |t| PodBuilder::safe_rm_rf(t) }
+          end
+        end
+
         unless Dir.glob("#{source_path}/**/*").select { |t| File.file?(t) }.empty?
           destination_folder = PodBuilder::prebuiltpath(root_name)
           FileUtils.mkdir_p(destination_folder)
@@ -448,15 +479,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
       
@@ -41,6 +41,9 @@ module PodBuilder
         build_settings["ONLY_ACTIVE_ARCH"] = "NO"
         build_settings["DEBUG_INFORMATION_FORMAT"] = "dwarf-with-dsym"
 
+        # https://thi.imhttps://thi.im/posts/swift-serialize-debugging-options/
+        build_settings["SWIFT_SERIALIZE_DEBUGGING_OPTIONS"] = "NO"
+
         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
@@ -57,11 +60,7 @@ module PodBuilder
 
         if Configuration.build_system == "Legacy"
           build_settings["BUILD_LIBRARY_FOR_DISTRIBUTION"] = "NO"
-          raise "\n\nCan't enable library evolution support with legacy build system!".red if Configuration.library_evolution_support
-        elsif Configuration.library_evolution_support
-          build_settings["BUILD_LIBRARY_FOR_DISTRIBUTION"] = "YES"
-        end
-        if Configuration.build_xcframeworks
+        elsif Configuration.library_evolution_support || item.build_xcframework
           build_settings["BUILD_LIBRARY_FOR_DISTRIBUTION"] = "YES"
         end
 
@@ -355,7 +354,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
@@ -397,9 +396,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
@@ -444,13 +443,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"))
@@ -608,12 +607,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)
 
@@ -625,7 +624,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

@@ -103,27 +103,27 @@ module PodBuilder
     #
     attr_accessor :libraries
 
-    # @return [String] source_files
+    # @return [String] Source_files
     #
     attr_accessor :source_files
 
-    # @return [String] license
+    # @return [String] License
     #
     attr_accessor :license
 
-    # @return [String] summary
+    # @return [String] Summary
     #
     attr_accessor :summary
 
-    # @return [Hash] source
+    # @return [Hash] Source
     #
     attr_accessor :source
 
-    # @return [Hash] authors
+    # @return [Hash] Authors
     #
     attr_accessor :authors
 
-    # @return [String] homepage
+    # @return [String] Homepage
     #
     attr_accessor :homepage
 
@@ -134,6 +134,14 @@ module PodBuilder
     # @return [Bool] Defines module
     #
     attr_accessor :defines_module
+
+    # @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
     #
@@ -235,6 +243,15 @@ module PodBuilder
       @source = spec.root.attributes_hash.fetch("source", { "git"=>"https://github.com/Subito-it/PodBuilder.git" })
       @authors = spec.root.attributes_hash.fetch("authors", {"PodBuilder"=>"pod@podbuilder.com"})
       @homepage = spec.root.attributes_hash.fetch("homepage", "https://github.com/Subito-it/PodBuilder")
+
+      if Configuration.build_xcframeworks_all
+        build_as_xcframework = !Configuration.build_xcframeworks_exclude.include?(@root_name)
+      else
+        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)
@@ -324,33 +341,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
@@ -461,6 +451,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 = []
 

data/lib/pod_builder/podspec.rb

@@ -53,7 +53,11 @@ module PodBuilder
           resources = []
           exclude_files = []
           vendored_frameworks.each do |vendored_framework|
-            binary_path = Dir.glob(PodBuilder::prebuiltpath("#{item.root_name}/#{vendored_framework}/**/#{File.basename(vendored_framework, ".*")}")).first
+            if vendored_framework.end_with?(".xcframework")
+              binary_path = Dir.glob(PodBuilder::prebuiltpath("#{item.root_name}/#{vendored_framework}/**/#{File.basename(vendored_framework, ".*")}")).reject { |t| t.include?("simulator") }.first
+            else
+              binary_path = Dir.glob(PodBuilder::prebuiltpath("#{item.root_name}/#{vendored_framework}/**/#{File.basename(vendored_framework, ".*")}")).first
+            end
 
             next if binary_path.nil?
 
@@ -138,7 +142,7 @@ module PodBuilder
         end
         if !install_using_frameworks && spec_var == "p1" && vendored_libraries.map { |t| File.basename(t) }.include?("lib#{item.root_name}.a" )
           module_path_files = Dir.glob(PodBuilder.prebuiltpath("#{item.root_name}/**/#{item.root_name}.modulemap"))
-          raise "\n\nToo many module maps found for #{item.root_name}".red if module_path_files.count > 1
+          raise "\n\nToo many module maps found for #{item.root_name}\n".red if module_path_files.count > 1
 
           rel_path = Pathname.new(PodBuilder::prebuiltpath).relative_path_from(Pathname.new(PodBuilder::project_path("Pods"))).to_s
           prebuilt_root_var = "#{item.root_name.upcase.gsub("-", "_")}_PREBUILT_ROOT"

data/lib/pod_builder/rome/post_install.rb

@@ -6,9 +6,11 @@ module PodBuilder
   class XcodeBuildSettings
     attr_reader :platform_name
     attr_reader :build_destination
+    attr_reader :configuration
 
-    def initialize(platform_name)
+    def initialize(platform_name, configuration)
       @platform_name = platform_name
+      @configuration = configuration
    
       case platform_name
       when "iphoneos" then @build_destination = "generic/platform=iOS"
@@ -19,7 +21,7 @@ module PodBuilder
       when "tvossimulator" then @build_destination = "generic/platform=tvOS Simulator"
       when "watchos" then @build_destination = "generic/platform=watchOS"
       when "watchossimulator" then @build_destination = "generic/platform=watchOS Simulator"
-      else raise "\n\nUnknown platform '#{platform_name}'".red end
+      else raise "\n\nUnknown platform '#{platform_name}'\n".red end
     end
   end
 
@@ -63,7 +65,7 @@ module PodBuilder
         `xcrun lipo -remove arm64 #{simulator_lib} -o #{simulator_lib}`
       end
       
-      raise "Lipo failed on #{device_lib}" unless system("xcrun lipo -create -output #{device_lib} #{device_lib} #{simulator_lib}")
+      raise "\n\nLipo failed on #{device_lib}\n".red unless system("xcrun lipo -create -output #{device_lib} #{device_lib} #{simulator_lib}")
       
       merge_header_into(device_swift_header_path, simulator_swift_header_path)
       
@@ -114,7 +116,7 @@ module PodBuilder
         `xcrun lipo -remove arm64 #{simulator_lib} -o #{simulator_lib}`
       end
       
-      raise "Lipo failed on #{device_lib}" unless system("xcrun lipo -create -output #{device_lib} #{device_lib} #{simulator_lib}")
+      raise "\n\nLipo failed on #{device_lib}\n".red unless system("xcrun lipo -create -output #{device_lib} #{device_lib} #{simulator_lib}")
       
       device_headers = Dir.glob("#{device_base}/**/*.h")
       simulator_headers = Dir.glob("#{simulator_base}/**/*.h")
@@ -245,7 +247,7 @@ module PodBuilder
     output = stdout + stderr
     unless status.success?
       if raise_on_failure
-        raise "#{full_command}\n\n#{output}"
+        raise "\n\n#{full_command}\n\n#{output}\n".red
       else
         UI.message("[!] Failed: #{full_command}".red)
       end
@@ -305,7 +307,7 @@ Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_conte
   build_dir.rmtree if build_dir.directory?
 
   targets = installer_context.umbrella_targets.select { |t| t.specs.any? }
-  raise "\n\nUnsupported target count".red unless targets.count == 1
+  raise "\n\nUnsupported target count\n".red unless targets.count == 1
   target = targets.first
 
   if build_xcframeworks
@@ -313,18 +315,18 @@ Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_conte
         
     case target.platform_name
     when :ios then 
-      xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("iphoneos"), PodBuilder::XcodeBuildSettings.new("iphonesimulator")]
+      xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("iphoneos", configuration), PodBuilder::XcodeBuildSettings.new("iphonesimulator", configuration)]
       if build_catalyst
-        xcodebuild_settings += [PodBuilder::XcodeBuildSettings.new("catalyst")]
+        xcodebuild_settings += [PodBuilder::XcodeBuildSettings.new("catalyst", configuration)]
       end
-    when :osx then xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("macos")]
-    when :tvos then xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("tvos"), PodBuilder::XcodeBuildSettings.new("tvossimulator")]
-    when :watchos then xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("watchos"), PodBuilder::XcodeBuildSettings.new("watchossimulator")]
-    else raise "\n\nUnknown platform '#{target.platform_name}'".red end
+    when :osx then xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("macos", configuration)]
+    when :tvos then xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("tvos", configuration), PodBuilder::XcodeBuildSettings.new("tvossimulator", configuration)]
+    when :watchos then xcodebuild_settings = [PodBuilder::XcodeBuildSettings.new("watchos", configuration), PodBuilder::XcodeBuildSettings.new("watchossimulator", configuration)]
+    else raise "\n\nUnknown platform '#{target.platform_name}'\n".red end
   
     xcodebuild_settings.each do |xcodebuild_setting|
       puts "Building xcframeworks for #{xcodebuild_setting.platform_name}".yellow
-      raise "\n\n#{build_destination} xcframework archive failed!".red if !system("xcodebuild archive -project #{project_path.to_s} -scheme Pods-DummyTarget -destination '#{xcodebuild_setting.build_destination}' -archivePath '#{build_dir}/#{xcodebuild_setting.platform_name}' SKIP_INSTALL=NO > /dev/null 2>&1")
+      raise "\n\n#{xcodebuild_setting.build_destination} xcframework archive failed!\n".red if !system("xcodebuild archive -project #{project_path.to_s} -scheme Pods-DummyTarget -configuration #{xcodebuild_setting.configuration} -destination '#{xcodebuild_setting.build_destination}' -archivePath '#{build_dir}/#{xcodebuild_setting.platform_name}' SKIP_INSTALL=NO > /dev/null 2>&1")
     end
 
     built_items = Dir.glob("#{build_dir}/#{xcodebuild_settings[0].platform_name}.xcarchive/Products/Library/Frameworks/*").reject { |t| File.basename(t, ".*") == "Pods_DummyTarget" }
@@ -346,7 +348,7 @@ Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_conte
       framework_name = File.basename(built_item_paths.first, ".*")
       xcframework_path = "#{base_destination}/#{framework_name}/#{framework_name}.xcframework"
       framework_params = built_item_paths.map { |t| "-framework '#{t}'"}.join(" ")
-      raise "\n\nFailed packing xcframework!".red if !system("xcodebuild -create-xcframework #{framework_params} -output '#{xcframework_path}' > /dev/null 2>&1")      
+      raise "\n\nFailed packing xcframework!\n".red if !system("xcodebuild -create-xcframework #{framework_params} -output '#{xcframework_path}' > /dev/null 2>&1")      
 
       if enable_dsym
         xcodebuild_settings.each do |xcodebuild_setting|
@@ -359,7 +361,7 @@ Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_conte
           end  
         end
       else
-        raise "Not implemented"
+        raise "\n\nNot implemented\n".red
       end
     end
 
@@ -375,7 +377,7 @@ Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_conte
     when [:osx, false] then PodBuilder::xcodebuild(sandbox, target.cocoapods_target_label, configuration, PodBuilder::Configuration.deterministic_build, prebuilt_root_paths)
     when [:tvos, false] then PodBuilder::build_for_iosish_platform_lib(sandbox, build_dir, target, 'appletvos', 'appletvsimulator', configuration, PodBuilder::Configuration.deterministic_build, prebuilt_root_paths)
     when [:watchos, false] then PodBuilder::build_for_iosish_platform_lib(sandbox, build_dir, target, 'watchos', 'watchsimulator', configuration, PodBuilder::Configuration.deterministic_build, prebuilt_root_paths)
-    else raise "\n\nUnknown platform '#{target.platform_name}'".red end
+    else raise "\n\nUnknown platform '#{target.platform_name}'\n".red end
 
     raise Pod::Informative, 'The build directory was not found in the expected location.' unless build_dir.directory?
     
@@ -434,7 +436,7 @@ Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_conte
         FileUtils.mv(dsym_source, sandbox_root.parent)
       end
     else
-      raise "Not implemented"
+      raise "\n\nNot implemented\n".red
     end    
   end
 

data/lib/pod_builder/version.rb

@@ -1,4 +1,4 @@
 module PodBuilder
-  VERSION = "2.2.1"
+  VERSION = "3.2.0"
 end
   

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pod-builder
 version: !ruby/object:Gem::Version
-  version: 2.2.1
+  version: 3.2.0
 platform: ruby
 authors:
 - Tomas Camin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-12-10 00:00:00.000000000 Z
+date: 2021-05-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -233,7 +233,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Prebuild CocoaPods pods