gemirro 0.13.3 → 0.13.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 82b43d4fb674a2d477ca6e36892ce24cb75dc27e
-  data.tar.gz: 6722f23ee14197f2eed5fdb3089b0bfaddb3321b
+  metadata.gz: 8a04e50004bf7949e1460687017117748843d66b
+  data.tar.gz: 43fea5e3676366061ddbdd9cb7ff0785feea2013
 SHA512:
-  metadata.gz: 72c637dddc14a1a6f6f9649df99fd6e04dbbcfce0926b99c44ef084a8bda8d148507616ac80497df4e7ed609c37b3a7112fa310d1d64319995699ea88c6df388
-  data.tar.gz: 0318c861a49c17318dec009db0284022459e839a6fdd2b74f26c5be1b4a784540b0d120ef09085bf95d932fe1e485f1434ad9734255e44194cc86c978894fbce
+  metadata.gz: 1e5019265061371cd6abba03fa3ed675898425615fba0bbe52a4f7eff53dede9f6a7d18e0f88a064585f34bcadf7076b7ec928117a4231a78769bdb50bd0b2f8
+  data.tar.gz: f3f21f3b670904827f8784ac52554441e9dde294adb88ffff2f76c439e91fb5672452a5ef6a59221efb3d86794ba53e3c2e25923e72fa1beaf1004229396c7ca

data/README.md

@@ -6,7 +6,7 @@ More, to mirroring a source, you only need to start the server, and gems will au
 
 ## Requirements
 
-* Ruby 1.9.2 or newer
+* Ruby 2.0.0 or newer
 * Enough space to store Gems
 * A recent version of Rubygems (`gem update --system`)
 
@@ -80,7 +80,7 @@ See `<command> --help` for more information on a specific command.
 
 ## Apache configuration
 
-You must active the apache `proxy` module.
+You must activate the apache `proxy` module.
 
 ```bash
 $ sudo a2enmod proxy
@@ -94,8 +94,8 @@ Create your VirtualHost and replace following `http://localhost:2000` with your
   ServerName mirror.gemirro
   ProxyPreserveHost On
   ProxyRequests off
