pod-builder 2.0.0.beta.30 → 2.0.0.beta.35

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d4cde0acf0b503aaa846162710c25456d306b0ae6e53492b717795e4540f5b9f
-  data.tar.gz: ddfc897b5dd2d280a2b5e636706d62c854d01bb34f605b813d70a48ace6fad75
+  metadata.gz: 037b4a3f456716c55d38e1ec98307ff27e87b97c908dd5e80e5895e9686f6e3f
+  data.tar.gz: 82dbc45cb9220beb68fe1b76c26ac11f6f8d45927e35fa73ecdd754609742830
 SHA512:
-  metadata.gz: 5f468d84983dc689b5dceef947da7d3ce586bd2d34be76ec439a236ba45435a6865b54d154233fdae038352ff1582cb68fcfdc520e5a101ed4842d86cde3e973
-  data.tar.gz: f45e9377fd79dff65b9c72580a29308172cc3cf3410093c2a6d2a7c9245c7000bc732b4d78473dd58ef14a3712a7cb9697bf87e7cd8331f8b413e0f550c04cc4
+  metadata.gz: d067c51f54eea74198b8d9ae7a85a25d271666ca3301836ab14920f52c5d9dc47aae4dfafea4de43953c09b5d15cbede4f7bcd0841772c55e7a6f057e61a737b
+  data.tar.gz: 504fd2e0adec760b4a58d26bec752b3dcf7755b425a0b109662e8cc4e852387b21d003c6d2abbe87e3d92d40412c9d544eb2c2bc65f41ae9e9a94a5e1eec58c3

data/README.md

@@ -12,6 +12,10 @@ Unless you're using a Ruby version manager you should generally install using `s
 
     $ sudo gem install pod-builder
 
+## Requirements
+
+Ruby 2.6.3 or newer. Cocoapods 1.9.0 or newer
+
 # Quick start
 
 You can the initialize your project to use the tool using the `init` command
@@ -21,15 +25,15 @@ You can the initialize your project to use the tool using the `init` command
 
  This will add a _PodBuilder_ folder which will contain all files needed and generated by the PodBuilder.
 
- To prebuild all dependencies run
-
-    $ pod_builder build_all
-
  To prebuild just one or more specific dependencies run
 
     $ pod_builder build Pod1 Pod2
 
-This will generate the pod frameworks which can be committed to your repo for a much faster compilation.
+ To prebuild all dependencies run
+
+    $ pod_builder build_all
+
+This will generate the pod frameworks which can be committed to your repo (using git LFS is highly suggested) for much faster compilation.
 
 Should PodBuilder not work the way you expect you can get rid of it by running
 
@@ -57,7 +61,7 @@ This acts as a sort of lockfile and reflects the current state of what is instal
 
 ## Commands
 
-Podbuilder comes with a set of commands:
+PodBuilder comes with a rich set of commands:
 
 - `init`: initializes a project to use PodBuilder
 - `deintegrate`: deintegrates PodBuilder's initialization
@@ -81,7 +85,7 @@ The following will happen:
 
 - Create a _PodBuilder_ folder in your repo's root.
 - Copy your original Podfile to _PodBuilder/Podfile_ (PodBuilder-Podfile)
-- Add an initially empty _PodBuilder.json_ configuration file
+- Add a _PodBuilder.json_ configuration file
 - Modify the original Podfile (Application-Podfile) with some minor customizations
 - Create/Update your Gemfile adding the `gem 'pod-builder'` entry
 
@@ -116,7 +120,7 @@ Will recompile all pods to the versions defined in the Restore-Podfile. You woul
 
 #### `install_sources` command
 
-When using PodBuilder you loose ability to directly access to the source code of a pod. To overcome this limitation you can use this command which downloads the pod's source code to _PodBuilder/Sources_ and with some [tricks](https://medium.com/@t.camin/debugging-prebuilt-frameworks-c9f52d42600b) restores the ability to use the debugger and step into the code of your prebuilt dependencies. This can be very helpful to catch the exact location of a crash when it occurs (showing something more useful than assembly code). It is however advisable to switch to the original pod when doing any advanced debugging during development of code that involves a pod.
+Once you prebuild a pod you can no longer debug its origianl code, to overcome this limitation you can use this command which downloads the pod's source code to _PodBuilder/Sources_ and with some [tricks](https://medium.com/@t.camin/debugging-prebuilt-frameworks-c9f52d42600b) restores the ability to use the debugger and step into the code of your prebuilt dependencies. This can be very helpful to catch the exact location of a crash when it occurs (showing something more useful than assembly code). It is however advisable to switch to the original pod when doing any advanced debugging during development of code that involves a pod.
 
 #### `generate_lldbinit` command
 
