cocoapods 0.6.0.rc4 → 0.6.0.rc5

data/lib/cocoapods.rb

@@ -1,5 +1,5 @@
 module Pod
-  VERSION = '0.6.0.rc4'
+  VERSION = '0.6.0.rc5'
 
   class PlainInformative < StandardError
   end

data/lib/cocoapods/command.rb

@@ -65,7 +65,9 @@ module Pod
 
     def self.run(*argv)
       sub_command = parse(*argv)
-      Setup.new(ARGV.new).run_if_needed
+      unless ENV['SKIP_SETUP']
+        Setup.new(ARGV.new).run_if_needed
+      end
       sub_command.run
 
     rescue Interrupt

data/lib/cocoapods/command/spec.rb

@@ -11,15 +11,16 @@ module Pod
       Creates a PodSpec, in the current working dir, called `NAME.podspec'.
       If a GitHub url is passed the spec is prepopulated.
 
-    $ pod spec lint [ NAME.podspec | REPO ]
+    $ pod spec lint [ NAME.podspec | DIRECTORY ]
 
-      Validates `NAME.podspec'. In case `NAME.podspec' is omitted, it defaults
-      to `*.podspec' in the current working dir. If the name of a repo is
-      provided it validates all its specs.}
+      Validates `NAME.podspec'. If a directory is provided it performs a quick
+      validation on all the podspec files found, including subfolders. In case
+      the argument is omitted, it defaults to the current working dir.
+      }
       end
 
       def self.options