-  ProxyPass / http://localhost:2000
-  ProxyPassReverse / http://localhost:2000
+  ProxyPass / http://localhost:2000/
+  ProxyPassReverse / http://localhost:2000/
 </VirtualHost>
 ```
 

data/lib/gemirro/cli/server.rb

@@ -43,7 +43,7 @@ Gemirro::CLI.options.command 'server' do
 
   def create_pid
     File.open(@pid_file, 'w') do |f|
-      f.write("#{Process.pid}")
+      f.write(Process.pid.to_s)
     end
   rescue Errno::EACCES
     STDOUT.reopen @orig_stdout

data/lib/gemirro/cli.rb

@@ -11,7 +11,7 @@ module Gemirro
       strict: true,
       help: true,
       banner: 'Usage: gemirro [COMMAND] [OPTIONS]'
-    }
+    }.freeze
 
     ##
     # @return [Slop]

data/lib/gemirro/configuration.rb

@@ -25,13 +25,13 @@ module Gemirro
     attr_accessor :source, :ignored_gems, :logger
 
     LOGGER_LEVEL = {
-      'debug' =>  Logger::DEBUG,
+      'debug' => Logger::DEBUG,
       'warning' => Logger::WARN,
       'info' => Logger::INFO,
       'unknown' => Logger::UNKNOWN,
       'error' => Logger::ERROR,
       'fatal' => Logger::FATAL
-    }
+    }.freeze
 
     ##
     # Returns the logger

data/lib/gemirro/gem.rb

@@ -15,7 +15,7 @@ module Gemirro
   class Gem
     attr_reader :name, :requirement, :platform, :version
     attr_accessor :gemspec
-    ONLY_LATEST = [:latest, :newest]
+    ONLY_LATEST = [:latest, :newest].freeze
 
     ##
     # Returns a `Gem::Version` instance based on the specified requirement.
@@ -58,7 +58,7 @@ module Gemirro
     # @return [TrueClass|FalseClass]
     #
     def version?
-      version && !version.segments.reject { |s| s == 0 }.empty?
+      version && !version.segments.reject(&:zero?).empty?
     end
 
     ##

data/lib/gemirro/gem_version.rb

@@ -55,8 +55,8 @@ module Gemirro
     def <=>(other)
       sort = other.name <=> @name
       sort = version <=> other.version if sort.zero?
-      sort = (other.ruby? && !ruby?) ? 1 : -1 if sort.zero? &&
-                                                 ruby? != other.ruby?
+      sort = other.ruby? && !ruby? ? 1 : -1 if sort.zero? &&
+                                               ruby? != other.ruby?
       sort = other.platform <=> @platform if sort.zero?
 
       sort

data/lib/gemirro/gems_fetcher.rb

@@ -30,11 +30,11 @@ module Gemirro
           if gem.gemspec?
             gemfile = fetch_gemspec(gem, version)
             Utils.configuration.mirror_gemspecs_directory
-              .add_file(gem.gemspec_filename(version), gemfile) if gemfile
+                 .add_file(gem.gemspec_filename(version), gemfile) if gemfile
           else
             gemfile = fetch_gem(gem, version)
             Utils.configuration.mirror_gems_directory
-              .add_file(gem.filename(version), gemfile) if gemfile
+                 .add_file(gem.filename(version), gemfile) if gemfile
           end
         end
       end
@@ -68,7 +68,8 @@ module Gemirro
     # @return [String]
     #
     def fetch_gemspec(gem, version)
-      filename  = gem.gemspec_filename(version)
+      filename = gem.gemspec_filename(version)
+      puts filename.inspect
       satisfied = if gem.only_latest?
                     true
                   else
@@ -81,7 +82,7 @@ module Gemirro
       end
 
       Utils.logger.info("Fetching #{filename}")
-      fetch_from_source(gem, version, true)
+      fetch_from_source(filename, gem, version, true)
     end
 
     ##
@@ -108,21 +109,21 @@ module Gemirro
       Utils.configuration.ignore_gem(gem.name, version)
       Utils.logger.info("Fetching #{filename}")
 
-      fetch_from_source(gem, version)
+      fetch_from_source(filename, gem, version)
     end
 
     ##
     #
-    #
+    # @param [String] filename
     # @param [Gemirro::Gem] gem
     # @param [Gem::Version] version
     # @return [String]
     #
-    def fetch_from_source(gem, version, gemspec = false)
+    def fetch_from_source(filename, gem, version, gemspec = false)
       data = nil
       begin
-        data = @source.fetch_gem(gem.name, version) unless gemspec
-        data = @source.fetch_gemspec(gem.name, version) if gemspec
+        data = @source.fetch_gem(filename) unless gemspec
+        data = @source.fetch_gemspec(filename) if gemspec
       rescue => e
         filename = gem.filename(version)
         Utils.logger.error("Failed to retrieve #{filename}: #{e.message}")

data/lib/gemirro/http.rb

@@ -21,7 +21,7 @@ module Gemirro
       response = client.get(url, follow_redirect: true)
 
       unless HTTP::Status.successful?(response.status)
-        fail HTTPClient::BadResponseError, response.reason
+        raise HTTPClient::BadResponseError, response.reason
       end
 
       response

data/lib/gemirro/indexer.rb

@@ -39,7 +39,7 @@ module Gemirro
       require 'zlib'
 
       unless defined?(Builder::XChar)
-        fail 'Gem::Indexer requires that the XML Builder ' \
+        raise 'Gem::Indexer requires that the XML Builder ' \
         'library be installed:' \
         "\n\tgem install builder"
       end
@@ -93,7 +93,7 @@ module Gemirro
     #
     def install_indicies
       Utils.logger
-        .debug("Downloading index into production dir #{@dest_directory}")
+           .debug("Downloading index into production dir #{@dest_directory}")
 
       files = @files
       files.delete @quick_marshal_dir if files.include? @quick_dir
@@ -184,13 +184,19 @@ module Gemirro
         Utils.logger.info("[#{index + 1}/#{gems.size}]: Processing #{gemfile.split('/')[-1]}")
         # rubocop:enable Metrics/LineLength
 
-        if File.size(gemfile) == 0
+        if File.size(gemfile).zero?
           Utils.logger.warn("Skipping zero-length gem: #{gemfile}")
           next
         end
 
         begin
-          spec = ::Gem::Package.new(gemfile).spec
+          if ::Gem::Package.respond_to? :open
+            spec = ::Gem::Package
+                   .open(File.open(gemfile, 'rb'), 'r', &:metadata)
+          else
+            spec = ::Gem::Package.new(gemfile).spec
+          end
+
           spec.loaded_from = gemfile
 
           # HACK: fuck this shit - borks all tests that use pl1
@@ -203,6 +209,13 @@ module Gemirro
             next
           end
 
+          version = spec.version.version
+          unless version =~ /^\d+\.\d+\.\d+.*/
+            msg = "Skipping gem #{spec.full_name} - invalid version #{version}"
+            Utils.logger.warn(msg)
+            next
+          end
+
           if ::Gem::VERSION >= '2.5.0'
             spec.abbreviate
             spec.sanitize
@@ -247,11 +260,11 @@ module Gemirro
 
       ::Gem::Specification.dirs = []
       ::Gem::Specification.all = *specs
-      if ::Gem::VERSION >= '2.5.0'
-        files = build_marshal_gemspecs specs
-      else
-        files = build_marshal_gemspecs
-      end
+      files = if ::Gem::VERSION >= '2.5.0'
+                build_marshal_gemspecs specs
+              else
+                build_marshal_gemspecs
+              end
 
       ::Gem.time('Updated indexes') do
         update_specs_index(released, @dest_specs_index, @specs_index)

data/lib/gemirro/server.rb

@@ -8,9 +8,9 @@ module Gemirro
   #
   class Server < Sinatra::Base
     # rubocop:disable Metrics/LineLength
-    URI_REGEXP = /^(.*)-(\d+(?:\.\d+){1,4}.*?)(?:-x86-(?:(?:mswin|mingw)(?:32|64)).*?)?\.(gem(?:spec\.rz)?)$/
-    GEMSPEC_TYPE = 'gemspec.rz'
-    GEM_TYPE = 'gem'
+    URI_REGEXP = /^(.*)-(\d+(?:\.\d+){1,4}.*?)(?:-(x86-(?:(?:mswin|mingw)(?:32|64)).*?|java))?\.(gem(?:spec\.rz)?)$/
+    GEMSPEC_TYPE = 'gemspec.rz'.freeze
+    GEM_TYPE = 'gem'.freeze
 
     access_logger = Logger.new(Utils.configuration.server.access_log).tap do |logger|
       ::Logger.class_eval { alias_method :write, :'<<' }
@@ -130,11 +130,11 @@ module Gemirro
       result = name.match(URI_REGEXP)
       return unless result
 
-      gem_name, gem_version, gem_type = result.captures
+      gem_name, gem_version, gem_platform, gem_type = result.captures
       return unless gem_name && gem_version
 
       begin
-        gem = Utils.stored_gem(gem_name, gem_version)
+        gem = Utils.stored_gem(gem_name, gem_version, gem_platform)
         gem.gemspec = true if gem_type == GEMSPEC_TYPE
 
         # rubocop:disable Metrics/LineLength
@@ -143,7 +143,7 @@ module Gemirro
         # rubocop:enable Metrics/LineLength
 
         Utils.logger
-          .info("Try to download #{gem_name} with version #{gem_version}")
+             .info("Try to download #{gem_name} with version #{gem_version}")
         Utils.gems_fetcher.source.gems.clear
         Utils.gems_fetcher.source.gems.push(gem)
         Utils.gems_fetcher.fetch

data/lib/gemirro/source.rb

@@ -58,26 +58,25 @@ module Gemirro
     # @param [String] version
     # @return [String]
     #
-    def fetch_gem(name, version)
+    def fetch_gem(filename)
       Utils.logger.info(
-        "Fetching gem #{name}, #{version} on #{@name} (#{@host})"
+        "Fetching gem #{filename} on #{@host}"
       )
-      Http.get(host + "/gems/#{name}-#{version}.gem").body
+      Http.get(host + "/gems/#{filename}").body
     end
 
     ##
     # Fetches the `.gemspec.rz` file of a given Gem and version.
     #
-    # @param [String] name
-    # @param [String] version
+    # @param [String] filename
     # @return [String]
     #
-    def fetch_gemspec(name, version)
+    def fetch_gemspec(filename)
       Utils.logger.info(
-        "Fetching gemspec #{name}, #{version} on #{@name} (#{@host})"
+        "Fetching gemspec #{filename} on #{@host}"
       )
       marshal = Gemirro::Configuration.marshal_identifier
-      Http.get(host + "/quick/#{marshal}/#{name}-#{version}.gemspec.rz").body
+      Http.get(host + "/quick/#{marshal}/#{filename}").body
     end
 
     ##

data/lib/gemirro/utils.rb

@@ -16,8 +16,7 @@ module Gemirro
     attr_reader(:cache,
                 :versions_fetcher,
                 :gems_fetcher,
-                :gems_orig_collection,
-                :gems_source_collection,
+                :gems_collection,
                 :stored_gems)
     ##
     # Cache class to store marshal and data into files
@@ -36,21 +35,32 @@ module Gemirro
     # @return [Gemirro::GemVersionCollection]
     #
     def self.gems_collection(orig = true)
-      return @gems_orig_collection if orig && !@gems_orig_collection.nil?
-      return @gems_source_collection if !orig && !@gems_source_collection.nil?
+      @gems_collection = {} if @gems_collection.nil?
+
+      is_orig = orig ? 1 : 0
+      data = @gems_collection[is_orig]
+      data = { files: {}, values: nil } if data.nil?
+
+      file_paths = specs_files_paths(orig)
+      has_file_changed = false
+      file_paths.pmap do |file_path|
+        next if data[:files].key?(file_path) &&
+                data[:files][file_path] == File.mtime(file_path)
+        has_file_changed = true
+      end
+
+      # Return result if no file changed
+      return @gems_collection[is_orig][:values] unless has_file_changed
 
       gems = []
-      specs_files_paths(orig).pmap do |specs_file_path|
-        next unless File.exist?(specs_file_path)
-        spec_gems = cache.cache(File.basename(specs_file_path)) do
-          Marshal.load(Zlib::GzipReader.open(specs_file_path).read)
-        end
-        gems.concat(spec_gems)
+      file_paths.pmap do |file_path|
+        next unless File.exist?(file_path)
+        gems.concat(Marshal.load(Zlib::GzipReader.open(file_path).read))
+        data[:files][file_path] = File.mtime(file_path)
       end
 
       collection = GemVersionCollection.new(gems)
-      @gems_source_collection = collection unless orig
-      @gems_orig_collection = collection if orig
+      data[:values] = collection
 
       collection
     end
@@ -67,8 +77,7 @@ module Gemirro
         File.join(configuration.destination,
                   [specs_file_type,
                    marshal_version,
-                   'gz' + (orig ? '.orig' : '')
-                  ].join('.'))
+                   'gz' + (orig ? '.orig' : '')].join('.'))
       end
     end
 
@@ -108,8 +117,8 @@ module Gemirro
     # @return [Gemirro::GemsFetcher]
     #
     def self.gems_fetcher
-      @gems_fetcher ||= Gemirro::GemsFetcher.new(
-        configuration.source, versions_fetcher)
+      @gems_fetcher ||= Gemirro::GemsFetcher
+                        .new(configuration.source, versions_fetcher)
     end
 
     ##
@@ -119,6 +128,7 @@ module Gemirro
     # @return [Gem]
     #
     def self.stored_gem(gem_name, gem_version, platform = 'ruby')
+      platform = 'ruby' if platform.nil?
       @stored_gems ||= {}
       # rubocop:disable Metrics/LineLength
       @stored_gems[gem_name] = {} unless @stored_gems.key?(gem_name)

data/lib/gemirro/version.rb

@@ -1,5 +1,5 @@
 # -*- coding: utf-8 -*-
 # Gemirro Version
 module Gemirro
-  VERSION = '0.13.3'
+  VERSION = '0.13.4'.freeze
 end

data/spec/gemirro/gems_fetcher_spec.rb

@@ -64,7 +64,7 @@ module Gemirro
       gem = Gem.new('gemirro')
       version = ::Gem::Version.new('0.0.1')
       allow(@source).to receive(:fetch_gem)
-        .with('gemirro', version).and_return('gemirro')
+        .with('gemirro-0.0.1.gem').and_return('gemirro')
 
       expect(@fetcher.fetch_gem(gem, version)).to eq('gemirro')
     end
@@ -76,7 +76,7 @@ module Gemirro
       gem = Gem.new('gemirro', :latest)
       version = ::Gem::Version.new('0.0.1')
       allow(@source).to receive(:fetch_gem)
-        .with('gemirro', version).and_return('gemirro')
+        .with('gemirro-0.0.1.gem').and_return('gemirro')
 
       expect(@fetcher.fetch_gem(gem, version)).to eq('gemirro')
     end
@@ -89,7 +89,7 @@ module Gemirro
       gem.gemspec = true
       version = ::Gem::Version.new('0.0.1')
       allow(@source).to receive(:fetch_gemspec)
-        .once.with('gemirro', version).and_return('gemirro')
+        .once.with('gemirro-0.0.1.gemspec.rz').and_return('gemirro')
 
       expect(@fetcher.fetch_gemspec(gem, version)).to eq('gemirro')
     end
@@ -102,7 +102,7 @@ module Gemirro
       gem.gemspec = true
       version = ::Gem::Version.new('0.0.1')
       allow(@source).to receive(:fetch_gemspec)
-        .once.with('gemirro', version).and_return('gemirro')
+        .once.with('gemirro-0.0.1.gemspec.rz').and_return('gemirro')
 
       expect(@fetcher.fetch_gemspec(gem, version)).to eq('gemirro')
     end

data/spec/gemirro/indexer_spec.rb

@@ -126,8 +126,10 @@ module Gemirro
                      'gems/gemirro-0.0.2.gem',
                      'gems/gemirro-0.0.3.gem',
                      'gems/gemirral-0.0.1.gem'])
-      allow(indexer).to receive(:build_marshal_gemspecs).once.and_return([
-        "#{indexer.directory}/quick/gemirro-0.0.1.gemspec.rz"])
+
+      allow(indexer).to receive(:build_marshal_gemspecs).once
+        .and_return(["#{indexer.directory}/quick/gemirro-0.0.1.gemspec.rz"])
+
       allow(indexer).to receive(:compress_indicies).once.and_return(true)
       allow(indexer).to receive(:compress_indices).once.and_return(true)
 
@@ -172,8 +174,9 @@ module Gemirro
       allow(indexer).to receive(:compress_indicies)
       allow(indexer).to receive(:compress_indices)
       allow(indexer).to receive(:build_zlib_file)
-      allow(indexer).to receive(:build_marshal_gemspecs).once.and_return([
-        "#{indexer.directory}/quick/gemirro-0.0.1.gemspec.rz"])
+      # rubocop:disable Metrics/LineLength
+      allow(indexer).to receive(:build_marshal_gemspecs).once.and_return(["#{indexer.directory}/quick/gemirro-0.0.1.gemspec.rz"])
+      # rubocop:enable Metrics/LineLength
 
       allow(Marshal).to receive(:load).and_return(['content'])
       allow(Marshal).to receive(:dump).and_return(['content'])

data/spec/gemirro/server_spec.rb

@@ -71,38 +71,40 @@ module Gemirro
         Struct.new('SuccessGzipReader', :read)
         gzip_reader = Struct::SuccessGzipReader.new(marshal_dump)
         MirrorDirectory.new('/var/www/gemirro')
-          .add_directory('quick/Marshal.4.8')
+                       .add_directory('quick/Marshal.4.8')
+        # rubocop:disable Metrics/LineLength
         MirrorFile.new('/var/www/gemirro/quick/Marshal.4.8/' \
                        'volay-0.1.0.gemspec.rz')
-          .write("x\x9C\x8D\x94]\x8F\xD2@\x14\x86\x89Y\xBB\xB4|\xEC\x12\xD7h" \
-                 "\xD4h\xD3K\x13J\x01\x97\xC84n\x9A\xA8\xBBi\xE2\xC5\x06\xBB" \
-                 "{\xC3\x85)\xE5\x00\x13f:u:E\xD1\xC4\xDF\xE6\xB5\xBF\xCAiK" \
-                 "\x11\xE3GK\xEF\x98\xF7\xBC\xCFy\xCF\xC9\xCCQ=A\x0F\xAE\x80" \
-                 "\"\xF4>\x82\x00/p\xE0\v\xCC\xC2;\xC1\xDD\xA3\xFA\xF4\xA1k4" \
-                 "\x06\xA6e\xF6_(Hy\xEBa\xD55\xB4\r#\xFEV\xB1k\xDE\r\xEAdu" \
-                 "\xB7\xC0cY1U\xE4\xA1\x95\x8A\xD3C7A\xAA\x87)\xB4\x9C\x1FO" \
-                 "\xBE\xD7\xE4OA\xEA\x17\x16\x82k\xD4o\xBC\xD7\x99\xC2x\xEC" \
-                 "\xAD@\xBFe$\xA1\xA0\xC7\xDBX\x00\xD5\x05/\xBC\xEFg\xDE\x13" \
-                 "\xF8\x98`\x0E\x14B1U\xE4w\xEC\x1A\xC7\x17\xAF2\x85\xADd\xC4" \
-                 "\xBE96\x87\xF9\x1F\xEA\xDF%\x8A\x95\xE3T\x9E\xCC2\xF3i\x9B" \
-                 "\xA1\xB3\xCC\xFE\rD\x10\xCE!\f\xB6\x1A\xD2\x9C\xD0\xA7\xB2" \
-                 "\xBF\x13\x8A?\x13<\xEB\x06\x04\xA7b\xD4q\xF8\xAF&\x0E!\xDF" \
-                 ".~\xEF\xE3\xDC\xCC@\xD2Hl\#@M\x9E\x84BN\x00\x9D:\x11\a\x0E" \
-                 "\x04\xFC\x18.\xD1#g\x93\xCF\xEB\xC3\x81m\\\xC1\x97\xD9" \
-                 "\x9Af7\\\xE3l\xD7_\xBC\x02BX\"\xD23\xBB\xF9o\x83A\xB1\x12" \
-                 "\xBBe\xB7\xED\x93K\xFB\xB4\x82\xB6\x80\xA9K\xB1\x1E\x96" \
-                 "\x10\xEA\x03sP\xCD\xBFP\x16\xEE\x8D\x85\xBF\x86E\\\x96" \
-                 "\xC02G\xF9\b\xEC\x16:\x9D\xC3\x06\b\x8B\xD2\xA9\x95\x84" \
-                 "\xD9\x97\xED\xC3p\x89+\x81\xA9}\xAB`\xD9\x9D\xFF\x03\xF6" \
-                 "\xD2\xC2\xBF\xCD\xFD`\xDD\x15\x10\x97\xED\xA4.[\xAB\xC6(" \
-                 "\x94\x05B\xE3\xB1\xBC\xA5e\xF6\xC3\xAA\x11\n\xE5>A\x8CiD " \
-                 "`\x9B\xF2\x04\xE3\xCA\t\xC6\x87\by-f,`Q\xD9\x1E,sp^q\x0F" \
-                 "\x85\xD4r\x8Dg\x11\x06\xCE\xC1\xE4>\x9D\xF9\xC9\xFC\xE5" \
-                 "\xC8YR\x1F\x133`4\xBB\xF9R~\xEF:\x93\xE8\x93\\\x92\xBF\r" \
-                 "\xA3\t\xF8\x84l\xF5<\xBF\xBE\xF9\xE3Q\xD2?q,\x04\x84:\x0E" \
-                 "\xF5\xF4\x1D1\xF3\xBA\xE7+!\"\xD4\xEB-\xB1X%\xB3\x14\xD3" \
-                 "\xCB\xEDw\xEE\xBD\xFDk\xE99OSz\xF3\xEA\xFA]w7\xF5\xAF\xB5" \
-                 "\x9F+\xFEG\x96")
+                  .write("x\x9C\x8D\x94]\x8F\xD2@\x14\x86\x89Y\xBB\xB4|\xEC\x12\xD7h" \
+                         "\xD4h\xD3K\x13J\x01\x97\xC84n\x9A\xA8\xBBi\xE2\xC5\x06\xBB" \
+                         "{\xC3\x85)\xE5\x00\x13f:u:E\xD1\xC4\xDF\xE6\xB5\xBF\xCAiK" \
+                         "\x11\xE3GK\xEF\x98\xF7\xBC\xCFy\xCF\xC9\xCCQ=A\x0F\xAE\x80" \
+                         "\"\xF4>\x82\x00/p\xE0\v\xCC\xC2;\xC1\xDD\xA3\xFA\xF4\xA1k4" \
+                         "\x06\xA6e\xF6_(Hy\xEBa\xD55\xB4\r#\xFEV\xB1k\xDE\r\xEAdu" \
+                         "\xB7\xC0cY1U\xE4\xA1\x95\x8A\xD3C7A\xAA\x87)\xB4\x9C\x1FO" \
+                         "\xBE\xD7\xE4OA\xEA\x17\x16\x82k\xD4o\xBC\xD7\x99\xC2x\xEC" \
+                         "\xAD@\xBFe$\xA1\xA0\xC7\xDBX\x00\xD5\x05/\xBC\xEFg\xDE\x13" \
+                         "\xF8\x98`\x0E\x14B1U\xE4w\xEC\x1A\xC7\x17\xAF2\x85\xADd\xC4" \
+                         "\xBE96\x87\xF9\x1F\xEA\xDF%\x8A\x95\xE3T\x9E\xCC2\xF3i\x9B" \
+                         "\xA1\xB3\xCC\xFE\rD\x10\xCE!\f\xB6\x1A\xD2\x9C\xD0\xA7\xB2" \
+                         "\xBF\x13\x8A?\x13<\xEB\x06\x04\xA7b\xD4q\xF8\xAF&\x0E!\xDF" \
+                         ".~\xEF\xE3\xDC\xCC@\xD2Hl\#@M\x9E\x84BN\x00\x9D:\x11\a\x0E" \
+                         "\x04\xFC\x18.\xD1#g\x93\xCF\xEB\xC3\x81m\\\xC1\x97\xD9" \
+                         "\x9Af7\\\xE3l\xD7_\xBC\x02BX\"\xD23\xBB\xF9o\x83A\xB1\x12" \
+                         "\xBBe\xB7\xED\x93K\xFB\xB4\x82\xB6\x80\xA9K\xB1\x1E\x96" \
+                         "\x10\xEA\x03sP\xCD\xBFP\x16\xEE\x8D\x85\xBF\x86E\\\x96" \
+                         "\xC02G\xF9\b\xEC\x16:\x9D\xC3\x06\b\x8B\xD2\xA9\x95\x84" \
+                         "\xD9\x97\xED\xC3p\x89+\x81\xA9}\xAB`\xD9\x9D\xFF\x03\xF6" \
+                         "\xD2\xC2\xBF\xCD\xFD`\xDD\x15\x10\x97\xED\xA4.[\xAB\xC6(" \
+                         "\x94\x05B\xE3\xB1\xBC\xA5e\xF6\xC3\xAA\x11\n\xE5>A\x8CiD " \
+                         "`\x9B\xF2\x04\xE3\xCA\t\xC6\x87\by-f,`Q\xD9\x1E,sp^q\x0F" \
+                         "\x85\xD4r\x8Dg\x11\x06\xCE\xC1\xE4>\x9D\xF9\xC9\xFC\xE5" \
+                         "\xC8YR\x1F\x133`4\xBB\xF9R~\xEF:\x93\xE8\x93\\\x92\xBF\r" \
+                         "\xA3\t\xF8\x84l\xF5<\xBF\xBE\xF9\xE3Q\xD2?q,\x04\x84:\x0E" \
+                         "\xF5\xF4\x1D1\xF3\xBA\xE7+!\"\xD4\xEB-\xB1X%\xB3\x14\xD3" \
+                         "\xCB\xEDw\xEE\xBD\xFDk\xE99OSz\xF3\xEA\xFA]w7\xF5\xAF\xB5" \
+                         "\x9F+\xFEG\x96")
+        # rubocop:enable Metrics/LineLength
 
         allow(Zlib::GzipReader).to receive(:open)
           .once
@@ -156,7 +158,7 @@ module Gemirro
         expect(last_response.status).to eq(404)
 
         MirrorFile.new('/var/www/gemirro/gems/gemirro-0.0.1.gem')
-          .write('content')
+                  .write('content')
         get '/gems/gemirro-0.0.1.gem'
         expect(last_response).to be_ok
         expect(last_response.status).to eq(200)
@@ -224,38 +226,40 @@ module Gemirro
 
       it 'should retrieve json when gem was found' do
         MirrorDirectory.new('/var/www/gemirro')
-          .add_directory('quick/Marshal.4.8')
+                       .add_directory('quick/Marshal.4.8')
+        # rubocop:disable Metrics/LineLength
         MirrorFile.new('/var/www/gemirro/quick/Marshal.4.8/' \
                        'volay-0.1.0.gemspec.rz')
-          .write("x\x9C\x8D\x94]\x8F\xD2@\x14\x86\x89Y\xBB\xB4|\xEC\x12\xD7h" \
-                 "\xD4h\xD3K\x13J\x01\x97\xC84n\x9A\xA8\xBBi\xE2\xC5\x06\xBB" \
-                 "{\xC3\x85)\xE5\x00\x13f:u:E\xD1\xC4\xDF\xE6\xB5\xBF\xCAiK" \
-                 "\x11\xE3GK\xEF\x98\xF7\xBC\xCFy\xCF\xC9\xCCQ=A\x0F\xAE\x80" \
-                 "\"\xF4>\x82\x00/p\xE0\v\xCC\xC2;\xC1\xDD\xA3\xFA\xF4\xA1k4" \
-                 "\x06\xA6e\xF6_(Hy\xEBa\xD55\xB4\r#\xFEV\xB1k\xDE\r\xEAdu" \
-                 "\xB7\xC0cY1U\xE4\xA1\x95\x8A\xD3C7A\xAA\x87)\xB4\x9C\x1FO" \
-                 "\xBE\xD7\xE4OA\xEA\x17\x16\x82k\xD4o\xBC\xD7\x99\xC2x\xEC" \
-                 "\xAD@\xBFe$\xA1\xA0\xC7\xDBX\x00\xD5\x05/\xBC\xEFg\xDE\x13" \
-                 "\xF8\x98`\x0E\x14B1U\xE4w\xEC\x1A\xC7\x17\xAF2\x85\xADd\xC4" \
-                 "\xBE96\x87\xF9\x1F\xEA\xDF%\x8A\x95\xE3T\x9E\xCC2\xF3i\x9B" \
-                 "\xA1\xB3\xCC\xFE\rD\x10\xCE!\f\xB6\x1A\xD2\x9C\xD0\xA7\xB2" \
-                 "\xBF\x13\x8A?\x13<\xEB\x06\x04\xA7b\xD4q\xF8\xAF&\x0E!\xDF" \
-                 ".~\xEF\xE3\xDC\xCC@\xD2Hl\#@M\x9E\x84BN\x00\x9D:\x11\a\x0E" \
-                 "\x04\xFC\x18.\xD1#g\x93\xCF\xEB\xC3\x81m\\\xC1\x97\xD9" \
-                 "\x9Af7\\\xE3l\xD7_\xBC\x02BX\"\xD23\xBB\xF9o\x83A\xB1\x12" \
-                 "\xBBe\xB7\xED\x93K\xFB\xB4\x82\xB6\x80\xA9K\xB1\x1E\x96" \
-                 "\x10\xEA\x03sP\xCD\xBFP\x16\xEE\x8D\x85\xBF\x86E\\\x96" \
-                 "\xC02G\xF9\b\xEC\x16:\x9D\xC3\x06\b\x8B\xD2\xA9\x95\x84" \
-                 "\xD9\x97\xED\xC3p\x89+\x81\xA9}\xAB`\xD9\x9D\xFF\x03\xF6" \
-                 "\xD2\xC2\xBF\xCD\xFD`\xDD\x15\x10\x97\xED\xA4.[\xAB\xC6(" \
-                 "\x94\x05B\xE3\xB1\xBC\xA5e\xF6\xC3\xAA\x11\n\xE5>A\x8CiD " \
-                 "`\x9B\xF2\x04\xE3\xCA\t\xC6\x87\by-f,`Q\xD9\x1E,sp^q\x0F" \
-                 "\x85\xD4r\x8Dg\x11\x06\xCE\xC1\xE4>\x9D\xF9\xC9\xFC\xE5" \
-                 "\xC8YR\x1F\x133`4\xBB\xF9R~\xEF:\x93\xE8\x93\\\x92\xBF\r" \
-                 "\xA3\t\xF8\x84l\xF5<\xBF\xBE\xF9\xE3Q\xD2?q,\x04\x84:\x0E" \
-                 "\xF5\xF4\x1D1\xF3\xBA\xE7+!\"\xD4\xEB-\xB1X%\xB3\x14\xD3" \
-                 "\xCB\xEDw\xEE\xBD\xFDk\xE99OSz\xF3\xEA\xFA]w7\xF5\xAF\xB5" \
-                 "\x9F+\xFEG\x96")
+                  .write("x\x9C\x8D\x94]\x8F\xD2@\x14\x86\x89Y\xBB\xB4|\xEC\x12\xD7h" \
+                         "\xD4h\xD3K\x13J\x01\x97\xC84n\x9A\xA8\xBBi\xE2\xC5\x06\xBB" \
+                         "{\xC3\x85)\xE5\x00\x13f:u:E\xD1\xC4\xDF\xE6\xB5\xBF\xCAiK" \
+                         "\x11\xE3GK\xEF\x98\xF7\xBC\xCFy\xCF\xC9\xCCQ=A\x0F\xAE\x80" \
+                         "\"\xF4>\x82\x00/p\xE0\v\xCC\xC2;\xC1\xDD\xA3\xFA\xF4\xA1k4" \
+                         "\x06\xA6e\xF6_(Hy\xEBa\xD55\xB4\r#\xFEV\xB1k\xDE\r\xEAdu" \
+                         "\xB7\xC0cY1U\xE4\xA1\x95\x8A\xD3C7A\xAA\x87)\xB4\x9C\x1FO" \
+                         "\xBE\xD7\xE4OA\xEA\x17\x16\x82k\xD4o\xBC\xD7\x99\xC2x\xEC" \
+                         "\xAD@\xBFe$\xA1\xA0\xC7\xDBX\x00\xD5\x05/\xBC\xEFg\xDE\x13" \
+                         "\xF8\x98`\x0E\x14B1U\xE4w\xEC\x1A\xC7\x17\xAF2\x85\xADd\xC4" \
+                         "\xBE96\x87\xF9\x1F\xEA\xDF%\x8A\x95\xE3T\x9E\xCC2\xF3i\x9B" \
+                         "\xA1\xB3\xCC\xFE\rD\x10\xCE!\f\xB6\x1A\xD2\x9C\xD0\xA7\xB2" \
+                         "\xBF\x13\x8A?\x13<\xEB\x06\x04\xA7b\xD4q\xF8\xAF&\x0E!\xDF" \
+                         ".~\xEF\xE3\xDC\xCC@\xD2Hl\#@M\x9E\x84BN\x00\x9D:\x11\a\x0E" \
+                         "\x04\xFC\x18.\xD1#g\x93\xCF\xEB\xC3\x81m\\\xC1\x97\xD9" \
+                         "\x9Af7\\\xE3l\xD7_\xBC\x02BX\"\xD23\xBB\xF9o\x83A\xB1\x12" \
+                         "\xBBe\xB7\xED\x93K\xFB\xB4\x82\xB6\x80\xA9K\xB1\x1E\x96" \
+                         "\x10\xEA\x03sP\xCD\xBFP\x16\xEE\x8D\x85\xBF\x86E\\\x96" \
+                         "\xC02G\xF9\b\xEC\x16:\x9D\xC3\x06\b\x8B\xD2\xA9\x95\x84" \
+                         "\xD9\x97\xED\xC3p\x89+\x81\xA9}\xAB`\xD9\x9D\xFF\x03\xF6" \
+                         "\xD2\xC2\xBF\xCD\xFD`\xDD\x15\x10\x97\xED\xA4.[\xAB\xC6(" \
+                         "\x94\x05B\xE3\xB1\xBC\xA5e\xF6\xC3\xAA\x11\n\xE5>A\x8CiD " \
+                         "`\x9B\xF2\x04\xE3\xCA\t\xC6\x87\by-f,`Q\xD9\x1E,sp^q\x0F" \
+                         "\x85\xD4r\x8Dg\x11\x06\xCE\xC1\xE4>\x9D\xF9\xC9\xFC\xE5" \
+                         "\xC8YR\x1F\x133`4\xBB\xF9R~\xEF:\x93\xE8\x93\\\x92\xBF\r" \
+                         "\xA3\t\xF8\x84l\xF5<\xBF\xBE\xF9\xE3Q\xD2?q,\x04\x84:\x0E" \
+                         "\xF5\xF4\x1D1\xF3\xBA\xE7+!\"\xD4\xEB-\xB1X%\xB3\x14\xD3" \
+                         "\xCB\xEDw\xEE\xBD\xFDk\xE99OSz\xF3\xEA\xFA]w7\xF5\xAF\xB5" \
+                         "\x9F+\xFEG\x96")
+        # rubocop:enable Metrics/LineLength
 
         gem = Gemirro::GemVersion.new('volay', '0.1.0', 'ruby')
         collection = Gemirro::GemVersionCollection.new([gem])

