sigh 0.3.5 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5018f22e6ad8a3bfa5abdc11ebe01ef8d2239ca3
-  data.tar.gz: 9c2429680ceaf4303f1886ddb310b0363bafdb03
+  metadata.gz: 4f526a0a5cfb3f72abca695aa7345504a520539f
+  data.tar.gz: 2d750cc363d9776803b4235217727a16a48821ca
 SHA512:
-  metadata.gz: 5046a1d9413148b4a56cd78cac1136b047a0c2db807c8b829bf034c903e7e43eac3c076e8e509ad566e216f1e52b3a7f829ca55088573114b0a7815446332db7
-  data.tar.gz: 9de2b40769911d889e650f22e28806c5d6d364f742d7f330f1132cb0198c5e20e312c6dd5307b88b97494bb79e41e782be2114ac2d0fed82ab9c70ad26f594ff
+  metadata.gz: 73a2d366758dc8eb917a751e0250a5286f1acb89ae3626eb79cfb2f7482916acdaea88d89f01590b354fa1d328f2bb1c21e105db33dd941a68f3984145ec1d7f
+  data.tar.gz: 6fddc9be1d9c09abb57764a6fb75a6226a3027f984ab7141ffe42d4d834eddddc33dc7a21704510b68694a53725e388d1586ba55ea6c41e56d4efb27a9fdb7e8

data/README.md

@@ -47,7 +47,7 @@ Get in contact with the developer on Twitter: [@KrauseFx](https://twitter.com/Kr
 
 -------
 
-<h5 align="center"><code>sigh</code> is part of <a href="http://fastlane.tools">fastlane</a>: connect all deployment tools into one streamlined workflow.</h5>
+<h5 align="center"><code>sigh</code> is part of <a href="https://fastlane.tools">fastlane</a>: connect all deployment tools into one streamlined workflow.</h5>
 
 
 # Features
@@ -169,9 +169,9 @@ If you're using [cert](https://github.com/KrauseFx/cert) in combination with [fa
 ```sigh``` uses the [password manager](https://github.com/KrauseFx/CredentialsManager) from `fastlane`. Take a look the [CredentialsManager README](https://github.com/KrauseFx/CredentialsManager) for more information.
 
 # Tips
-## [`fastlane`](http://fastlane.tools) Toolchain
+## [`fastlane`](https://fastlane.tools) Toolchain
 
-- [`fastlane`](http://fastlane.tools): Connect all deployment tools into one streamlined workflow
+- [`fastlane`](https://fastlane.tools): Connect all deployment tools into one streamlined workflow
 - [`deliver`](https://github.com/KrauseFx/deliver): Upload screenshots, metadata and your app to the App Store using a single command
 - [`snapshot`](https://github.com/KrauseFx/snapshot): Automate taking localized screenshots of your iOS app on every device
 - [`frameit`](https://github.com/KrauseFx/frameit): Quickly put your screenshots into the right device frames

data/bin/sigh

@@ -6,6 +6,7 @@ require 'sigh'
 require 'commander'
 require 'credentials_manager/password_manager'
 require 'credentials_manager/appfile_config'
+require 'sigh/options'
 
 HighLine.track_eof = false
 
@@ -16,53 +17,33 @@ class SighApplication
     program :version, Sigh::VERSION
     program :description, 'CLI for \'sigh\' - Because you would rather spend your time building stuff than fighting provisioning'
     program :help, 'Author', 'Felix Krause <sigh@krausefx.com>'
-    program :help, 'Website', 'http://fastlane.tools'
+    program :help, 'Website', 'https://fastlane.tools'
     program :help, 'GitHub', 'https://github.com/krausefx/sigh'
     program :help_formatter, :compact
 
     always_trace!
 
-    global_option '--adhoc', 'By default, sigh will create and renew App Store profiles. Setting this flag will generate Adhoc profiles instead.'
-    global_option '--skip_install', 'By default, the certificate will be added on your local machine. Setting this flag will skip this action.'
-    global_option '--development', 'Renew the development certificate instead of the production one.'
-    global_option '--force', 'Renew non-development provisioning profiles regardless of state.'
-
-    global_option '-a', '--identifier STRING', String, 'The bundle identifier of your app.'
-    global_option '-u', '--username STRING', String, 'Your Apple ID username.'
-    global_option '-n', '--cert_name STRING', String, 'The name of the generated certificate file.'
-    global_option '-o', '--output STRING', String, 'The folder in which the file should be generated.'
-    global_option '-d', '--cert_date STRING', String, 'The certificate with the given expiry date used to renew. (e.g. "Nov 11, 2017")'
-    global_option '-c', '--cert_owner STRING', String, 'The certificate name to use for new profiles, or to renew with. (e.g. "Felix Krause")'
-    global_option '-f', '--filename STRING', String, 'Filename to use for the provisioning profile'
+    FastlaneCore::CommanderGenerator.new.generate(Sigh::Options.available_options)
 
     command :renew do |c|
       c.syntax = 'sigh renew'
       c.description = 'Renews the certificate (in case it expired) and outputs the path to the generated file'
 
       c.action do |args, options|
-        app = app_identifier(options)
-        username(options)
-
-        type = FastlaneCore::DeveloperCenter::APPSTORE
-        type = FastlaneCore::DeveloperCenter::ADHOC if options.adhoc 
-        type = FastlaneCore::DeveloperCenter::DEVELOPMENT if options.development
+        Sigh.config = FastlaneCore::Configuration.create(Sigh::Options.available_options, options.__hash__)
 
-        ENV['SIGH_CERTIFICATE'] = options.cert_owner if options.cert_owner
-        ENV['SIGH_CERTIFICATE_EXPIRE_DATE'] = options.cert_date if options.cert_date
-
-        path = FastlaneCore::DeveloperCenter.new.run(app, type, options.cert_name, options.force)
+        path = Sigh::DeveloperCenter.new.run
 
         if path
-          file_name = options.filename || File.basename(path)
-          if options.filename == nil
-            file_name = File.basename(path)
+          if Sigh.config[:filename]
+            file_name = Sigh.config[:filename]
           else
-            file_name = options.filename
+            file_name = File.basename(path)
           end
-          output_path = options.output || '.'
-          output = File.join(output_path.gsub("~", ENV["HOME"]), file_name)
-          FileUtils.mv(path, output)
-          system("open -g '#{output}'") unless options.skip_install
+          
+          output = File.join(Sigh.config[:output_path].gsub("~", ENV["HOME"]), file_name)
+          (FileUtils.mv(path, output) rescue nil) # in case it already exists
+          system("open -g '#{output}'") unless Sigh.config[:skip_install]
           puts output.green
         end
       end
@@ -84,21 +65,6 @@ class SighApplication
     run!
   end
 
-  def username(options)
-    user = options.username
-    user ||= ENV["SIGH_USERNAME"]
-    user ||= CredentialsManager::AppfileConfig.try_fetch_value(:apple_id)
-    
-    CredentialsManager::PasswordManager.shared_manager(user) if user
-  end
-
-  def app_identifier(options)
-    value = options.identifier
-    value ||= ENV["SIGH_APP_IDENTIFIER"]
-    value ||= CredentialsManager::AppfileConfig.try_fetch_value(:app_identifier)
-    value ||= ask("App Identifier (Bundle ID, e.g. com.krausefx.app): ")
-    return value
-  end
 end
 
 SighApplication.new.run

data/lib/sigh.rb

@@ -6,6 +6,11 @@ require 'sigh/resign'
 require 'fastlane_core'
 
 module Sigh
+  # Use this to just setup the configuration attribute and set it later somewhere else
+  class << self
+    attr_accessor :config
+  end
+
   Helper = FastlaneCore::Helper # you gotta love Ruby: Helper.* should use the Helper class contained in FastlaneCore
 
   ENV['FASTLANE_TEAM_ID'] ||= ENV["SIGH_TEAM_ID"]

data/lib/sigh/developer_center.rb

@@ -1,28 +1,30 @@
 require 'fastlane_core/developer_center/developer_center'
 require 'sigh/developer_center_signing'
 
-module FastlaneCore
-  class DeveloperCenter
+module Sigh
+  class DeveloperCenter < FastlaneCore::DeveloperCenter
     # Types of certificates
     APPSTORE = "AppStore"
     ADHOC = "AdHoc"
     DEVELOPMENT = "Development"
 
     PROFILES_URL_DEV = "https://developer.apple.com/account/ios/profile/profileList.action?type=limited"
-
     
-    def run(app_identifier, type, cert_name = nil, force = false)
-      cert = maintain_app_certificate(app_identifier, type, force)
+    def run
+      @type = Sigh::DeveloperCenter::APPSTORE
+      @type = Sigh::DeveloperCenter::ADHOC if Sigh.config[:adhoc]
+      @type = Sigh::DeveloperCenter::DEVELOPMENT if Sigh.config[:development]
+
+      cert = maintain_app_certificate # create/download the certificate
       
-      type_name = type
-      type_name = "Distribution" if type == APPSTORE # both enterprise and App Store
-      cert_name ||= "#{type_name}_#{app_identifier}.mobileprovision" # default name
+      type_name = "Distribution" if @type == APPSTORE # both enterprise and App Store
+      cert_name ||= "#{type_name}_#{Sigh.config[:app_identifier]}.mobileprovision" # default name
       cert_name += '.mobileprovision' unless cert_name.include?'mobileprovision'
 
       output_path = File.join(TMP_FOLDER, cert_name)
       File.write(output_path, cert)
 
-      store_provisioning_id_in_environment(output_path) unless ENV["SIGH_SKIP_ANALYSER"]
+      store_provisioning_id_in_environment(output_path)
 
       return output_path
     end
@@ -33,9 +35,10 @@ module FastlaneCore
       ENV["SIGH_UDID"] = udid if udid
     end
 
-    def maintain_app_certificate(app_identifier, type, force)
+    def maintain_app_certificate(force = nil)
+      force = Sigh.config[:force] if (force == nil)
       begin
-        if type == DEVELOPMENT 
+        if @type == DEVELOPMENT 
           visit PROFILES_URL_DEV
         else
           visit PROFILES_URL
@@ -48,29 +51,29 @@ module FastlaneCore
         certs = post_ajax(@list_certs_url)
 
         Helper.log.info "Checking if profile is available. (#{certs['provisioningProfiles'].count} profiles found)"
-        required_cert_types = type == DEVELOPMENT ? ['iOS Development'] : ['iOS Distribution', 'iOS UniversalDistribution']
+        required_cert_types = (@type == DEVELOPMENT ? ['iOS Development'] : ['iOS Distribution', 'iOS UniversalDistribution'])
         certs['provisioningProfiles'].each do |current_cert|
           next unless required_cert_types.include?(current_cert['type'])
           
           details = profile_details(current_cert['provisioningProfileId'])
 
-          if details['provisioningProfile']['appId']['identifier'] == app_identifier
+          if details['provisioningProfile']['appId']['identifier'] == Sigh.config[:app_identifier]
             # that's an Ad Hoc profile. I didn't find a better way to detect if it's one ... skipping it
-            next if type == APPSTORE && details['provisioningProfile']['deviceCount'] > 0
+            next if @type == APPSTORE && details['provisioningProfile']['deviceCount'] > 0
 
             # that's an App Store profile ... skipping it
-            next if type != APPSTORE && details['provisioningProfile']['deviceCount'] == 0
+            next if @type != APPSTORE && details['provisioningProfile']['deviceCount'] == 0
 
             # We found the correct certificate
-            if force && type != DEVELOPMENT
+            if force && @type != DEVELOPMENT
               provisioningProfileId = current_cert['provisioningProfileId']
-              renew_profile(provisioningProfileId, type) # This one needs to be forcefully renewed
-              return maintain_app_certificate(app_identifier, type, false) # recursive
+              renew_profile(provisioningProfileId) # This one needs to be forcefully renewed
+              return maintain_app_certificate(false) # recursive
             elsif current_cert['status'] == 'Active'
               return download_profile(details['provisioningProfile']['provisioningProfileId']) # this one is already finished. Just download it.
             elsif ['Expired', 'Invalid'].include? current_cert['status']
-              renew_profile(current_cert['provisioningProfileId'], type) # This one needs to be renewed
-              return maintain_app_certificate(app_identifier, type, false) # recursive
+              renew_profile(current_cert['provisioningProfileId']) # This one needs to be renewed
+              return maintain_app_certificate(false) # recursive
             end
 
             break
@@ -79,18 +82,19 @@ module FastlaneCore
 
         Helper.log.info "Could not find existing profile. Trying to create a new one."
         # Certificate does not exist yet, we need to create a new one
-        create_profile(app_identifier, type)
+        create_profile
         # After creating the profile, we need to download it
-        return maintain_app_certificate(app_identifier, type, false) # recursive
+        return maintain_app_certificate(false) # recursive
 
       rescue => ex
         error_occured(ex)
       end
     end
 
-    def create_profile(app_identifier, type)
-      Helper.log.info "Creating new profile for app '#{app_identifier}' for type '#{type}'.".yellow
-      certificates = code_signing_certificates(type)
+    def create_profile
+      app_identifier = Sigh.config[:app_identifier]
+      Helper.log.info "Creating new profile for app '#{app_identifier}' for type '#{@type}'.".yellow
+      certificates = code_signing_certificates(@type)
 
       create_url = "https://developer.apple.com/account/ios/profile/profileCreate.action"
       visit create_url
@@ -106,8 +110,8 @@ module FastlaneCore
       end
 
       value = enterprise ? 'inhouse' : 'store'
-      value = 'limited' if type == DEVELOPMENT
-      value = 'adhoc' if type == ADHOC
+      value = 'limited' if @type == DEVELOPMENT
+      value = 'adhoc' if @type == ADHOC
 
       first(:xpath, "//input[@type='radio' and @value='#{value}']").click
       click_next
@@ -137,7 +141,7 @@ module FastlaneCore
       clicked = false
       certificates.each do |cert|
         cert_id = cert['certificateId']
-        input = if type == DEVELOPMENT
+        input = if @type == DEVELOPMENT
           # development uses a checkbox and has no [] around the value
           first(:xpath, "//input[@type='checkbox' and @value='#{cert_id}']")
         else
@@ -156,7 +160,7 @@ module FastlaneCore
       end
       click_next
 
-      if type != APPSTORE
+      if @type != APPSTORE
         # 4) Devices selection
         wait_for_elements('.selectAll.column')
         sleep 3
@@ -167,15 +171,15 @@ module FastlaneCore
 
       # 5) Choose a profile name
       wait_for_elements('.distributionType')
-      profile_name = ENV["SIGH_PROVISIONING_PROFILE_NAME"]
-      profile_name ||= [app_identifier, type].join(' ')
+      profile_name = Sigh.config[:provisioning_file_name]
+      profile_name ||= [app_identifier, @type].join(' ')
       fill_in "provisioningProfileName", with: profile_name
       click_next
       wait_for_elements('.row-details')
     end
 
-    def renew_profile(profile_id, type)
-      certificate = code_signing_certificates(type).first
+    def renew_profile(profile_id)
+      certificate = code_signing_certificates(@type).first
 
       details_url = "https://developer.apple.com/account/ios/profile/profileEdit.action?type=&provisioningProfileId=#{profile_id}"
       Helper.log.info "Renewing provisioning profile '#{profile_id}' using URL '#{details_url}'"
@@ -188,7 +192,7 @@ module FastlaneCore
       if certs.count == 1
         certs.first.click
 
-        if type != APPSTORE
+        if @type != APPSTORE
           # Add all devices
           wait_for_elements('.selectAll.column')
           sleep 3

data/lib/sigh/developer_center_signing.rb

@@ -1,5 +1,5 @@
-module FastlaneCore
-  class DeveloperCenter
+module Sigh
+  class DeveloperCenter < FastlaneCore::DeveloperCenter
     # Returns a array of hashes, that contains information about the iOS certificate
     # @example
       # [{"certRequestId"=>"B23Q2P396B",
@@ -37,9 +37,9 @@ module FastlaneCore
       ret_certs = []
 
       # Select certificate
-      certificate_name = ENV['SIGH_CERTIFICATE']
-      cert_date = ENV['SIGH_CERTIFICATE_EXPIRE_DATE']
-      cert_id = ENV['SIGH_CERTIFICATE_ID']
+      certificate_name = Sigh.config[:cert_owner_name]
+      cert_date = Sigh.config[:cert_date]
+      cert_id = Sigh.config[:cert_id]
 
       # The other profiles are push profiles
       certificate_type = type == DEVELOPMENT ? 'iOS Development' : 'iOS Distribution'

data/lib/sigh/options.rb

@@ -0,0 +1,74 @@
+require 'fastlane_core'
+
+module Sigh
+  class Options
+    def self.available_options
+      @@options ||= [
+        FastlaneCore::ConfigItem.new(key: :adhoc,
+                                     env_name: "SIGH_AD_HOC",
+                                     description: "Setting this flag will generate AdHoc profiles instead of App Store Profiles",
+                                     is_string: false),
+        FastlaneCore::ConfigItem.new(key: :skip_install,
+                                     env_name: "SIGH_SKIP_INSTALL",
+                                     description: "By default, the certificate will be added on your local machine. Setting this flag will skip this action",
+                                     is_string: false),
+        FastlaneCore::ConfigItem.new(key: :development,
+                                     env_name: "SIGH_DEVELOPMENT",
+                                     description: "Renew the development certificate instead of the production one",
+                                     is_string: false),
+        FastlaneCore::ConfigItem.new(key: :force,
+                                     env_name: "SIGH_FORCE",
+                                     description: "Renew non-development provisioning profiles regardless of its state",
+                                     is_string: false),
+        FastlaneCore::ConfigItem.new(key: :app_identifier,
+                                     short_option: "-a",
+                                     env_name: "SIGH_APP_IDENTIFIER",
+                                     description: "The bundle identifier of your app",
+                                     default_value: CredentialsManager::AppfileConfig.try_fetch_value(:app_identifier)),
+        FastlaneCore::ConfigItem.new(key: :username,
+                                     short_option: "-u",
+                                     env_name: "SIGH_USERNAME",
+                                     description: "Your Apple ID Username",
+                                     default_value: CredentialsManager::AppfileConfig.try_fetch_value(:apple_id),
+                                     verify_block: Proc.new do |value|
+                                       CredentialsManager::PasswordManager.shared_manager(value)
+                                     end),
+        FastlaneCore::ConfigItem.new(key: :provisioning_file_name,
+                                     short_option: "-n",
+                                     env_name: "SIGH_PROVISIONING_PROFILE_NAME",
+                                     description: "The name of the generated certificate file"),
+        FastlaneCore::ConfigItem.new(key: :output_path,
+                                     short_option: "-o",
+                                     env_name: "SIGH_OUTPUT_PATH",
+                                     description: "Directory in which the profile should be stored",
+                                     default_value: ".",
+                                     verify_block: Proc.new do |value|
+                                       raise "Could not find output directory '#{value}'".red unless File.exists?(value)
+                                     end),
+        FastlaneCore::ConfigItem.new(key: :cert_id,
+                                     short_option: "-i",
+                                     env_name: "SIGH_CERTIFICATE_ID",
+                                     description: "The ID of the certificate to use",
+                                     optional: true),
+        FastlaneCore::ConfigItem.new(key: :cert_owner_name,
+                                     short_option: "-c",
+                                     env_name: "SIGH_CERTIFICATE",
+                                     description: "The certificate name to use for new profiles, or to renew with. (e.g. \"Felix Krause\")",
+                                     optional: true),
+        FastlaneCore::ConfigItem.new(key: :cert_date,
+                                     short_option: "-d",
+                                     env_name: "SIGH_CERTIFICATE_EXPIRE_DATE",
+                                     description: "The certificate with the given expiry date used to renew. (e.g. \"Nov 11, 2017\")",
+                                     optional: true),
+        FastlaneCore::ConfigItem.new(key: :filename,
+                                     short_option: "-f",
+                                     env_name: "SIGH_PROFILE_FILE_NAME",
+                                     optional: true,
+                                     description: "Filename to use for the generated provisioning profile (must include .mobileprovision)",
+                                     verify_block: Proc.new do |value|
+                                       raise "The output name must end with .mobileprovision".red unless value.end_with?".mobileprovision"
+                                     end)
+      ]
+    end
+  end
+end

data/lib/sigh/profile_analyser.rb

@@ -3,7 +3,7 @@ require 'plist'
 module Sigh
   class ProfileAnalyser
     def self.run(path)
-      plist = Plist::parse_xml(`security cms -D -i #{path}`)
+      plist = Plist::parse_xml(`security cms -D -i '#{path}'`)
       if plist.count > 10
         Helper.log.info("Provisioning profile of app '#{plist['AppIDName']}' with the name '#{plist['Name']}' successfully generated and analysed.".green)
         return plist["UUID"]

data/lib/sigh/resign.rb

@@ -1,60 +1,92 @@
+require 'shellwords'
+
 module Sigh
   # Resigns an existing ipa file
   class Resign
     def run(options, args)
-      get_inputs(options, args)
+      # get the command line inputs and parse those into the vars we need...
+      ipa, signing_identity, provisioning_profile = get_inputs(options, args)
+
+      # ... then invoke out programmatic interface with these vars
+      resign(ipa, signing_identity, provisioning_profile)
+    end
+
+    def self.resign(ipa, signing_identity, provisioning_profile)
+      self.new.resign(ipa, signing_identity, provisioning_profile)
+    end
+
+    def resign(ipa, signing_identity, provisioning_profile)
+      resign_path = find_resign_path
+      signing_identity = find_signing_identity(signing_identity)
+
+      # validate that we have valid values for all these params, we don't need to check signing_identity because `find_signing_identity` will only ever return a valid value
+      validate_params(resign_path, ipa, provisioning_profile)
 
       command = [
-        @resign_path,
-        "'#{@ipa}'",
-        "'#{@signing_identity}'",
-        "-p '#{@provisioning_profile}'",
-        "'#{@ipa}'"
+        resign_path.shellescape,
+        ipa.shellescape,
+        signing_identity.shellescape,
+        "-p #{provisioning_profile.shellescape}",
+        ipa.shellescape
       ].join(' ')
 
       puts command.magenta
       output = `#{command}`
       puts output
-      if output.include?"Assuming Distribution Identity"
-        Helper.log.info "Successfully signed #{@ipa}!".green
+      if output.include?('Assuming Distribution Identity')
+        Helper.log.info "Successfully signed #{ipa}!".green
+        true
       else
-        Helper.log.fatal "Something went wrong while code signing #{@ipa}".red
+        Helper.log.fatal "Something went wrong while code signing #{ipa}".red
+        false
       end
     end
 
     def get_inputs(options, args)
-      @resign_path = File.join(Helper.gem_path('sigh'), 'lib', 'assets', 'resign.sh')
-      raise "Could not find resign.sh file. Please try re-installing the gem.".red unless File.exists?@resign_path
-
-      @ipa = args.first || find_ipa || ask("Path to ipa file: ")
-      validate_ipa_file!
-      @signing_identity = options.signing_identity || ask_for_signing_identity
-      validate_signing_identity
-      @provisioning_profile = options.provisioning_profile || find_provisioning_profile || ask("Path to provisioning file: ")
-      validate_provisioning_file!
+      ipa = args.first || find_ipa || ask('Path to ipa file: ')
+      signing_identity = options.signing_identity || ask_for_signing_identity
+      provisioning_profile = options.provisioning_profile || find_provisioning_profile || ask('Path to provisioning file: ')
+
+      return ipa, signing_identity, provisioning_profile
+    end
+
+    def find_resign_path
+      File.join(Helper.gem_path('sigh'), 'lib', 'assets', 'resign.sh')
     end
 
     def find_ipa
-      Dir[File.join(Dir.pwd, "*.ipa")].sort { |a,b| File.mtime(a) <=> File.mtime(b) }.first
+      Dir[File.join(Dir.pwd, '*.ipa')].sort { |a, b| File.mtime(a) <=> File.mtime(b) }.first
     end
 
     def find_provisioning_profile
-      Dir[File.join(Dir.pwd, "*.mobileprovision")].sort { |a,b| File.mtime(a) <=> File.mtime(b) }.first
+      Dir[File.join(Dir.pwd, '*.mobileprovision')].sort { |a, b| File.mtime(a) <=> File.mtime(b) }.first
     end
 
-    def validate_ipa_file!
-      raise "ipa file could not be found or is not an ipa file (#{@ipa})".red unless (File.exists?(@ipa) and @ipa.end_with?".ipa")
+    def find_signing_identity(signing_identity)
+      until installed_identies.include?(signing_identity)
+        Helper.log.error "Couldn't find signing identity '#{signing_identity}'."
+        signing_identity = ask_for_signing_identity
+      end
+
+      signing_identity
     end
 
-    def validate_provisioning_file!
-      raise "Provisioning profile file could not be found or is not a .mobileprovision file (#{@provisioning_profile})".red unless (File.exists?(@provisioning_profile) and @provisioning_profile.end_with?".mobileprovision")
+    def validate_params(resign_path, ipa, provisioning_profile)
+      validate_resign_path(resign_path)
+      validate_ipa_file(ipa)
+      validate_provisioning_file(provisioning_profile)
     end
 
-    def validate_signing_identity
-      while not installed_identies.include?@signing_identity
-        Helper.log.error "Couldn't find signing identity '#{@signing_identity}'."
-        @signing_identity = ask_for_signing_identity
-      end
+    def validate_resign_path(resign_path)
+      raise 'Could not find resign.sh file. Please try re-installing the gem.'.red unless File.exist?(resign_path)
+    end
+
+    def validate_ipa_file(ipa)
+      raise "ipa file could not be found or is not an ipa file (#{ipa})".red unless File.exist?(ipa) && ipa.end_with?('.ipa')
+    end
+
+    def validate_provisioning_file(provisioning_profile)
+      raise "Provisioning profile file could not be found or is not a .mobileprovision file (#{provisioning_profile})".red unless File.exists?(provisioning_profile) && provisioning_profile.end_with?('.mobileprovision')
     end
 
     def print_available_identities
@@ -63,7 +95,7 @@ module Sigh
 
     def ask_for_signing_identity
       print_available_identities
-      ask("Signing Identity: ")
+      ask('Signing Identity: ')
     end
 
     # Array of available signing identities
@@ -74,7 +106,7 @@ module Sigh
         (ids << current.match(/.*\"(.*)\"/)[1]) rescue nil # the last line does not match
       end
 
-      return ids
+      ids
     end
   end
-end
+end

data/lib/sigh/version.rb

@@ -1,3 +1,3 @@
 module Sigh
-  VERSION = "0.3.5"
+  VERSION = "0.4.1"
 end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: sigh
 version: !ruby/object:Gem::Version
-  version: 0.3.5
+  version: 0.4.1
 platform: ruby
 authors:
 - Felix Krause
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-27 00:00:00.000000000 Z
+date: 2015-03-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fastlane_core
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - '>='
       - !ruby/object:Gem::Version
-        version: 0.2.0
+        version: 0.3.0
 - !ruby/object:Gem::Dependency
   name: plist
   requirement: !ruby/object:Gem::Requirement
@@ -153,10 +153,11 @@ files:
 - lib/sigh/dependency_checker.rb
 - lib/sigh/developer_center.rb
 - lib/sigh/developer_center_signing.rb
+- lib/sigh/options.rb
 - lib/sigh/profile_analyser.rb
 - lib/sigh/resign.rb
 - lib/sigh/version.rb
-homepage: http://fastlane.tools
+homepage: https://fastlane.tools
 licenses:
 - MIT
 metadata: {}