@@ -253,6 +257,8 @@ Xcode build settings to use. You can override the default values which are:
 } 
 ```
 
+If your project uses bitcode change "ENABLE_BITCODE" to "YES". Please note that all pods that have dependencies to XCTest need to be add a `"ENABLE_BITCODE": "NO"` to the `build_settings_overrides` below.
+
 #### `build_settings_overrides`
 
 Like `build_settings` but per pod. Pod name can also refer to subspec.
@@ -272,7 +278,7 @@ Like `build_settings` but per pod. Pod name can also refer to subspec.
 
 #### `build_system`
 
-Specify which build system to use to compile frameworks. Either `Legacy` (standard build system) or `Latest` (new build system). Default value: `Legacy`.
+Specify which build system to use to compile frameworks. Either `Legacy` (standard build system) or `Latest` (new build system). Default value: `Latest`.
 
 #### `library_evolution_support`
 

data/exe/pod_builder

@@ -8,7 +8,7 @@ if show_version
   exit(0)
 end
 
-if ENV['DEBUGGING']
+if ENV["DEBUGGING"]
   puts "Running in debug, injecting $LOAD_PATH"
   libdir = File.expand_path("#{File.dirname(__FILE__)}/../lib")
   $LOAD_PATH.unshift(libdir) unless $LOAD_PATH.include?(libdir)
@@ -76,9 +76,9 @@ Options:
         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("-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
@@ -282,7 +282,7 @@ Usage:
         opts.banner = "
 Usage:
         
-    $ pod_builder switch [OPTIONS] PODNAME[s]
+    $ pod_builder switch [OPTIONS] <PODNAME...>
 
       Switch integration between prebuilt/development/default pod version. Multiple space separated pods can be passed
         
@@ -297,8 +297,14 @@ 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
+        opts.on("-c", "--child-deps", "Include dependencies of the specified <PODNAME...>") do |o|
+          OPTIONS[:resolve_child_dependencies] = true
+        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("-u", "--skip-repo-update", "Skip CocoaPods repo update (only when passing --parent-deps") do |o|
+          OPTIONS[:update_repos] = false
         end
       end,
       :call => [        
@@ -373,7 +379,7 @@ end
 
 command_ret = -1
 begin
-  unless ENV['USER'] != "root"
+  unless ENV["USER"] != "root"
     raise "\n\nFor safety do not run this as root\n".red
   end
 

data/lib/pod_builder/command/build.rb

@@ -72,6 +72,12 @@ module PodBuilder
         podfiles_items = [pods_to_build_debug] + [pods_to_build_release]
 
         install_using_frameworks = Podfile::install_using_frameworks(analyzer)
+
+        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 if Configuration.react_native_project
+        else
+          prepare_defines_modules_override(all_buildable_items)
+        end
         
         install_result = InstallResult.new
         podfiles_items.reject { |x| x.empty? }.each do |podfile_items|
@@ -118,6 +124,14 @@ module PodBuilder
 
       private
 
+      def self.prepare_defines_modules_override(all_buildable_items)
+        all_buildable_items.each do |item|
+          unless item.defines_module.nil?
+            Pod::PodTarget.modules_override[item.root_name] = item.defines_module
+          end
+        end
+      end
+
       def self.check_not_building_subspecs(pods_to_build)
         pods_to_build.each do |pod_to_build|
           if pod_to_build.include?("/")
@@ -150,7 +164,7 @@ module PodBuilder
       end
 
       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)
+        if (development_pods = pods.select { |x| x.is_development_pod }) && development_pods.count > 0 && (OPTIONS[:allow_warnings].nil?  && Configuration.allow_building_development_pods == false && Configuration.react_native_project == false)
           pod_names = development_pods.map(&:name).join(", ")
           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
@@ -187,7 +201,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)
 
-        return pods_to_build
+        if OPTIONS[:resolve_parent_dependencies]
+          dependencies = []
+          buildable_items.each do |pod|
+            if !(pod.dependencies(buildable_items) & pods_to_build).empty?
+              dependencies.push(pod)
+            end
+          end
+          pods_to_build += dependencies
+        end
+
+        return pods_to_build.uniq
       end      
     end
   end

data/lib/pod_builder/command/switch.rb

@@ -24,8 +24,29 @@ module PodBuilder
           pod_names_to_switch.push(pod_name_to_switch)
         end
 
+        if OPTIONS[:resolve_parent_dependencies] == true
+          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)
+
+          pod_names_to_switch.each do |pod_name|
+            if pod = (all_buildable_items.detect { |t| t.name == pod_name } || all_buildable_items.detect { |t| t.root_name == pod_name })
+              dependencies = []
+              all_buildable_items.each do |pod|
+                if !(pod.dependency_names & pod_names_to_switch).empty?
+                  dependencies.push(pod.root_name)
+                end
+              end
+              pod_names_to_switch += dependencies
+            end
+          end
+
+          pod_names_to_switch.uniq!
+        end
+
         dep_pod_names_to_switch = []
-        if OPTIONS[:switch_all] == true
+        if OPTIONS[:resolve_child_dependencies] == true
           pod_names_to_switch.each do |pod|
             podspec_path = PodBuilder::prebuiltpath("#{pod}/#{pod}.podspec")
             unless File.exist?(podspec_path)

data/lib/pod_builder/command/update.rb

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

data/lib/pod_builder/configuration.rb

@@ -18,34 +18,37 @@ module PodBuilder
       "Google-Mobile-Ads-SDK" => {
         "module_name": "GoogleMobileAds"
       },
-      "React-jsiexecutor" => {
-        "remove_module_maps": ["glog"]
-      }, 
-      "React-Core" => {
-        "remove_module_maps": ["glog"]
+      "glog" => {
+        "pod_target_xcconfig": { "DEFINES_MODULE": "NO" }
       },
-      "React-Core/Default" => {
-        "public_headers": "React/**/*.{h}"
+      "DoubleConversion" => {
+        "pod_target_xcconfig": { "DEFINES_MODULE": "NO" }
       },
-      "React-jsi" => {
-        "remove_module_maps": ["glog"]
-      },
-      "React-cxxreact" => {
-        "remove_module_maps": ["glog"]
-      }, 
       "Folly" => {
-        "remove_module_maps": ["glog"]
+        "pod_target_xcconfig": { "DEFINES_MODULE": "NO" }
+      },
+      "Flipper-DoubleConversion" => {
+        "pod_target_xcconfig": { "DEFINES_MODULE": "NO" }
+      },
+      "Flipper-Folly" => {
+        "pod_target_xcconfig": { "DEFINES_MODULE": "NO" }
+      }
+    }.freeze
+    DEFAULT_BUILD_SETTINGS_OVERRIDES = {
+      "SBTUITestTunnelClient" => {
+        "ENABLE_BITCODE": "NO"
       }
     }.freeze
     DEFAULT_SKIP_PODS = ["GoogleMaps"]
-    DEFAULT_FORCE_PREBUILD_PODS = ["GoogleTagManager"]
-    DEFAULT_BUILD_SYSTEM = "Legacy".freeze # either Latest (New build system) or Legacy (Standard build system)
+    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_FOR_APPLE_SILICON = false
     DEFAULT_BUILD_USING_REPO_PATHS = false
     
     private_constant :DEFAULT_BUILD_SETTINGS
+    private_constant :DEFAULT_BUILD_SETTINGS_OVERRIDES
     private_constant :DEFAULT_BUILD_SYSTEM
     private_constant :DEFAULT_LIBRARY_EVOLUTION_SUPPORT
     
@@ -82,7 +85,7 @@ module PodBuilder
     
     @allow_building_development_pods = false
     @build_settings = DEFAULT_BUILD_SETTINGS
-    @build_settings_overrides = {}
+    @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

data/lib/pod_builder/install.rb

@@ -20,7 +20,12 @@ begin
       
       if overrides = PodBuilder::Configuration.spec_overrides[spec.name]
         overrides.each do |k, v|
-          spec.attributes_hash[k] = v
+          if spec.attributes_hash[k].is_a?(Hash)
+            current = spec.attributes_hash[k]
+            spec.attributes_hash[k] = current.merge(v)
+          else
+            spec.attributes_hash[k] = v
+          end
         end
       end
       
@@ -47,6 +52,24 @@ begin
       end
     end
   end
+
+  class Pod::PodTarget
+    @@modules_override = Hash.new
+
+    def self.modules_override= (x)
+      @@modules_override = x
+    end
+
+    def self.modules_override
+        return @@modules_override
+    end
+
+    alias_method :swz_defines_module?, :defines_module?
+
+    def defines_module?      
+      return @@modules_override.has_key?(name) ? @@modules_override[name] : swz_defines_module?
+    end
+  end
   
   # Starting from CocoaPods 1.10.0 and later resources are no longer copied inside the .framework
   # when building static frameworks. While this is correct when using CP normally, for redistributable
@@ -71,28 +94,6 @@ begin
       return res
     end
   end  
-  
-  class Pod::Target::BuildSettings
-    alias_method :swz_save_as, :save_as
-
-    @specs_remove_module_maps = Hash.new
-
-    class << self
-      attr_accessor :specs_remove_module_maps
-    end
-
-    def save_as(path)
-      Pod::Target::BuildSettings.specs_remove_module_maps.each do |root_name, module_maps_to_remove|
-        if target.name == root_name
-          module_maps_to_remove.each do |module_map_to_remove|
-            xcconfig.attributes["OTHER_CFLAGS"] = xcconfig.attributes["OTHER_CFLAGS"].gsub(/-fmodule-map-file=\S*#{module_map_to_remove}.modulemap.*?(\s|$)/, '')
-           end
-        end
-      end
-     
-      swz_save_as(path)
-    end
-  end
 rescue LoadError
   # CocoaPods 1.6.2 or earlier
 end
@@ -160,8 +161,6 @@ module PodBuilder
         FileUtils.touch(lock_file)
         
         prebuilt_entries = use_prebuilt_entries_for_unchanged_pods(podfile_path, podfile_items)
-
-        prepare_for_static_framework_workarounds(podfile_content, podfile_items)
         
         install
         
@@ -177,10 +176,13 @@ module PodBuilder
         return InstallResult.new(licenses, prebuilt_info)
       rescue Exception => e
         if File.directory?("#{Configuration.build_path}/Pods/Pods.xcodeproj")
-          if ENV['DEBUGGING']
+          activate_pod_scheme()
+
+          if ENV["DEBUGGING"]
             system("xed #{Configuration.build_path}/Pods")  
           elsif !OPTIONS.has_key?(:no_stdin_available)
             confirm = ask("\n\nOh no! Something went wrong during prebuild phase! Do you want to open the prebuild project to debug the error, you will need to add and run the Pods-Dummy scheme? [Y/N] ".red) { |yn| yn.limit = 1, yn.validate = /[yn]/i }
+
             if confirm.downcase == 'y'
               system("xed #{Configuration.build_path}/Pods")  
             end
@@ -241,16 +243,6 @@ module PodBuilder
       return ret
     end
     private 
-
-    def self.prepare_for_static_framework_workarounds(podfile_content, podfile_items)
-      unless podfile_content.include?("use_modular_headers!")
-        return
-      end
-
-      podfile_items.each do |podfile_item|
-        Pod::Target::BuildSettings.specs_remove_module_maps[podfile_item.root_name] = podfile_item.remove_module_maps
-      end
-    end
     
     def self.license_specifiers
       acknowledge_file = "#{Configuration.build_path}/Pods/Target Support Files/Pods-DummyTarget/Pods-DummyTarget-acknowledgements.plist"
@@ -335,7 +327,7 @@ module PodBuilder
     end
     
     def self.install
-      puts "Prebuilding items".yellow
+      puts "Preparing build".yellow
       
       CLAide::Command::PluginManager.load_plugins("cocoapods")
       
@@ -403,9 +395,11 @@ module PodBuilder
           next
         end
 
-        destination_folder = PodBuilder::prebuiltpath(root_name)
-        FileUtils.mkdir_p(destination_folder)
-        FileUtils.cp_r("#{source_path}/.", destination_folder)  
+        unless Dir.glob("#{source_path}/**/*").select { |t| File.file?(t) }.empty?
+          destination_folder = PodBuilder::prebuiltpath(root_name)
+          FileUtils.mkdir_p(destination_folder)
+          FileUtils.cp_r("#{source_path}/.", destination_folder)  
+        end
       end
       
       # Folder won't exist if no dSYM were generated (all static libs)
@@ -495,5 +489,19 @@ module PodBuilder
         return replace_path
       end
     end 
+
+    def self.activate_pod_scheme
+      if scheme_file = Dir.glob("#{Configuration.build_path}/Pods/**/xcschememanagement.plist").first
+        plist = CFPropertyList::List.new(:file => scheme_file)
+        data = CFPropertyList.native_types(plist.value)
+
+        if !data.dig("SchemeUserState", "Pods-DummyTarget.xcscheme").nil?
+          data["SchemeUserState"]["Pods-DummyTarget.xcscheme"]["isShown"] = true
+        end
+
+        plist.value = CFPropertyList.guess(data)
+        plist.save(scheme_file, CFPropertyList::List::FORMAT_BINARY)  
+      end
+    end
   end
 end

data/lib/pod_builder/podfile.rb

@@ -37,29 +37,36 @@ module PodBuilder
         item_build_settings = Configuration.build_settings_overrides[item.name] || {}
         
         # These settings need to be set as is to properly build frameworks
-        build_settings['SWIFT_COMPILATION_MODE'] = 'wholemodule'
-        build_settings['ONLY_ACTIVE_ARCH'] = 'NO'
-        build_settings['DEBUG_INFORMATION_FORMAT'] = "dwarf-with-dsym"
+        build_settings["SWIFT_COMPILATION_MODE"] = "wholemodule"
+        build_settings["ONLY_ACTIVE_ARCH"] = "NO"
+        build_settings["DEBUG_INFORMATION_FORMAT"] = "dwarf-with-dsym"
 
-        build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = platform.deployment_target.version # Fix compilation warnings on Xcode 12
+        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"
+        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"
 
         # Improve compile speed
-        build_settings['COMPILER_INDEX_STORE_ENABLE'] = 'NO'
-        build_settings['SWIFT_INDEX_STORE_ENABLE'] = 'NO'
-        build_settings['MTL_ENABLE_INDEX_STORE'] = 'NO'
+        build_settings["COMPILER_INDEX_STORE_ENABLE"] = "NO"
+        build_settings["SWIFT_INDEX_STORE_ENABLE"] = "NO"
+        build_settings["MTL_ENABLE_INDEX_STORE"] = "NO"
 
         if Configuration.build_system == "Legacy"
-          build_settings['BUILD_LIBRARY_FOR_DISTRIBUTION'] = "NO"
+          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"
+          build_settings["BUILD_LIBRARY_FOR_DISTRIBUTION"] = "YES"
         end
 
-        build_settings['SWIFT_VERSION'] = item_build_settings["SWIFT_VERSION"] || item.swift_version || project_swift_version(analyzer)
+        build_settings["SWIFT_VERSION"] = item_build_settings["SWIFT_VERSION"] || item.swift_version || project_swift_version(analyzer)
+
+        if build_settings["ENABLE_BITCODE"] == "YES"
+          build_settings["BITCODE_GENERATION_MODE"] = "bitcode"
+        end
 
         item_build_settings.each do |k, v|
           build_settings[k] = v
@@ -387,8 +394,8 @@ module PodBuilder
     def self.install_using_frameworks(analyzer)
       target_settings = analyzer.podfile.target_definition_list.map(&:uses_frameworks?).uniq
       if target_settings.count == 1
-        if target_settings.first == false && ENV['DEBUGGING'].nil?
-          raise "\n\nOnly framework packaging currently supported. Please add 'use_frameworks!' at Podfile root level (not nested in targets)".red
+        if target_settings.first == false && ENV["DEBUGGING"].nil?
+          raise "\n\nOnly framework packaging currently supported. Please add 'use_frameworks!' at root level (not nested in targets) in #{PodBuilder::basepath("Podfile")}".red
         end
         return target_settings.first
       elsif target_settings.count > 1

data/lib/pod_builder/podfile_item.rb

@@ -75,13 +75,13 @@ module PodBuilder
     #
     attr_accessor :is_external
 
-    # @return [String] The pod's build configuration
+    # @return [String] Header directory name
     #
-    attr_accessor :build_configuration
+    attr_accessor :header_dir
 
-    # @return [Array<String>] When building static frameworks we sometimes have to remove module maps from Other C flags to make compilation succeed
+    # @return [String] The pod's build configuration
     #
-    attr_accessor :remove_module_maps
+    attr_accessor :build_configuration
 
     # @return [String] The pod's vendored frameworks
     #
@@ -130,6 +130,10 @@ module PodBuilder
     # @return [Array<String>] Default subspecs
     #
     attr_accessor :default_subspecs
+
+    # @return [Bool] Defines module
+    #
+    attr_accessor :defines_module
     
     # Initialize a new instance
     #
@@ -159,6 +163,11 @@ module PodBuilder
         @commit = spec.root.source[:commit]
         @is_external = false
       end    
+
+      @defines_module = nil # nil is not specified
+      if override = spec.attributes_hash.dig("pod_target_xcconfig", "DEFINES_MODULE")
+        @defines_module = (override == "YES")
+      end
       
       @vendored_frameworks = extract_vendored_frameworks(spec, all_specs)
       @vendored_libraries = extract_vendored_libraries(spec, all_specs)
@@ -176,6 +185,8 @@ module PodBuilder
       @libraries += extract_array(spec, "library")
       @libraries += extract_array(spec, "libraries")  
 
+      @header_dir = spec.attributes_hash["header_dir"]
+
       @version = spec.root.version.version
       @available_versions = spec.respond_to?(:spec_source) ? spec.spec_source.versions(@root_name)&.map(&:to_s) : [@version]
       
@@ -212,8 +223,6 @@ module PodBuilder
       @build_configuration = spec.root.attributes_hash.dig("pod_target_xcconfig", "prebuild_configuration") || "release"
       @build_configuration.downcase!
 
-      @remove_module_maps = spec.root.attributes_hash["remove_module_maps"] || []
-
       default_license = "MIT"
       @license = spec.root.attributes_hash.fetch("license", {"type"=>"#{default_license}"})["type"] || default_license
       @summary = spec.root.attributes_hash.fetch("summary", "A summary for #{@name}")

data/lib/pod_builder/podspec.rb

@@ -39,7 +39,7 @@ module PodBuilder
         existing_vendored_frameworks = vendored_frameworks.select(&if_exists)
         existing_vendored_frameworks_basename = vendored_frameworks.map { |t| File.basename(t) }.select(&if_exists)
         vendored_frameworks = (existing_vendored_frameworks + existing_vendored_frameworks_basename).uniq
-        
+
         vendored_libraries = item.vendored_libraries
         if install_using_frameworks
           existing_vendored_libraries = vendored_libraries.map { |t| "#{item.module_name}/#{t}" }.select(&if_exists)
@@ -56,10 +56,10 @@ module PodBuilder
 
           exclude_files = static_frameworks.map { |x| x.vendored_framework_path.nil? ? nil : "#{x.vendored_framework_path}/Info.plist" }.compact.flatten.uniq
           public_headers = []
-        else
+        else          
           public_headers = Dir.glob(PodBuilder::prebuiltpath("#{item.root_name}/#{item.root_name}/Headers/**/*.h"))
           vendored_libraries +=  ["#{item.root_name}/lib#{item.root_name}.a"]
-          vendored_libraries.map! { |t| "#{item.root_name}/#{t}" }.select(&if_exists)
+          vendored_libraries = vendored_libraries.select(&if_exists)
            
           resources = ["#{item.root_name}/*.{nib,bundle,xcasset,strings,png,jpg,tif,tiff,otf,ttf,ttc,plist,json,caf,wav,p12,momd}"]
 
@@ -97,6 +97,9 @@ module PodBuilder
         if public_headers.count > 0
           podspec += "#{indentation}#{spec_var}.public_header_files = '#{item.root_name}/Headers/**/*.h'\n"
         end
+        if !item.header_dir.nil? && !install_using_frameworks
+          podspec += "#{indentation}#{spec_var}.header_dir = '#{item.header_dir}'\n"
+        end
 
         if item.xcconfig.keys.count > 0
           xcconfig = Hash.new
@@ -122,20 +125,26 @@ module PodBuilder
             podspec += "#{indentation}#{spec_var}.xcconfig = #{xcconfig.to_s}\n"
           end
         end
-        if !install_using_frameworks && spec_var == "p1"
+        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
+
+          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"
+
+          static_cfg = Hash.new
           if module_path_file = module_path_files.first
-            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"
             module_map_rel = module_path_file.gsub(PodBuilder::prebuiltpath("#{item.root_name}/#{item.root_name}/"), "")
-            static_cfg = { prebuilt_root_var => "$(PODS_ROOT)/#{rel_path}",
-                          "SWIFT_INCLUDE_PATHS" => "$(inherited) \"$(#{prebuilt_root_var})/#{item.root_name}/#{item.root_name}\"",
+            static_cfg = { "SWIFT_INCLUDE_PATHS" => "$(inherited) \"$(#{prebuilt_root_var})/#{item.root_name}/#{item.root_name}\"",
                           "OTHER_CFLAGS" => "$(inherited) -fmodule-map-file=\"$(#{prebuilt_root_var})/#{item.root_name}/#{item.root_name}/#{module_map_rel}\"",
                           "OTHER_SWIFT_FLAGS" => "$(inherited) -Xcc -fmodule-map-file=\"$(#{prebuilt_root_var})/#{item.root_name}/#{item.root_name}/#{module_map_rel}\""
-                          }
-            podspec += "#{indentation}#{spec_var}.xcconfig = #{static_cfg.to_s}\n"
-                        end
+                          }            
+          end
+          static_cfg[prebuilt_root_var] = "$(PODS_ROOT)/#{rel_path}"
+
+          podspec += "#{indentation}#{spec_var}.xcconfig = #{static_cfg.to_s}\n"
+          # This seems to be a viable workaround to https://github.com/CocoaPods/CocoaPods/issues/9559 and https://github.com/CocoaPods/CocoaPods/issues/8454
+          podspec += "#{indentation}#{spec_var}.user_target_xcconfig = { \"OTHER_LDFLAGS\" => \"$(inherited) -L\\\"$(#{prebuilt_root_var})/#{item.root_name}/#{item.root_name}\\\" -l\\\"#{item.root_name}\\\"\" }\n"
         end
   
         deps = item.dependency_names.sort

data/lib/pod_builder/rome/post_install.rb

@@ -17,7 +17,8 @@ module PodBuilder
     target_label = target.cocoapods_target_label
     
     xcodebuild(sandbox, target_label, device, deployment_target, configuration, deterministic_build, [], {})
-    excluded_archs = build_for_apple_silicon ? [] : ["arm64"]
+    excluded_archs = ["i386"] # Fixes https://github.com/Subito-it/PodBuilder/issues/17
+    excluded_archs += build_for_apple_silicon ? [] : ["arm64"]
     xcodebuild(sandbox, target_label, simulator, deployment_target, configuration, deterministic_build, excluded_archs, {})
 
     spec_names = target.specs.map { |spec| [spec.root.name, spec.root.module_name] }.uniq
@@ -80,25 +81,27 @@ module PodBuilder
 
     spec_names.each do |root_name, module_name|
       simulator_base = "#{build_dir}/#{configuration}-#{simulator}/#{root_name}"
-      simulator_lib = "#{simulator_base}/lib#{module_name}.a"
+      simulator_lib = "#{simulator_base}/lib#{root_name}.a"
 
       device_base = "#{build_dir}/#{configuration}-#{device}/#{root_name}" 
       device_lib = "#{device_base}/lib#{root_name}.a"
   
-      if File.file?(device_lib) && File.file?(simulator_lib)
-        # Starting with Xcode 12b3 the simulator binary contains an arm64 slice as well which conflict with the one in the device_lib
-        # when creating the fat library. A naive workaround is to remove the arm64 from the simulator_lib however this is wrong because 
-        # we might actually need to have 2 separated arm64 slices, one for simulator and one for device each built with different
-        # compile time directives (e.g #if targetEnvironment(simulator))
-        #
-        # For the time being we remove the arm64 slice bacause otherwise the `xcrun lipo -create -output ...` would fail.
-        if `xcrun lipo -info #{simulator_lib}`.include?("arm64")
-          `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}")
+      unless File.file?(device_lib) && File.file?(simulator_lib)
+        next
       end
 
+      # Starting with Xcode 12b3 the simulator binary contains an arm64 slice as well which conflict with the one in the device_lib
+      # when creating the fat library. A naive workaround is to remove the arm64 from the simulator_lib however this is wrong because 
+      # we might actually need to have 2 separated arm64 slices, one for simulator and one for device each built with different
+      # compile time directives (e.g #if targetEnvironment(simulator))
+      #
+      # For the time being we remove the arm64 slice bacause otherwise the `xcrun lipo -create -output ...` would fail.
+      if `xcrun lipo -info #{simulator_lib}`.include?("arm64")
+        `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}")
+
       device_headers = Dir.glob("#{device_base}/**/*.h")
       simulator_headers = Dir.glob("#{simulator_base}/**/*.h")
       device_headers.each do |device_path|
@@ -129,7 +132,7 @@ module PodBuilder
         if public_headers_path.downcase != module_public_headers_path.downcase && File.directory?(public_headers_path) && File.directory?(module_public_headers_path)
           # For pods with module_name != name we have to move the modulemap files to the root_name one
           module_public_headers_path = "#{Configuration.build_path}/Pods/Headers/Public/#{module_name}"  
-          FileUtils.cp_r("#{module_public_headers_path}/.", public_headers_path)
+          FileUtils.cp_r("#{module_public_headers_path}/.", public_headers_path, :remove_destination => true)
         end
         Dir.glob("#{public_headers_path}/**/*.*").each do |path|
           destination_folder = "#{device_base}/Headers" + path.gsub(public_headers_path, "")
@@ -258,14 +261,16 @@ module PodBuilder
     project = Xcodeproj::Project.open(project_path)
     project.targets.each do |target|
       config = target.build_configurations.find { |config| config.name.eql? configuration }
-      config.build_settings['DEBUG_INFORMATION_FORMAT'] = 'dwarf-with-dsym'
-      config.build_settings['ONLY_ACTIVE_ARCH'] = 'NO'
+      config.build_settings["DEBUG_INFORMATION_FORMAT"] = "dwarf-with-dsym"
+      config.build_settings["ONLY_ACTIVE_ARCH"] = "NO"
     end
     project.save
   end
 end
 
 Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_context, user_options|
+  puts "Building".yellow
+
   enable_dsym = user_options.fetch('dsym', true)
   configuration = user_options.fetch('configuration', 'Debug')
   uses_frameworks = user_options.fetch('uses_frameworks', true)
@@ -323,7 +328,10 @@ Pod::HooksManager.register('podbuilder-rome', :post_install) do |installer_conte
         file_accessor = Pod::Sandbox::FileAccessor.new(sandbox.pod_dir(spec.root.name), consumer)
         files += file_accessor.vendored_libraries
         files += file_accessor.vendored_frameworks
-        files += file_accessor.resources
+        begin
+          files += file_accessor.resources
+        rescue
+        end
 
         FileUtils.mkdir_p(destination)        
         files.each do |file|

data/lib/pod_builder/version.rb

@@ -1,4 +1,4 @@
 module PodBuilder
-  VERSION = "2.0.0.beta.30"
+  VERSION = "2.0.0.beta.35"
 end
   

data/pod-builder.gemspec

@@ -21,6 +21,8 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
+  spec.required_ruby_version = '>= 2.6'
+
   spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "rake", ">= 12.3.3"
   spec.add_development_dependency "ruby-debug-ide", '0.6.1'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pod-builder
 version: !ruby/object:Gem::Version
-  version: 2.0.0.beta.30
+  version: 2.0.0.beta.35
 platform: ruby
 authors:
 - Tomas Camin
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-09-29 00:00:00.000000000 Z
+date: 2020-10-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -226,7 +226,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">"