data/spec/gemirro/source_spec.rb

@@ -39,17 +39,17 @@ module Gemirro
     it 'should fetch gem' do
       Struct.new('FetchGem', :body)
       result = Struct::FetchGem.new(true)
-      allow(Http).to receive(:get).once.with(
-        'https://rubygems.org/gems/gemirro-0.0.1.gem').and_return(result)
-      expect(@source.fetch_gem('gemirro', '0.0.1')).to be_truthy
+      allow(Http).to receive(:get).once
+        .with('https://rubygems.org/gems/gemirro-0.0.1.gem').and_return(result)
+      expect(@source.fetch_gem('gemirro-0.0.1.gem')).to be_truthy
     end
 
     it 'should fetch gemspec' do
       Struct.new('FetchGemspec', :body)
       result = Struct::FetchGemspec.new(true)
-      allow(Http).to receive(:get).once.with(
-        'https://rubygems.org/quick/Marshal.4.8/gemirro-0.0.1.gemspec.rz').and_return(result)
-      expect(@source.fetch_gemspec('gemirro', '0.0.1')).to be_truthy
+      allow(Http).to receive(:get).once
+        .with('https://rubygems.org/quick/Marshal.4.8/gemirro-0.0.1.gemspec.rz').and_return(result)
+      expect(@source.fetch_gemspec('gemirro-0.0.1.gemspec.rz')).to be_truthy
     end
 
     it 'should add gems' do

