cocoapods-trunk 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 05d7b8d9b45b2789d9a5ba64682e53f65b6316d8
-  data.tar.gz: d5c2c20b498a1c9f6ba68486028c7da89b425446
+  metadata.gz: 94cb63750c2a1c96259b49e0f5a551cdb70cf456
+  data.tar.gz: 52a4b68041e9c1fe535f39e24390b6244a3fbbdb
 SHA512:
-  metadata.gz: d01e7c395ee6338c8cb00c1bf2068e680ba01bcd075103aaf0d84f5bdf0945dca2695c06b1cb46722b005451b63e869a335632dc43d0da2ac4fc4c737984ed07
-  data.tar.gz: ed35d3e3d694edf9542089db96e389b9c47b621a9c776c2e9f5c9f3df21326454bbc6b44a02b5666167a49df8840e02e66cec793ffc47ebf37ac33ee4e6d3cf5
+  metadata.gz: 3a71c8f8aae5896c00ecd59fb8693929ae5862e3e9c0d4c2d565a17c75a662418e8fba83e1355e66d67c7ab0a56740e0add1c11a44fbcec576b9c9eed3bc0e46
+  data.tar.gz: ee6ba3be0dcc8972b15ad510e7364df215d72372956a09683a902b491d6428e017d2b3db7d436db31e6490bd8e276cfc4f2cdddc40c7b3fe2f01fb88c957a634

data/.rubocop_cocoapods.yml

@@ -42,6 +42,9 @@ Encoding:
 TrailingComma:
   EnforcedStyleForMultiline: comma
 
+Style/MultilineOperationIndentation:
+  EnforcedStyle: indented
+
 # Clashes with CLAide Command#validate!
 GuardClause:
   Enabled: false
@@ -50,6 +53,25 @@ GuardClause:
 Next:
   Enabled: false
 
+# Arbitrary max lengths for classes simply do not work and enabling this will
+# lead to a never ending stream of annoyance and changes.
+Metrics/ClassLength:
+  Enabled: false
+
+# No enforced convention here.
+Metrics/BlockNesting:
+  Enabled: false
+
+# It will be obvious which code is complex, Rubocop should only lint simple
+# rules for us.
+Metrics/AbcSize:
+  Enabled: false
+
+# It will be obvious which code is complex, Rubocop should only lint simple
+# rules for us.
+Metrics/CyclomaticComplexity:
+  Enabled: false
+
 #- CocoaPods support for Ruby 1.8.7 ------------------------------------------#
 
 HashSyntax:

data/CHANGELOG.md

@@ -1,3 +1,19 @@
+## 0.4.1
+
+##### Enhancements
+
+* Improved code readability and structure by splitting subcommands
+  into individual files.  
+  [Olivier Halligon](https://github.com/alisoftware)
+  [#21](https://github.com/CocoaPods/CocoaPods/issues/21)
+
+##### Bug Fixes
+
+* Updates for changes in CocoaPods regarding `--allow-warnings`.  
+  [Kyle Fuller](https://github.com/kylef)
+  [Cocoapods#2831](https://github.com/CocoaPods/CocoaPods/pull/2831)
+
+
 ## 0.4.0
 
 ##### Bug Fixes

data/Gemfile

@@ -17,6 +17,6 @@ group :development do
   gem 'prettybacon'
 
   gem 'codeclimate-test-reporter', :require => nil
-  gem 'rubocop', '~> 0.26.0'
+  gem 'rubocop'
 end
 

data/Gemfile.lock

@@ -1,20 +1,20 @@
 GIT
   remote: https://github.com/CocoaPods/CLAide.git
-  revision: 00986e2b185c6eca9d30a7d50073f2a819a0f61a
+  revision: b1eccc296b68b1a432e42ecf0c4b799d8551c75c
   branch: master
   specs:
     claide (0.7.0)
 
 GIT
   remote: https://github.com/CocoaPods/CocoaPods.git
-  revision: e7bb85be814527b65a3785ed93d869f75df3e932
+  revision: 1703a3464674baecf54bd7e766f4b37ed8fc43f7
   branch: master
   specs:
-    cocoapods (0.35.0.rc1)
+    cocoapods (0.35.0.rc2)
       activesupport (>= 3.2.15)
       claide (~> 0.7.0)
-      cocoapods-core (= 0.35.0.rc1)
-      cocoapods-downloader (~> 0.7.2)
+      cocoapods-core (= 0.35.0.rc2)
+      cocoapods-downloader (~> 0.8.0)
       cocoapods-plugins (~> 0.3.1)
       cocoapods-trunk (~> 0.4.0)
       cocoapods-try (~> 0.4.2)
@@ -23,14 +23,14 @@ GIT
       molinillo (~> 0.1.1)
       nap (~> 0.8)
       open4 (~> 1.3)
-      xcodeproj (~> 0.20.1)
+      xcodeproj (~> 0.20.2)
 
 GIT
   remote: https://github.com/CocoaPods/Core.git
-  revision: 4c7709ceb26c1d555a765d8a8ec8198bcb94fc0c
+  revision: 60a486e8141323390dc5ff7f6676cf893a251a48
   branch: master
   specs:
-    cocoapods-core (0.35.0.rc1)
+    cocoapods-core (0.35.0.rc2)
       activesupport (>= 3.2.15)
       fuzzy_match (~> 2.0.4)
       nap (~> 0.8.0)
@@ -38,14 +38,14 @@ GIT
 PATH
   remote: .
   specs:
-    cocoapods-trunk (0.4.0)
+    cocoapods-trunk (0.4.1)
       nap (>= 0.8)
       netrc (= 0.7.8)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (4.1.7)
+    activesupport (4.1.8)
       i18n (~> 0.6, >= 0.6.9)
       json (~> 1.7, >= 1.7.7)
       minitest (~> 5.1)
@@ -55,7 +55,7 @@ GEM
     astrolabe (1.3.0)
       parser (>= 2.2.0.pre.3, < 3.0)
     bacon (1.2.0)
-    cocoapods-downloader (0.7.2)
+    cocoapods-downloader (0.8.0)
     cocoapods-plugins (0.3.2)
       nap
     cocoapods-try (0.4.2)
@@ -76,7 +76,7 @@ GEM
       rb-inotify (>= 0.9)
       rb-kqueue (>= 0.2)
     metaclass (0.0.4)
-    minitest (5.4.2)
+    minitest (5.4.3)
     mocha (1.1.0)
       metaclass (~> 0.0.1)
     mocha-on-bacon (0.2.2)
@@ -100,9 +100,9 @@ GEM
       ffi (>= 0.5.0)
     rb-kqueue (0.2.3)
       ffi (>= 0.5.0)
-    rubocop (0.26.1)
+    rubocop (0.27.1)
       astrolabe (~> 1.3)
-      parser (>= 2.2.0.pre.4, < 3.0)
+      parser (>= 2.2.0.pre.7, < 3.0)
       powerpack (~> 0.0.6)
       rainbow (>= 1.99.1, < 3.0)
       ruby-progressbar (~> 1.4)
@@ -116,7 +116,7 @@ GEM
     thread_safe (0.3.4)
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
-    xcodeproj (0.20.1)
+    xcodeproj (0.20.2)
       activesupport (>= 3)
       colored (~> 1.2)
 
@@ -137,4 +137,4 @@ DEPENDENCIES
   mocha-on-bacon
   prettybacon
   rake
-  rubocop (~> 0.26.0)
+  rubocop

data/cocoapods-trunk.gemspec

@@ -1,11 +1,11 @@
 # coding: utf-8
 lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'trunk/version'
+require 'cocoapods_trunk'
 
 Gem::Specification.new do |spec|
   spec.name          = "cocoapods-trunk"
-  spec.version       = Cocoapods::Trunk::VERSION
+  spec.version       = CocoaPodsTrunk::VERSION
   spec.authors       = ["Eloy Durán"]
   spec.email         = ["eloy.de.enige@gmail.com"]
   spec.summary       = "Interact with trunk.cocoapods.org"

data/lib/cocoapods_trunk.rb

@@ -0,0 +1,3 @@
+module CocoaPodsTrunk
+  VERSION = '0.4.1'
+end

data/lib/pod/command/trunk.rb

@@ -13,334 +13,10 @@ module Pod
       SCHEME_AND_HOST = ENV['TRUNK_SCHEME_AND_HOST'] || 'https://trunk.cocoapods.org'
       BASE_URL = "#{SCHEME_AND_HOST}/api/v1"
 
-      class Register < Trunk
-        self.summary = 'Manage sessions'
-        self.description = <<-DESC
-          Register a new account, or create a new session.
-
-          If this is your first registration, both an `EMAIL` address and your
-          `NAME` are required. If you've already registered with trunk, you may
-          omit the `NAME` (unless you would like to change it).
-
-          It is recommended that you provide a description of the session, so
-          that it will be easier to identify later on. For instance, when you
-          would like to clean-up your sessions. A common example is to specify
-          the location where the machine, that you are using the session for, is
-          physically located.
-
-          Examples:
-
-              $ pod trunk register eloy@example.com 'Eloy Durán' --description='Personal Laptop'
-              $ pod trunk register eloy@example.com --description='Work Laptop'
-              $ pod trunk register eloy@example.com
-        DESC
-
-        self.arguments = [
-          CLAide::Argument.new('EMAIL', true),
-          CLAide::Argument.new('NAME',  false),
-        ]
-
-        def self.options
-          [
-            ['--description=DESCRIPTION', 'An arbitrary description to ' \
-                                          'easily identify your session ' \
-                                          'later on.'],
-          ].concat(super)
-        end
-
-        def initialize(argv)
-          @session_description = argv.option('description')
-          @email, @name = argv.shift_argument, argv.shift_argument
-          super
-        end
-
-        def validate!
-          super
-          unless @email
-            help! 'Specify at least your email address.'
-          end
-        end
-
-        def run
-          body = {
-            'email' => @email,
-            'name' => @name,
-            'description' => @session_description,
-          }.to_json
-          json = json(request_path(:post, 'sessions', body, default_headers))
-          save_token(json['token'])
-          # TODO UI.notice inserts an empty line :/
-          puts '[!] Please verify the session by clicking the link in the ' \
-               "verification email that has been sent to #{@email}".yellow
-        rescue REST::Error => e
-          raise Informative, 'There was an error registering with trunk: ' \
-                             "#{e.message}"
-        end
-
-        def save_token(token)
-          netrc['trunk.cocoapods.org'] = @email, token
-          netrc.save
-        end
-      end
-
-      class Me < Trunk
-        self.summary = 'Display information about your sessions'
-        self.description = <<-DESC
-          Includes information about your registration, followed by all your
-          sessions.
-
-          These are your current session, other valid sessions, unverified
-          sessions, and expired sessions.
-        DESC
-
-        def validate!
-          super
-          unless token
-            help! 'You need to register a session first.'
-          end
-        end
-
-        def run
-          me = json(request_path(:get, 'sessions', auth_headers))
-          owner = json(request_path(:get, "owners/#{me['email']}"))
-          UI.labeled 'Name', owner['name']
-          UI.labeled 'Email', owner['email']
-          UI.labeled 'Since', formatted_time(owner['created_at'])
-
-          pods = owner['pods'] || []
-          pods = pods.map { |pod| pod['name'] }
-          pods = 'None' unless pods.any?
-          UI.labeled 'Pods', pods
-
-          sessions = me['sessions'].map do |session|
-            hash = {
-              :created_at => formatted_time(session['created_at']),
-              :valid_until => formatted_time(session['valid_until']),
-              :created_from_ip => session['created_from_ip'],
-              :description => session['description'],
-            }
-            if Time.parse(session['valid_until']) <= Time.now.utc
-              hash[:color] = :red
-            elsif session['verified']
-              hash[:color] = session['current'] ? :cyan : :green
-            else
-              hash[:color] = :yellow
-              hash[:valid_until] = 'Unverified'
-            end
-            hash
-          end
-
-          columns = [:created_at, :valid_until, :created_from_ip, :description].map do |key|
-            find_max_size(sessions, key)
-          end
-
-          sessions = sessions.map do |session|
-            created_at      = session[:created_at].ljust(columns[0])
-            valid_until     = session[:valid_until].rjust(columns[1])
-            created_from_ip = session[:created_from_ip].ljust(columns[2])
-            description     = session[:description]
-            msg = "#{created_at} - #{valid_until}. IP: #{created_from_ip}"
-            msg << " Description: #{description}" if description
-            msg.send(session[:color])
-          end
-
-          UI.labeled 'Sessions', sessions
-
-        rescue REST::Error => e
-          raise Informative, 'There was an error fetching your info ' \
-                             "from trunk: #{e.message}"
-        end
-
-        private
-
-        def find_max_size(sessions, key)
-          sessions.map { |s| (s[key] || '').size }.max
-        end
-
-        class CleanSessions < Me
-          self.summary = 'Remove sessions'
-          self.description = <<-DESC
-            By default this will clean-up your sessions by removing expired and
-            unverified sessions.
-
-            To remove all your sessions, except for the one you are currently
-            using, specify the `--all` flag.
-          DESC
-
-          def self.options
-            [
-              ['--all', 'Removes all your sessions, except for the current one'],
-            ].concat(super)
-          end
-
-          def initialize(argv)
-            @remove_all = argv.flag?('all', false)
-            super
-          end
-
-          def validate!
-            super
-            unless token
-              help! 'You need to register a session first.'
-            end
-          end
-
-          def run
-            path = @remove_all ? 'sessions/all' : 'sessions'
-            request_path(:delete, path, auth_headers)
-          rescue REST::Error => e
-            raise Informative, 'There was an error cleaning up your ' \
-                               "sessions from trunk: #{e.message}"
-          end
-        end
-      end
-
-      class AddOwner < Trunk
-        self.summary = 'Add an owner to a pod'
-        self.description = <<-DESC
-          Adds the registered user with specified `OWNER-EMAIL` as an owner
-          of the given `POD`.
-          An ‘owner’ is a registered user whom is allowed to make changes to a
-          pod, such as pushing new versions and adding other ‘owners’.
-        DESC
-
-        self.arguments = [
-          CLAide::Argument.new('POD', true),
-          CLAide::Argument.new('OWNER-EMAIL', true),
-        ]
-
-        def initialize(argv)
-          @pod, @email = argv.shift_argument, argv.shift_argument
-          super
-        end
-
-        def validate!
-          super
-          unless token
-            help! 'You need to register a session first.'
-          end
-          unless @pod && @email
-            help! 'Specify the pod name and the new owner’s email address.'
-          end
-        end
-
-        def run
-          body = { 'email' => @email }.to_json
-          json = json(request_path(:patch, "pods/#{@pod}/owners", body, auth_headers))
-          UI.labeled 'Owners', json.map { |o| "#{o['name']} <#{o['email']}>" }
-        rescue REST::Error => e
-          raise Informative, "There was an error adding #{@email} to " \
-                             "#{@pod} on trunk: #{e.message}"
-        end
-      end
-
-      class Push < Trunk
-        self.summary = 'Publish a podspec'
-        self.description = <<-DESC
-          Publish the podspec at `PATH` to make it available to all users of
-          the ‘master’ spec-repo. If `PATH` is not provided, defaults to the
-          current directory.
-
-          Before pushing the podspec to cocoapods.org, this will perform a local
-          lint of the podspec, including a build of the library. However, it
-          remains *your* responsibility to ensure that the published podspec
-          will actually work for your users. Thus it is recommended that you
-          *first* try to use the podspec to integrate the library into your demo
-          and/or real application.
-
-          If this is the first time you publish a spec for this pod, you will
-          automatically be registered as the ‘owner’ of this pod. (Note that
-          ‘owner’ in this case implies a person that is allowed to publish new
-          versions and add other ‘owners’, not necessarily the library author.)
-        DESC
-
-        self.arguments = [
-          CLAide::Argument.new('PATH', false),
-        ]
-
-        def self.options
-          [
-            ['--allow-warnings', 'Allows push even if there are lint warnings'],
-          ].concat(super)
-        end
-
-        def initialize(argv)
-          @allow_warnings = argv.flag?('allow-warnings')
-          @path = argv.shift_argument || '.'
-          find_podspec_file if File.directory?(@path)
-          super
-        end
-
-        def validate!
-          super
-          unless token
-            help! 'You need to register a session first.'
-          end
-          unless @path
-            help! 'Please specify the path to the podspec file.'
-          end
-          unless File.exist?(@path) && !File.directory?(@path)
-            help! "The specified path `#{@path}` does not point to " \
-              'an existing podspec file.'
-          end
-        end
-
-        def run
-          validate_podspec
-          response = request_path(:post, 'pods', spec.to_json, auth_headers)
-          url = response.headers['location'].first
-          json = json(request_url(:get, url, default_headers))
-
-          # Using UI.labeled here is dangerous, as it wraps the URL and indents
-          # it, which breaks the URL when you try to copy-paste it.
-          $stdout.puts "  - Data URL: #{json['data_url']}"
-
-          messages = json['messages'].map do |entry|
-            at, message = entry.to_a.flatten
-            "#{formatted_time(at)}: #{message}"
-          end
-          UI.labeled 'Log messages', messages
-        rescue REST::Error => e
-          raise Informative, 'There was an error pushing a new version ' \
-                             "to trunk: #{e.message}"
-        end
-
-        private
-
-        def find_podspec_file
-          podspecs = Dir[Pathname(@path) + '*.podspec{.json,}']
-          case podspecs.count
-          when 0
-            UI.notice "No podspec found in directory `#{@path}`"
-          when 1
-            UI.notice "Found podspec `#{podspecs[0]}`"
-          else
-            UI.notice "Multiple podspec files in directory `#{@path}`. " \
-              'You need to explicitly specify which one to use.'
-          end
-          @path = (podspecs.count == 1) ? podspecs[0] : nil
-        end
-
-        def spec
-          @spec ||= Pod::Specification.from_file(@path)
-        rescue Informative # TODO: this should be a more specific error
-          raise Informative, 'Unable to interpret the specified path as a ' \
-                             'podspec.'
-        end
-
-        # Performs a full lint against the podspecs.
-        #
-        # TODO: Currently copied verbatim from `pod push`.
-        def validate_podspec
-          UI.puts 'Validating podspec'.yellow
-          validator = Validator.new(spec, %w(https://github.com/CocoaPods/Specs.git))
-          validator.only_errors = @allow_warnings
-          validator.validate
-          unless validator.validated?
-            raise Informative, 'The podspec does not validate.'
-          end
-        end
-      end
+      require 'pod/command/trunk/register'
+      require 'pod/command/trunk/me'
+      require 'pod/command/trunk/add_owner'
+      require 'pod/command/trunk/push'
 
       private
 

data/lib/pod/command/trunk/add_owner.rb

@@ -0,0 +1,44 @@
+module Pod
+  class Command
+    class Trunk
+      class AddOwner < Trunk
+        self.summary = 'Add an owner to a pod'
+        self.description = <<-DESC
+                Adds the registered user with specified `OWNER-EMAIL` as an owner
+                of the given `POD`.
+                An ‘owner’ is a registered user whom is allowed to make changes to a
+                pod, such as pushing new versions and adding other ‘owners’.
+        DESC
+
+        self.arguments = [
+          CLAide::Argument.new('POD', true),
+          CLAide::Argument.new('OWNER-EMAIL', true),
+        ]
+
+        def initialize(argv)
+          @pod, @email = argv.shift_argument, argv.shift_argument
+          super
+        end
+
+        def validate!
+          super
+          unless token
+            help! 'You need to register a session first.'
+          end
+          unless @pod && @email
+            help! 'Specify the pod name and the new owner’s email address.'
+          end
+        end
+
+        def run
+          body = { 'email' => @email }.to_json
+          json = json(request_path(:patch, "pods/#{@pod}/owners", body, auth_headers))
+          UI.labeled 'Owners', json.map { |o| "#{o['name']} <#{o['email']}>" }
+        rescue REST::Error => e
+          raise Informative, "There was an error adding #{@email} to " \
+                                   "#{@pod} on trunk: #{e.message}"
+        end
+      end
+    end
+  end
+end

data/lib/pod/command/trunk/me.rb

@@ -0,0 +1,117 @@
+module Pod
+  class Command
+    class Trunk
+      class Me < Trunk
+        self.summary = 'Display information about your sessions'
+        self.description = <<-DESC
+                Includes information about your registration, followed by all your
+                sessions.
+
+                These are your current session, other valid sessions, unverified
+                sessions, and expired sessions.
+        DESC
+
+        def validate!
+          super
+          unless token
+            help! 'You need to register a session first.'
+          end
+        end
+
+        def run
+          me = json(request_path(:get, 'sessions', auth_headers))
+          owner = json(request_path(:get, "owners/#{me['email']}"))
+          UI.labeled 'Name', owner['name']
+          UI.labeled 'Email', owner['email']
+          UI.labeled 'Since', formatted_time(owner['created_at'])
+
+          pods = owner['pods'] || []
+          pods = pods.map { |pod| pod['name'] }
+          pods = 'None' unless pods.any?
+          UI.labeled 'Pods', pods
+
+          sessions = me['sessions'].map do |session|
+            hash = {
+              :created_at => formatted_time(session['created_at']),
+              :valid_until => formatted_time(session['valid_until']),
+              :created_from_ip => session['created_from_ip'],
+              :description => session['description'],
+            }
+            if Time.parse(session['valid_until']) <= Time.now.utc
+              hash[:color] = :red
+            elsif session['verified']
+              hash[:color] = session['current'] ? :cyan : :green
+            else
+              hash[:color] = :yellow
+              hash[:valid_until] = 'Unverified'
+            end
+            hash
+          end
+
+          columns = [:created_at, :valid_until, :created_from_ip, :description].map do |key|
+            find_max_size(sessions, key)
+          end
+
+          sessions = sessions.map do |session|
+            created_at      = session[:created_at].ljust(columns[0])
+            valid_until     = session[:valid_until].rjust(columns[1])
+            created_from_ip = session[:created_from_ip].ljust(columns[2])
+            description     = session[:description]
+            msg = "#{created_at} - #{valid_until}. IP: #{created_from_ip}"
+            msg << " Description: #{description}" if description
+            msg.send(session[:color])
+          end
+
+          UI.labeled 'Sessions', sessions
+
+        rescue REST::Error => e
+          raise Informative, 'There was an error fetching your info ' \
+                                   "from trunk: #{e.message}"
+        end
+
+        private
+
+        def find_max_size(sessions, key)
+          sessions.map { |s| (s[key] || '').size }.max
+        end
+
+        class CleanSessions < Me
+          self.summary = 'Remove sessions'
+          self.description = <<-DESC
+                  By default this will clean-up your sessions by removing expired and
+                  unverified sessions.
+
+                  To remove all your sessions, except for the one you are currently
+                  using, specify the `--all` flag.
+          DESC
+
+          def self.options
+            [
+              ['--all', 'Removes all your sessions, except for the current one'],
+            ].concat(super)
+          end
+
+          def initialize(argv)
+            @remove_all = argv.flag?('all', false)
+            super
+          end
+
+          def validate!
+            super
+            unless token
+              help! 'You need to register a session first.'
+            end
+          end
+
+          def run
+            path = @remove_all ? 'sessions/all' : 'sessions'
+            request_path(:delete, path, auth_headers)
+          rescue REST::Error => e
+            raise Informative, 'There was an error cleaning up your ' \
+                                     "sessions from trunk: #{e.message}"
+          end
+        end
+      end
+    end
+  end
+end

data/lib/pod/command/trunk/push.rb

@@ -0,0 +1,114 @@
+module Pod
+  class Command
+    class Trunk
+      class Push < Trunk
+        self.summary = 'Publish a podspec'
+        self.description = <<-DESC
+                Publish the podspec at `PATH` to make it available to all users of
+                the ‘master’ spec-repo. If `PATH` is not provided, defaults to the
+                current directory.
+
+                Before pushing the podspec to cocoapods.org, this will perform a local
+                lint of the podspec, including a build of the library. However, it
+                remains *your* responsibility to ensure that the published podspec
+                will actually work for your users. Thus it is recommended that you
+                *first* try to use the podspec to integrate the library into your demo
+                and/or real application.
+
+                If this is the first time you publish a spec for this pod, you will
+                automatically be registered as the ‘owner’ of this pod. (Note that
+                ‘owner’ in this case implies a person that is allowed to publish new
+                versions and add other ‘owners’, not necessarily the library author.)
+        DESC
+
+        self.arguments = [
+          CLAide::Argument.new('PATH', false),
+        ]
+
+        def self.options
+          [
+            ['--allow-warnings', 'Allows push even if there are lint warnings'],
+          ].concat(super)
+        end
+
+        def initialize(argv)
+          @allow_warnings = argv.flag?('allow-warnings')
+          @path = argv.shift_argument || '.'
+          find_podspec_file if File.directory?(@path)
+          super
+        end
+
+        def validate!
+          super
+          unless token
+            help! 'You need to register a session first.'
+          end
+          unless @path
+            help! 'Please specify the path to the podspec file.'
+          end
+          unless File.exist?(@path) && !File.directory?(@path)
+            help! "The specified path `#{@path}` does not point to " \
+                    'an existing podspec file.'
+          end
+        end
+
+        def run
+          validate_podspec
+          response = request_path(:post, 'pods', spec.to_json, auth_headers)
+          url = response.headers['location'].first
+          json = json(request_url(:get, url, default_headers))
+
+          # Using UI.labeled here is dangerous, as it wraps the URL and indents
+          # it, which breaks the URL when you try to copy-paste it.
+          $stdout.puts "  - Data URL: #{json['data_url']}"
+
+          messages = json['messages'].map do |entry|
+            at, message = entry.to_a.flatten
+            "#{formatted_time(at)}: #{message}"
+          end
+          UI.labeled 'Log messages', messages
+        rescue REST::Error => e
+          raise Informative, 'There was an error pushing a new version ' \
+                                   "to trunk: #{e.message}"
+        end
+
+        private
+
+        def find_podspec_file
+          podspecs = Dir[Pathname(@path) + '*.podspec{.json,}']
+          case podspecs.count
+          when 0
+            UI.notice "No podspec found in directory `#{@path}`"
+          when 1
+            UI.notice "Found podspec `#{podspecs[0]}`"
+          else
+            UI.notice "Multiple podspec files in directory `#{@path}`. " \
+                        'You need to explicitly specify which one to use.'
+          end
+          @path = (podspecs.count == 1) ? podspecs[0] : nil
+        end
+
+        def spec
+          @spec ||= Pod::Specification.from_file(@path)
+        rescue Informative # TODO: this should be a more specific error
+          raise Informative, 'Unable to interpret the specified path as a ' \
+                               'podspec.'
+        end
+
+        # Performs a full lint against the podspecs.
+        #
+        # TODO: Currently copied verbatim from `pod push`.
+        def validate_podspec
+          UI.puts 'Validating podspec'.yellow
+
+          validator = Validator.new(spec, %w(https://github.com/CocoaPods/Specs.git))
+          validator.allow_warnings = @allow_warnings
+          validator.validate
+          unless validator.validated?
+            raise Informative, 'The podspec does not validate.'
+          end
+        end
+      end
+    end
+  end
+end

data/lib/pod/command/trunk/register.rb

@@ -0,0 +1,75 @@
+module Pod
+  class Command
+    class Trunk
+      class Register < Trunk
+        self.summary = 'Manage sessions'
+        self.description = <<-DESC
+              Register a new account, or create a new session.
+
+              If this is your first registration, both an `EMAIL` address and your
+              `NAME` are required. If you've already registered with trunk, you may
+              omit the `NAME` (unless you would like to change it).
+
+              It is recommended that you provide a description of the session, so
+              that it will be easier to identify later on. For instance, when you
+              would like to clean-up your sessions. A common example is to specify
+              the location where the machine, that you are using the session for, is
+              physically located.
+
+              Examples:
+
+                  $ pod trunk register eloy@example.com 'Eloy Durán' --description='Personal Laptop'
+                  $ pod trunk register eloy@example.com --description='Work Laptop'
+                  $ pod trunk register eloy@example.com
+        DESC
+
+        self.arguments = [
+          CLAide::Argument.new('EMAIL', true),
+          CLAide::Argument.new('NAME',  false),
+        ]
+
+        def self.options
+          [
+            ['--description=DESCRIPTION', 'An arbitrary description to ' \
+                                              'easily identify your session ' \
+                                              'later on.'],
+          ].concat(super)
+        end
+
+        def initialize(argv)
+          @session_description = argv.option('description')
+          @email, @name = argv.shift_argument, argv.shift_argument
+          super
+        end
+
+        def validate!
+          super
+          unless @email
+            help! 'Specify at least your email address.'
+          end
+        end
+
+        def run
+          body = {
+            'email' => @email,
+            'name' => @name,
+            'description' => @session_description,
+          }.to_json
+          json = json(request_path(:post, 'sessions', body, default_headers))
+          save_token(json['token'])
+          # TODO UI.notice inserts an empty line :/
+          puts '[!] Please verify the session by clicking the link in the ' \
+                   "verification email that has been sent to #{@email}".yellow
+        rescue REST::Error => e
+          raise Informative, 'There was an error registering with trunk: ' \
+                                 "#{e.message}"
+        end
+
+        def save_token(token)
+          netrc['trunk.cocoapods.org'] = @email, token
+          netrc.save
+        end
+      end
+    end
+  end
+end

data/spec/command/trunk/push_spec.rb

@@ -30,7 +30,7 @@ module Pod
           # Execute `pod trunk push` with this dir as parameter
           command = Command.parse(%w(          trunk push          ) + [dir])
           path = command.instance_eval { @path }
-          return path ? File.basename(path) : nil
+          return File.basename(path) if path
         end
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cocoapods-trunk
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Eloy Durán
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-06 00:00:00.000000000 Z
+date: 2014-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nap
@@ -87,9 +87,12 @@ files:
 - Rakefile
 - cocoapods-trunk.gemspec
 - lib/cocoapods_plugin.rb
+- lib/cocoapods_trunk.rb
 - lib/pod/command/trunk.rb
-- lib/trunk.rb
-- lib/trunk/version.rb
+- lib/pod/command/trunk/add_owner.rb
+- lib/pod/command/trunk/me.rb
+- lib/pod/command/trunk/push.rb
+- lib/pod/command/trunk/register.rb
 - spec/command/trunk/addowner_spec.rb
 - spec/command/trunk/me_spec.rb
 - spec/command/trunk/push_spec.rb

data/lib/trunk.rb

@@ -1,7 +0,0 @@
-require 'cocoapods/trunk/version'
-
-module Cocoapods
-  module Trunk
-    # Your code goes here...
-  end
-end

data/lib/trunk/version.rb

@@ -1,5 +0,0 @@
-module Cocoapods
-  module Trunk
-    VERSION = '0.4.0'
-  end
-end