xcode-install 2.4.1 → 2.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e28c74420ebf53e8f127263aa2752211dff9bcd3
-  data.tar.gz: e48f8d0fd268fdd9756a15dbbfdf162c29a5268d
+  metadata.gz: f963f106ba542c0705d573d9262d8fb92595df9d
+  data.tar.gz: ac247b88aba2b1548179b04c7e7a6eb6358c5004
 SHA512:
-  metadata.gz: d0224e430c804c00c9a04e6cc37c426f7a363c29f449ebbd64fb12ed8b1f3b6acc2b23c612ad5dfd4ef30b6610d3e731688b4cc7fc58f9ac685da3c66bcb8535
-  data.tar.gz: 6d861cc3114665ec9f1bdd53ca438b29e56ce3345b83e1831e8858e3935d61981e767d3c17818e54e4a600e770b70e9ea76e7ae09870b4ffa1e8f23adc581c57
+  metadata.gz: e525ac44989fb47d31f0b976228eb1952f4d710fef60def7c3aaee7059312332c9d29e258b3a3a0760f6d39d2a1b067af9f3b3fc622c49a3780a14bd8fd61593
+  data.tar.gz: e630821045db39f07f64840f17e6d96df9ba3cfaeca05cc3391f9b9dc8843d48038382580b96743c3ea5868afda8aed92e8135153bea24320680b99f4f7933c9

data/README.md

@@ -153,7 +153,7 @@ XcodeInstall normally relies on the Spotlight index to locate installed versions
 indexing is happening, it might show inaccurate results and it will not be able to see installed
 versions on unindexed volumes. 
 
-To workaround the Spotlight limitation, XcodeInstall searches `/Applications` folder to locate Xcodes when Spotlight is disabled on the machine, or when Spotlight query for Xcode does not return any results. But it still won't work if your Xcodes are located under `/Applications` folder.
+To workaround the Spotlight limitation, XcodeInstall searches `/Applications` folder to locate Xcodes when Spotlight is disabled on the machine, or when Spotlight query for Xcode does not return any results. But it still won't work if your Xcodes are not located under `/Applications` folder.
 
 ## Thanks
 

data/lib/xcode/install/install.rb

@@ -23,6 +23,7 @@ module XcodeInstall
       def initialize(argv)
         @installer = Installer.new
         @version = argv.shift_argument
+        @version ||= File.read('.xcode-version') if File.exist?('.xcode-version')
         @url = argv.option('url')
         @force = argv.flag?('force', false)
         @should_clean = argv.flag?('clean', true)
@@ -37,7 +38,10 @@ module XcodeInstall
         super
 
         help! 'A VERSION argument is required.' unless @version
-        fail Informative, "Version #{@version} already installed." if @installer.installed?(@version) && !@force
+        if @installer.installed?(@version) && !@force
+          print "Version #{@version} already installed."
+          exit(0)
+        end
         fail Informative, "Version #{@version} doesn't exist." unless @url || @installer.exist?(@version)
         fail Informative, "Invalid URL: `#{@url}`" unless !@url || @url =~ /\A#{URI.regexp}\z/
       end

data/lib/xcode/install/version.rb

@@ -1,3 +1,3 @@
 module XcodeInstall
-  VERSION = '2.4.1'.freeze
+  VERSION = '2.5.0'.freeze
 end

data/lib/xcode/install.rb

@@ -95,7 +95,7 @@ module XcodeInstall
 
           # Call back the block for other processes that might be interested
           matched = progress_content.match(/^\s*(\d+)/)
-          next unless matched.length == 2
+          next unless matched && matched.length == 2
           percent = matched[1].to_i
           progress_block.call(percent) if progress_block
         end
@@ -277,7 +277,7 @@ HELP
       fail Informative, "Failed to download Xcode #{version}." if dmg_path.nil?
 
       if install
-        install_dmg(dmg_path, "-#{version.to_s.split(' ')[0]}", switch, clean)
+        install_dmg(dmg_path, "-#{version.to_s.split(' ').join('.')}", switch, clean)
       else
         puts "Downloaded Xcode #{version} to '#{dmg_path}'"
       end
@@ -293,7 +293,12 @@ HELP
 
     def list_annotated(xcodes_list)
       installed = installed_versions.map(&:version)
-      xcodes_list.map { |x| installed.include?(x) ? "#{x} (installed)" : x }.join("\n")
+      xcodes_list.map do |x|
+        xcode_version = x.split(' ').first # exclude "beta N", "for Lion".
+        xcode_version << '.0' unless xcode_version.include?('.')
+
+        installed.include?(xcode_version) ? "#{x} (installed)" : x
+      end.join("\n")
     end
 
     def list
@@ -383,7 +388,7 @@ HELP
     def installed
       result = `mdfind "kMDItemCFBundleIdentifier == 'com.apple.dt.Xcode'" 2>/dev/null`.split("\n")
       if result.empty?
-        result = `find /Applications -name '*.app' -type d -maxdepth 1 -exec sh -c \
+        result = `find /Applications -maxdepth 1 -name '*.app' -type d -exec sh -c \
         'if [ "$(/usr/libexec/PlistBuddy -c "Print :CFBundleIdentifier" \
         "{}/Contents/Info.plist" 2>/dev/null)" == "com.apple.dt.Xcode" ]; then echo "{}"; fi' ';'`.split("\n")
       end
@@ -435,7 +440,7 @@ HELP
         return [] if scan.empty?
 
         version = scan.first.gsub(/<.*?>/, '').gsub(/.*Xcode /, '')
-        link = body.scan(%r{<button .*"(.+?.xip)".*</button>}).first.first
+        link = body.scan(%r{<button .*"(.+?.(dmg|xip))".*</button>}).first.first
         notes = body.scan(%r{<a.+?href="(/go/\?id=xcode-.+?)".*>(.*)</a>}).first.first
         links << Xcode.new(version, link, notes)
       end
@@ -589,7 +594,7 @@ HELP
     end
 
     def bundle_version
-      @bundle_version ||= Gem::Version.new(plist_entry(':DTXcode').to_i.to_s.split(//).join('.'))
+      @bundle_version ||= Gem::Version.new(bundle_version_string)
     end
 
     def uuid
@@ -652,6 +657,15 @@ HELP
 
     :private
 
+    def bundle_version_string
+      digits = plist_entry(':DTXcode').to_i.to_s
+      if digits.length < 3
+        digits.split(//).join('.')
+      else
+        "#{digits[0..-3]}.#{digits[-2]}.#{digits[-1]}"
+      end
+    end
+
     def plist_entry(keypath)
       `/usr/libexec/PlistBuddy -c "Print :#{keypath}" "#{path}/Contents/Info.plist"`.chomp
     end

data/spec/install_spec.rb

@@ -35,6 +35,14 @@ module XcodeInstall
         Installer.any_instance.expects(:install_dmg).with('/some/path', '-6.3', true, true)
         Command::Install.run(['6.3', '--no-progress'])
       end
+
+      it 'reads .xcode-version' do
+        Installer.any_instance.expects(:download).with('6.3', true, nil, nil).returns('/some/path')
+        Installer.any_instance.expects(:install_dmg).with('/some/path', '-6.3', true, true)
+        File.expects(:exist?).with('.xcode-version').returns(true)
+        File.expects(:read).returns('6.3')
+        Command::Install.run([])
+      end
     end
 
     it 'parses hdiutil output' do

data/spec/list_spec.rb

@@ -4,7 +4,6 @@ module XcodeInstall
   describe Command::List do
     before do
       installer.stubs(:exists).returns(true)
-      installer.stubs(:installed_versions).returns([])
     end
 
     def installer
@@ -23,11 +22,36 @@ module XcodeInstall
       installer.stubs(:xcodes).returns(xcodes)
     end
 
+    def fake_installed_xcode(name)
+      xcode_path = "/Applications/Xcode-#{name}.app"
+      xcode_version = name
+      xcode_version << '.0' unless name.include? '.'
+
+      installed_xcode = InstalledXcode.new(xcode_path)
+      installed_xcode.stubs(:version).returns(xcode_version)
+      installed_xcode.stubs(:bundle_version).returns(Gem::Version.new(xcode_version))
+      installed_xcode
+    end
+
+    def fake_installed_xcodes(*names)
+      xcodes = names.map { |name| fake_installed_xcode(name) }
+      installer.stubs(:installed_versions).returns(xcodes)
+    end
+
     describe '#list' do
       it 'lists all versions' do
         fake_xcodes '1', '2.3', '2.3.1', '2.3.2', '3 some', '4 beta', '10 beta'
+        fake_installed_xcodes
         installer.list.should == "1\n2.3\n2.3.1\n2.3.2\n3 some\n4 beta\n10 beta"
       end
     end
+
+    describe '#list_annotated' do
+      it 'lists all versions with annotations' do
+        fake_xcodes '1', '2.3', '2.3.1', '2.3.2', '3 some', '4.3.1 for Lion', '9.4.1', '10 beta'
+        fake_installed_xcodes '2.3', '4.3.1', '10'
+        installer.list.should == "1\n2.3 (installed)\n2.3.1\n2.3.2\n3 some\n4.3.1 for Lion (installed)\n9.4.1\n10 beta (installed)"
+      end
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: xcode-install
 version: !ruby/object:Gem::Version
-  version: 2.4.1
+  version: 2.5.0
 platform: ruby
 authors:
 - Boris Bügling
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-06-07 00:00:00.000000000 Z
+date: 2018-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: claide
@@ -160,7 +160,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.10
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: Xcode installation manager.