data/spec/gemirro/versions_file_spec.rb

@@ -9,14 +9,10 @@ module Gemirro
     include FakeFS::SpecHelpers
 
     it 'should be initialized' do
-      @versions_file = VersionsFile.new([
-        ['gemirro', '0.0.1'],
-        ['gemirro', '0.0.2']
-      ])
-      expect(@versions_file.versions).to eq([
-        ['gemirro', '0.0.1'],
-        ['gemirro', '0.0.2']
-      ])
+      @versions_file = VersionsFile.new([['gemirro', '0.0.1'],
+                                         ['gemirro', '0.0.2']])
+      expect(@versions_file.versions).to eq([['gemirro', '0.0.1'],
+                                             ['gemirro', '0.0.2']])
       expect(@versions_file.versions_hash).to eq(
         'gemirro' => [
           ['gemirro', '0.0.1'],
@@ -28,28 +24,22 @@ module Gemirro
     it 'should load versions file' do
       spec = StringIO.new('w')
       w_gz = Zlib::GzipWriter.new(spec)
-      w_gz.write(Marshal.dump([
-        ['gemirro', '0.0.1'],
-        ['gemirro', '0.0.2']
-      ]))
+      w_gz.write(Marshal.dump([['gemirro', '0.0.1'],
+                               ['gemirro', '0.0.2']]))
       w_gz.close
       prerelease = StringIO.new('w')
       w_gz = Zlib::GzipWriter.new(prerelease)
-      w_gz.write(Marshal.dump([
-        ['gemirro', '0.0.1.alpha1'],
-        ['gemirro', '0.0.2.alpha2']
-      ]))
+      w_gz.write(Marshal.dump([['gemirro', '0.0.1.alpha1'],
+                               ['gemirro', '0.0.2.alpha2']]))
       w_gz.close
 
       result = VersionsFile.load(spec.string, prerelease.string)
       expect(result).to be_a(VersionsFile)
 
-      expect(result.versions).to eq([
-        ['gemirro', '0.0.1'],
-        ['gemirro', '0.0.2'],
-        ['gemirro', '0.0.1.alpha1'],
-        ['gemirro', '0.0.2.alpha2']
-      ])
+      expect(result.versions).to eq([['gemirro', '0.0.1'],
+                                     ['gemirro', '0.0.2'],
+                                     ['gemirro', '0.0.1.alpha1'],
+                                     ['gemirro', '0.0.2.alpha2']])
       expect(result.versions_hash).to eq(
         'gemirro' => [
           ['gemirro', '0.0.1'],

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gemirro
 version: !ruby/object:Gem::Version
-  version: 0.13.3
+  version: 0.13.4
 platform: ruby
 authors:
 - Pierre Rambaud
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-07 00:00:00.000000000 Z
+date: 2016-10-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: slop
@@ -289,9 +289,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.0
+rubygems_version: 2.6.7
 signing_key: 
 specification_version: 4
 summary: Gem for easily creating your own gems mirror.
 test_files: []
-has_rdoc: