cocoapods-binary-cache 0.1.7 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e068bdcb9e91a8e599e3ad1732837a1a7dbd49a4409e05b97b1cdc476bfb76a7
-  data.tar.gz: 21d3e3fa0374a8818efa927bb822ce9386e43e18ec26a1ab5be7d8b481a688d6
+  metadata.gz: 939a85addd611d4a4bcd2b9ce1eeb7e036d58a937a14bdd4f772a6c8170e4394
+  data.tar.gz: 0372d8f94f6e486259669f4db5431585a08645c1a23ad363258581fc5f679eff
 SHA512:
-  metadata.gz: f5588698b1d818071b2bfb568bcbd44bc1e02084f7caf0b4b554a50277f2064209ec527339de4c8166ab74ff358b0a173f149b5db67f33a808c8d562f8d02625
-  data.tar.gz: dd21d19ff94f2a8dcf3b4b1dd7372484993e0aba4705512fde75be14e66d72ddadc592fc88b9ea81ae3983f156e5fe251e4dbcfe8047f7904db1b0fd3cf0cbcf
+  metadata.gz: aa2bbc570dc6a1f462a85286578b8dc41a7ce189df2017177d083a70df9ac0d026c8ca8dea310eee2ae5b517b3418004045b8bcc782aa0440f755d8bab1dd0d4
+  data.tar.gz: c98e55ae66d034897db4969bf9c12b0386a69a5139bc78ada10298875c5a9487d499852bb77195ffca42f11b0d2ab97fad940348da90d21296f6f0d6952b64af

data/lib/cocoapods-binary-cache/cache/validation_result.rb

@@ -7,6 +7,10 @@ module PodPrebuild
       @hit = hit.to_set - missed_with_reasons.keys
     end
 
+    def all
+      (hit + missed).to_set
+    end
+
     def missed
       @missed_with_reasons.keys.to_set
     end

data/lib/cocoapods-binary-cache/dependencies_graph/dependencies_graph.rb

@@ -1,56 +1,50 @@
 # Copyright 2019 Grabtaxi Holdings PTE LTE (GRAB), All rights reserved.
 # Use of this source code is governed by an MIT-style license that can be found in the LICENSE file
 
-require 'rgl/adjacency'
-require 'rgl/dot'
-require_relative 'graph_visualizer'
+require "rgl/adjacency"
+require "rgl/dot"
+require_relative "graph_visualizer"
 
-# Using RGL graph because GraphViz doesn't store adjacent of a node/vertex but we need to traverse a substree from any node
+# Using RGL graph because GraphViz doesn't store adjacent of a node/vertex
+# but we need to traverse a substree from any node
 # https://github.com/monora/rgl/blob/master/lib/rgl/adjacency.rb
 
 class DependenciesGraph
   def initialize(lockfile)
     @lockfile = lockfile
-    @invert_edge = true # A normal edge is an edge (one direction) from library A to library B which is a dependency of A.
+    # A normal edge is an edge (one direction) from library A to library B which is a dependency of A.
+    @invert_edge = true
   end
 
   # Input : a list of library names.
   # Output: a set of library names which are clients (directly and indirectly) of those input libraries.
   def get_clients(libnames)
-    result = Set.new()
+    result = Set.new
     libnames.each do |lib|
       if graph.has_vertex?(lib)
         result.merge(traverse_sub_tree(graph, lib))
       else
-        puts "Warning: cannot find lib: #{lib}"
+        Pod::UI.puts "Warning: cannot find lib: #{lib}"
       end
     end
     result
   end
 
-  def write_graphic_file(output_graphic_fmt, filename='graph', highlight_nodes=Set[])
-    if !output_graphic_fmt
-      puts 'Error: Need graphic format.'
-      return
-    end
-    graph.write_to_graphic_file(output_graphic_fmt, dotfile=filename, options={}, highlight_nodes)
+  def write_graphic_file(options)
+    graph.write_to_graphic_file(options)
   end
 
   private
 
   def dependencies
-    @dependencies ||= begin
-      if @lockfile
-        @lockfile.to_hash['PODS']
-      else
-        nil
-      end
-    end
+    @dependencies ||= (@lockfile && @lockfile.to_hash["PODS"])
   end
 
   # Convert array of dictionaries -> a dictionary with format {A: [A's dependencies]}
   def pod_to_dependencies
-    dependencies.map { |d| d.is_a?(Hash) ? d : { d => [] } }.reduce({}) { |combined, individual| combined.merge!(individual) }
+    dependencies
+      .map { |d| d.is_a?(Hash) ? d : { d => [] } }
+      .reduce({}) { |combined, individual| combined.merge!(individual) }
   end
 
   def add_vertex(graph, pod)
@@ -65,14 +59,15 @@ class DependenciesGraph
 
   def graph
     @graph ||= begin
-      graph = RGL::DirectedAdjacencyGraph.new()
-
+      graph = RGL::DirectedAdjacencyGraph.new
       pod_to_dependencies.each do |pod, dependencies|
         pod_node = add_vertex(graph, pod)
         next if pod_node.nil?
+
         dependencies.each do |dependency|
           dep_node = add_vertex(graph, dependency)
           next if dep_node.nil?
+
           if @invert_edge
             graph.add_edge(dep_node, pod_node)
           else
@@ -85,11 +80,11 @@ class DependenciesGraph
   end
 
   def traverse_sub_tree(graph, vertex)
-    visited_nodes = Set.new()
+    visited_nodes = Set.new
     graph.each_adjacent(vertex) do |v|
       visited_nodes.add(v)
       visited_nodes.merge(traverse_sub_tree(graph, v))
     end
     visited_nodes
   end
-end
+end

data/lib/cocoapods-binary-cache/dependencies_graph/graph_visualizer.rb

@@ -2,73 +2,64 @@
 # Use of this source code is governed by an MIT-style license that can be found in the LICENSE file
 # https://github.com/monora/rgl/blob/0b526e16f9fb344abf387f4c5523d7917ce8f4b1/lib/rgl/dot.rb
 
-require 'rgl/rdot'
+require "rgl/rdot"
 
 module RGL
   module Graph
-    def to_dot_graph(params={}, highlight_nodes)
-      params['name'] ||= self.class.name.gsub(/:/, '_')
-      fontsize       = params['fontsize'] ? params['fontsize'] : '12'
-      graph          = (directed? ? DOT::Digraph : DOT::Graph).new(params)
-      edge_class     = directed? ? DOT::DirectedEdge : DOT::Edge
-      vertex_options = params['vertex'] || {}
-      edge_options   = params['edge'] || {}
+    def to_dot_graph(options)
+      highlight_nodes = options[:highlight_nodes] || Set.new
+      options["name"] ||= self.class.name.gsub(/:/, "_")
+      fontsize = options["fontsize"] || "12"
+      graph = (directed? ? DOT::Digraph : DOT::Graph).new(options)
+      edge_class = directed? ? DOT::DirectedEdge : DOT::Edge
+      vertex_options = options["vertex"] || {}
+      edge_options = options["edge"] || {}
 
       each_vertex do |v|
-        default_vertex_options =  {
-          'name'     => vertex_id(v),
-          'fontsize' => fontsize,
-          'label'    => vertex_label(v),
-          'style'    => 'filled',
+        default_vertex_options = {
+          "name" => vertex_id(v),
+          "fontsize" => fontsize,
+          "label" => vertex_label(v),
+          "style" => "filled"
         }
-        if highlight_nodes.include?(v)
-          default_vertex_options = default_vertex_options.merge({
-            'color' => 'red',
-            'fillcolor' => 'red'
-          })
-        else
-          default_vertex_options = default_vertex_options.merge({
-            'color' => 'blue',
-            'fillcolor' => 'blue'
-          })
-        end
-
+        default_vertex_options.merge!("color" => "red", "fillcolor" => "red") if highlight_nodes.include?(v)
         each_vertex_options = default_vertex_options.merge(vertex_options)
-        vertex_options.each{|option, val| each_vertex_options[option] = val.call(v) if val.is_a?(Proc)}
+        vertex_options.each { |option, val| each_vertex_options[option] = val.call(v) if val.is_a?(Proc) }
         graph << DOT::Node.new(each_vertex_options)
       end
 
       each_edge do |u, v|
         default_edge_options = {
-          'from'     => vertex_id(u),
-          'to'       => vertex_id(v),
-          'fontsize' => fontsize
+          "from" => vertex_id(u),
+          "to" => vertex_id(v),
+          "fontsize" => fontsize
         }
         each_edge_options = default_edge_options.merge(edge_options)
-        edge_options.each{|option, val| each_edge_options[option] = val.call(u, v) if val.is_a?(Proc)}
+        edge_options.each { |option, val| each_edge_options[option] = val.call(u, v) if val.is_a?(Proc) }
         graph << edge_class.new(each_edge_options)
       end
 
       graph
     end
 
-    def write_to_graphic_file(fmt='png', dotfile="graph", options={}, highlight_nodes)
-      src = dotfile + ".dot"
-      dot = dotfile + "." + fmt
+    def write_to_graphic_file(options)
+      output_path = Pathname.new(options[:output_path])
+      fmt = output_path.extname.delete_prefix(".")
+      dotfile = output_path.sub_ext(".dot")
 
-      File.open(src, 'w') do |f|
-        f << self.to_dot_graph(params=options, highlight_nodes=highlight_nodes).to_s << "\n"
+      File.open(dotfile, "w") do |f|
+        f << to_dot_graph(options).to_s
       end
 
-      unless system("dot -T#{fmt} #{src} -o #{dot}")
-        message = <<-HEREDOC # Use <<- to indent End of String terminator
+      unless system("dot -T#{fmt} #{dotfile} -o #{output_path}")
+        message = <<~HEREDOC
           Error executing dot. Did you install GraphViz?
           Try installing it via Homebrew: `brew install graphviz`.
           Visit https://graphviz.org/download/ for more installation instructions.
         HEREDOC
         raise message
       end
-      dot
+      output_path
     end
   end
 end

data/lib/cocoapods-binary-cache/diagnosis/diagnosis.rb

@@ -10,7 +10,15 @@ module PodPrebuild
     end
 
     def run
-      @diagnosers.each(&:run)
+      diagnosis = @diagnosers.map(&:run)
+      errors = diagnosis.select { |d| d[0] == :error }.map { |d| d[1] }
+      warnings = diagnosis.select { |d| d[0] == :error }.map { |d| d[1] }
+
+      warnings.each { |d| Pod::UI.puts "⚠️  #{d[1]}" }
+      errors.each { |d| Pod::UI.puts "🚩  #{d[1]}" }
+      return if errors.empty? || !PodPrebuild.config.strict_diagnosis?
+
+      raise "There are #{errors.count} error(s) spotted after the diagnosis"
     end
   end
 end

data/lib/cocoapods-binary-cache/diagnosis/integration.rb

@@ -15,7 +15,9 @@ module PodPrebuild
           PodPrebuild.config.prebuilt_path(path: "#{module_name}.framework")
         framework_path.exist?
       end
-      Pod::UI.puts "🚩 Unintegrated frameworks: #{unintegrated}".yellow unless unintegrated.empty?
+      return [] if unintegrated.empty?
+
+      [[:error, "Unintegrated frameworks: #{unintegrated}"]]
     end
   end
 end

data/lib/cocoapods-binary-cache/helper/podspec.rb

@@ -1,8 +1,11 @@
 module Pod
   class Specification
-    # TODO: this detect objc lib as empty source, eg. Realm
     def empty_source_files?
-      return subspecs.all?(&:empty_source_files?) unless subspecs.empty?
+
+      unless subspecs.empty?
+        # return early if there are some files in subpec(s) but process the spec itself
+        return false unless subspecs.all?(&:empty_source_files?)
+      end
 
       check = lambda do |patterns|
         patterns = [patterns] if patterns.is_a?(String)

data/lib/cocoapods-binary-cache/helper/prebuild_order.rb

@@ -0,0 +1,12 @@
+module PodPrebuild
+  module BuildOrder
+    def self.order_targets(targets)
+      # It's more efficient to build frameworks that have more dependencies first
+      # so that the build parallelism is ultilized
+      # >> --- MyFramework ----------------------------------|
+      #        >> --- ADependency ---|
+      #          >> --- AnotherADependency ---|
+      targets.sort_by { |t| -t.recursive_dependent_targets.count }
+    end
+  end
+end

data/lib/cocoapods-binary-cache/hooks/post_install.rb

@@ -12,9 +12,9 @@ module PodPrebuild
     private
 
     def diagnose
-      Pod::UI.section("Diagnosing cocoapods-binary-cache") do
+      Pod::UI.title("Diagnosing cocoapods-binary-cache") do
         PodPrebuild::Diagnosis.new(
-          cache_validation: PodPrebuild::StateStore.cache_validation,
+          cache_validation: PodPrebuild.state.cache_validation,
           standard_sandbox: @installer_context.sandbox,
           specs: @installer_context.umbrella_targets.map(&:specs).flatten
         ).run
@@ -22,7 +22,8 @@ module PodPrebuild
     end
 
     def edit_scheme_for_code_coverage
-      return unless PodPrebuild.config.dev_pods_enabled? && @installer_context.sandbox.instance_of?(Pod::PrebuildSandbox)
+      return unless PodPrebuild.config.dev_pods_enabled?
+      return unless @installer_context.sandbox.instance_of?(Pod::PrebuildSandbox)
 
       # Modify pods scheme to support code coverage
       # If we don't prebuild dev pod -> no need to care about this in Pod project

data/lib/cocoapods-binary-cache/hooks/pre_install.rb

@@ -15,17 +15,13 @@ module PodPrebuild
     def run
       return if @installer_context.sandbox.is_a?(Pod::PrebuildSandbox)
 
-      require_relative "../pod-binary/helper/feature_switches"
-
       log_section "🚀  Prebuild frameworks"
       ensure_valid_podfile
       save_installation_states
-      Pod::UI.section("Prepare environment") { prepare_environment }
       create_prebuild_sandbox
-      Pod::UI.section("Detect implicit dependencies") { detect_implicit_dependencies }
-      Pod::UI.section("Validate prebuilt cache") { validate_cache }
+      Pod::UI.title("Detect implicit dependencies") { detect_implicit_dependencies }
+      Pod::UI.title("Validate prebuilt cache") { validate_cache }
       prebuild! if PodPrebuild.config.prebuild_job?
-      Pod::UI.section("Reset environment") { reset_environment }
 
       PodPrebuild::Env.next_stage!
       log_section "🤖  Resume pod installation"
@@ -36,7 +32,6 @@ module PodPrebuild
 
     def save_installation_states
       save_pod_install_options
-      save_states_from_dsl
     end
 
     def save_pod_install_options
@@ -55,26 +50,6 @@ module PodPrebuild
       end
     end
 
-    def prepare_environment
-      Pod::Installer.force_disable_integration true # don't integrate targets
-      Pod::Config.force_disable_write_lockfile true # disbale write lock file for perbuild podfile
-      Pod::Installer.disable_install_complete_message true # disable install complete message
-    end
-
-    def reset_environment
-      Pod::Installer.force_disable_integration false
-      Pod::Config.force_disable_write_lockfile false
-      Pod::Installer.disable_install_complete_message false
-      Pod::UserInterface.warnings = [] # clean the warning in the prebuild step, it's duplicated.
-    end
-
-    def save_states_from_dsl
-      # Note: DSL is reloaded when creating an installer (Pod::Installer.new).
-      # Any mutation to DSL is highly discouraged
-      # --> Rather, perform mutation on PodPrebuild::StateStore instead
-      PodPrebuild::StateStore.excluded_pods += PodPrebuild.config.excluded_pods
-    end
-
     def create_prebuild_sandbox
       standard_sandbox = installer_context.sandbox
       @prebuild_sandbox = Pod::PrebuildSandbox.from_standard_sandbox(standard_sandbox)
@@ -88,11 +63,9 @@ module PodPrebuild
         .group_by { |spec| spec.name.split("/")[0] }
         .select { |_, specs| specs.all?(&:empty_source_files?) }
         .keys
-
-      PodPrebuild::StateStore.excluded_pods += pods_with_empty_source_files
+      PodPrebuild.config.update_detected_excluded_pods!(pods_with_empty_source_files)
+      PodPrebuild.config.update_detected_prebuilt_pod_names!(@original_installer.prebuilt_pod_names)
       Pod::UI.puts "Exclude pods with empty source files: #{pods_with_empty_source_files.to_a}"
-
-      # TODO (thuyen): Detect dependencies of a prebuilt pod and treat them as prebuilt pods as well
     end
 
     def validate_cache
@@ -104,13 +77,13 @@ module PodPrebuild
         validate_prebuilt_settings: PodPrebuild.config.validate_prebuilt_settings,
         generated_framework_path: prebuild_sandbox.generate_framework_path,
         sandbox_root: prebuild_sandbox.root,
-        ignored_pods: PodPrebuild::StateStore.excluded_pods,
-        prebuilt_pod_names: @original_installer.prebuilt_pod_names
+        ignored_pods: PodPrebuild.config.excluded_pods,
+        prebuilt_pod_names: PodPrebuild.config.prebuilt_pod_names
       ).validate
       path_to_save_cache_validation = PodPrebuild.config.save_cache_validation_to
       @cache_validation.update_to(path_to_save_cache_validation) unless path_to_save_cache_validation.nil?
       cache_validation.print_summary
-      PodPrebuild::StateStore.cache_validation = cache_validation
+      PodPrebuild.state.update(:cache_validation => cache_validation)
     end
 
     def prebuild!

data/lib/cocoapods-binary-cache/pod-binary/helper/build.rb

@@ -7,6 +7,7 @@ module Pod
       target = options[:target]
       return if target.nil?
 
+      Pod::UI.puts "Building target: #{target}...".magenta
       options[:sandbox] = Pod::Sandbox.new(Pathname(options[:sandbox])) unless options[:sandbox].is_a?(Pod::Sandbox)
       options[:build_dir] = build_dir(options[:sandbox].root)
 

data/lib/cocoapods-binary-cache/pod-binary/helper/detected_prebuilt_pods/installer.rb

@@ -3,7 +3,7 @@ module Pod
     # Returns the names of pod targets detected as prebuilt, including
     # those declared in Podfile and their dependencies
     def prebuilt_pod_names
-      prebuilt_pod_targets.map(&:name)
+      prebuilt_pod_targets.map(&:name).to_set
     end
 
     # Returns the pod targets detected as prebuilt, including

data/lib/cocoapods-binary-cache/pod-binary/helper/prebuild_sandbox.rb

@@ -5,42 +5,43 @@ module Pod
     # [String] standard_sandbox_path
     def self.from_standard_sanbox_path(path)
       prebuild_sandbox_path = Pathname.new(path).realpath + ".." + PodPrebuild.config.prebuild_sandbox_path
-      self.new(prebuild_sandbox_path)
+      new(prebuild_sandbox_path)
     end
 
     def self.from_standard_sandbox(sandbox)
-      self.from_standard_sanbox_path(sandbox.root)
+      from_standard_sanbox_path(sandbox.root)
     end
 
     def standard_sanbox_path
-      self.root.parent
+      root.parent
     end
 
     def generate_framework_path
-      self.root + "GeneratedFrameworks"
+      root + "GeneratedFrameworks"
     end
 
     # @param name [String] pass the target.name (may containing platform suffix)
     # @return [Pathname] the folder containing the framework file.
     def framework_folder_path_for_target_name(name)
-      self.generate_framework_path + name
+      generate_framework_path + name
     end
 
     def exsited_framework_target_names
-      exsited_framework_name_pairs.map { |pair| pair[0] }.uniq
+      existed_framework_name_pairs.map { |pair| pair[0] }.uniq
     end
 
     def exsited_framework_pod_names
-      exsited_framework_name_pairs.map { |pair| pair[1] }.uniq
+      existed_framework_name_pairs.map { |pair| pair[1] }.uniq
     end
 
     def existed_target_names_for_pod_name(pod_name)
-      exsited_framework_name_pairs.select { |pair| pair[1] == pod_name }.map { |pair| pair[0] }
+      existed_framework_name_pairs.select { |pair| pair[1] == pod_name }.map { |pair| pair[0] }
     end
 
     def save_pod_name_for_target(target)
       folder = framework_folder_path_for_target_name(target.name)
       return unless folder.exist?
+
       flag_file_path = folder + "#{target.pod_name}.pod_name"
       File.write(flag_file_path.to_s, "")
     end
@@ -53,16 +54,16 @@ module Pod
       end
       name = name.basename(".pod_name").to_s unless name.nil?
       name ||= Pathname.new(target_folder_path).basename.to_s # for compatibility with older version
+      name
     end
 
     # Array<[target_name, pod_name]>
-    def exsited_framework_name_pairs
+    def existed_framework_name_pairs
       return [] unless generate_framework_path.exist?
-      generate_framework_path.children().map do |framework_path|
-        if framework_path.directory? && (not framework_path.children.empty?)
+
+      generate_framework_path.children.map do |framework_path|
+        if framework_path.directory? && !framework_path.children.empty?
           [framework_path.basename.to_s, pod_name_for_target_folder(framework_path)]
-        else
-          nil
         end
       end.reject(&:nil?).uniq
     end