-        [ ["--quick",       "Lint skips checks that would require to donwload and build the spec"],
+        [ ["--quick",       "Lint skips checks that would require to download and build the spec"],
           ["--only-errors", "Lint validates even if warnings are present"],
           ["--no-clean",    "Lint leaves the build directory intact for inspection"] ].concat(super)
       end
@@ -74,10 +75,10 @@ module Pod
         invalid_count = lint_podspecs
         count = specs_to_lint.count
         if invalid_count == 0
-          lint_passed_message = count == 1 ? "#{podspecs_to_lint.first.basename} passed validation" : "All the #{count} specs passed validation"
+          lint_passed_message = count == 1 ? "#{podspecs_to_lint.first.basename} passed validation." : "All the specs passed validation."
           puts lint_passed_message.green << "\n\n" unless config.silent?
         else
-          raise Informative, count == 1 ? "The spec did not pass validation" : "#{invalid_count} out of #{count} specs failed validation"
+          raise Informative, count == 1 ? "The spec did not pass validation." : "#{invalid_count} out of #{count} specs failed validation."
         end
       end
 
@@ -87,12 +88,12 @@ module Pod
         invalid_count = 0
         specs_to_lint.each do |spec|
           # Show immediatly which pod is being processed.
-          print " -> #{spec}\r" unless config.silent? || is_repo?
+          print " -> #{spec}\r" unless config.silent? || @multiple_files
           $stdout.flush
 
           linter          = Linter.new(spec)
           linter.lenient  = @only_errors
-          linter.quick    = @quick || is_repo?
+          linter.quick    = @quick || @multiple_files
           linter.no_clean = @no_clean
           invalid_count  += 1 unless linter.lint
 
@@ -104,7 +105,7 @@ module Pod
 
           puts unless config.silent? || should_skip?(linter)
         end
-        puts "Analyzed #{specs_to_lint.count} specs in #{podspecs_to_lint.count} podspecs files.\n\n" if is_repo? && !config.silent?
+        puts "Analyzed #{specs_to_lint.count} specs in #{podspecs_to_lint.count} podspecs files.\n\n" if @multiple_files && !config.silent?
         invalid_count
       end
 
@@ -119,7 +120,7 @@ module Pod
       end
 
       def should_skip?(linter)
-        is_repo? && linter.errors.empty? && linter.warnings.empty? && linter.notes.empty?
+        @multiple_files && linter.errors.empty? && linter.warnings.empty? && linter.notes.empty?
       end
 
       def print_messages(spec, type, messages)
@@ -129,14 +130,14 @@ module Pod
 
       def podspecs_to_lint
         @podspecs_to_lint ||= begin
-          if (is_repo?)
-            files = (config.repos_dir + @repo_or_podspec).glob('**/*.podspec')
-          elsif @repo_or_podspec
-            files = [Pathname.new(@repo_or_podspec)]
-            raise Informative, "Unable to find a spec named #{@repo_or_podspec}" unless files[0].exist? && @repo_or_podspec.include?('.podspec')
+          path = Pathname.new(@repo_or_podspec || '.')
+          if path.directory?
+            files = path.glob('**/*.podspec')
+            raise Informative, "No specs found in the current directory." if files.empty?
+            @multiple_files = true
           else
-            files = Pathname.pwd.glob('*.podspec')
-            raise Informative, "No specs found in the current directory" if files.empty?
+            files = [path]
+            raise Informative, "Unable to find a spec named `#{@repo_or_podspec}'." unless files[0].exist? && @repo_or_podspec.include?('.podspec')
           end
           files
         end
@@ -152,10 +153,6 @@ module Pod
         end
       end
 
-      def is_repo?
-        @is_repo ||= @repo_or_podspec && (config.repos_dir + @repo_or_podspec).exist? && !@repo_or_podspec.include?('/')
-      end
-
       # Linter class
       #
       class Linter
@@ -311,7 +308,7 @@ module Pod
             # Some values are multiplaform
             patterns = patterns.is_a?(Hash) ? patterns.values.flatten(1) : patterns
             patterns.each do |pattern|
-              # Skip Filelist that would otherwise be resolved from the working directory resulting
+              # Skip FileList that would otherwise be resolved from the working directory resulting
               # in a potentially very expensi operation
               next if pattern.is_a?(FileList)
               invalid = pattern.is_a?(Array) ? pattern.any? { |path| path.start_with?('/') } : pattern.start_with?('/')
@@ -338,7 +335,7 @@ module Pod
           messages << "The description should end with a dot"               if @spec.description !~ /.*\./ && @spec.description != @spec.summary
           messages << "Git sources should specify either a tag or a commit" if source[:git] && !source[:commit] && !source[:tag]
           messages << "Github repositories should end in `.git'"            if github_source? && source[:git] !~ /.*\.git/
-          # messages << "Github repositories should use `https' link"         if github_source? && source[:git] !~ /https:\/\/github.com/
+          messages << "Github repositories should use `https' link"         if github_source? && source[:git] !~ /https:\/\/github.com/
           messages << "Comments must be deleted"                            if text =~ /^\w*#\n\w*#/ # allow a single line comment as it is generally used in subspecs
           messages
         end

data/lib/cocoapods/generator/documentation.rb

@@ -71,7 +71,9 @@ module Pod
       end
 
       def already_installed?
-        Pathname.new(File.expand_path("~/Library/Developer/Shared/Documentation/DocSets/org.cocoapods.#{name.gsub(/ /,'-')}.docset")).exist?
+        index = spec_appledoc_options.index('--company-id')
+        company_id = index ? spec_appledoc_options[index + 1] : docs_id
+        Pathname.new(File.expand_path("~/Library/Developer/Shared/Documentation/DocSets/#{company_id}.#{name.gsub(/ /,'-')}.docset")).exist?
       end
 
       def generate(install = false)

data/lib/cocoapods/local_pod.rb

@@ -450,10 +450,7 @@ module Pod
         if pattern.directory? && options[:glob]
           pattern += options[:glob]
         end
-
-        pattern.glob.map do |file|
-          file
-        end
+        pattern.glob
       end.flatten
     end
   end

data/lib/cocoapods/platform.rb

@@ -33,12 +33,6 @@ module Pod
     #       Platform.new(:ios)
     #       Platform.new(:ios, '4.3')
     #
-    # @overload initialize(name, opts)
-    #   @deprecated Remove after adding a warning to {Podfile} class.
-    #   @param [Symbol] name The name of platform.
-    #   @param [Hash] opts The options to create a platform with.
-    #   @option opts [String, Version] :deployment_target The deployment target.
-    #
     # @overload initialize(platform)
     #   @param [Platform] platform Another {Platform}.
     #
@@ -51,23 +45,9 @@ module Pod
       if input.is_a? Platform
         @symbolic_name = input.name
         @deployment_target = input.deployment_target
-        @declared_deployment_target = input.declared_deployment_target
       else
         @symbolic_name = input
-
         target = target[:deployment_target] if target.is_a?(Hash)
-        @declared_deployment_target = target
-
-        unless target
-          case @symbolic_name
-          when :ios
-            target = '4.3'
-          when :osx
-            target = '10.6'
-          else
-            target = ''
-          end
-        end
         @deployment_target = Version.create(target)
       end
     end
@@ -82,10 +62,6 @@ module Pod
     #
     attr_reader :deployment_target
 
-    # @return [Version] The deployment target declared on initialization.
-    #
-    attr_reader :declared_deployment_target
-
     # @param [Platform, Symbol] other The other platform to check.
     #
     # @note If a symbol is passed the comparison does not take into account
@@ -109,7 +85,11 @@ module Pod
     #
     def supports?(other)
       other = Platform.new(other)
+      if other.deployment_target && deployment_target
       (other.name == name) && (other.deployment_target <= deployment_target)
+      else
+        other.name == name
+      end
     end
 
     # @return [String] A string representation including the deployment target.
@@ -121,7 +101,7 @@ module Pod
       when :osx
         s = 'OS X'
       end
-      s << " #{declared_deployment_target}" if declared_deployment_target
+      s << " #{deployment_target}" if deployment_target
       s
     end
 
@@ -134,7 +114,7 @@ module Pod
     # @return Whether the platform requires legacy architectures for iOS.
     #
     def requires_legacy_ios_archs?
-      (name == :ios) && (deployment_target < Version.new("4.3"))
+      (name == :ios) && deployment_target && (deployment_target < Version.new("4.3"))
     end
   end
 end

data/lib/cocoapods/podfile.rb

@@ -173,19 +173,35 @@ module Pod
     end
 
     # Specifies the platform for which a static library should be build.
-    #
     # This can be either `:osx` for Mac OS X applications, or `:ios` for iOS
     # applications.
     #
-    # For iOS applications, you can set the deployment target by passing a :deployment_target
-    # option, e.g:
+    #   @param [Symbol] name The name of platform.
+    #   @param [String, Version] target The optional deployment.
+    #     If not provided a default value according to the platform name will
+    #     be assigned.
+    #
+    #   @example
     #
-    #   platform :ios, :deployment_target => "4.0"
+    #       platform :ios, "4.0"
+    #       platform :ios
     #
-    # If the deployment target requires it (< 4.3), armv6 will be added to ARCHS.
+    # @note If the deployment target requires it (< 4.3), armv6 will be added
+    #   to ARCHS.
     #
-    def platform(platform, options={})
-      @target_definition.platform = Platform.new(platform, options)
+    def platform(name, target = nil)
+      # Support for deprecated options parameter
+      target = target[:deployment_target] if target.is_a?(Hash)
+
+      unless target
+        case name
+        when :ios
+          target = '4.3'
+        when :osx
+          target = '10.6'
+        end
+      end
+      @target_definition.platform = Platform.new(name, target)
     end
 
     # Specifies the Xcode workspace that should contain all the projects.

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cocoapods
 version: !ruby/object:Gem::Version
-  version: 0.6.0.rc4
+  version: 0.6.0.rc5
   prerelease: 6
 platform: ruby
 authors:
@@ -232,7 +232,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3019462591125545831
+      hash: -1664248091750403569
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: