gitlab_git 7.0.0.rc3 → 7.0.0.rc4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ff461d9ee572c8e82ed303b222d4719bce93dcc5
-  data.tar.gz: 16c2b9ada62bb3452b237ef921bfc004ac2aaaa6
+  metadata.gz: ac5cbe56279c663c2bbff8af444c61da6f004056
+  data.tar.gz: ab6cb0ba7984203a57ca8112e3538a2525237fc8
 SHA512:
-  metadata.gz: a2cf2dceb0d689ee5489724ffb920cc61605de1728ba00fb9168e136a9f262c8d75cdcedc19a4605e420b3b2ba0cce8bd643d46424a05ad77926fcd3e63bdcdd
-  data.tar.gz: c55192100961a6311fc981b240fec387bc87faeefb1ec637f608c0e9008e67bbe32055f694b2fe325a23f2e361637e1fe3d7d8482008738b9c17c2c94cb69ed1
+  metadata.gz: 9595dccf43da4638106dd90aead029a5d33b1a51684ae83cab7591a4b9a0943d4f6ab4412e613c1c06e7d959dfefad323414ea48a66ded7c84d83ffe75b02450
+  data.tar.gz: 17a0b75f441a6c10a26a3ce05049613b250ef0adf7d1682f84dfd4845fa43808a979360047e863edd759cb8d0abd99a57a0d01a83e5e01596d446154de5a71f3

data/VERSION

@@ -1 +1 @@
-7.0.0.rc3
+7.0.0.rc4

data/lib/gitlab_git/repository.rb

@@ -326,7 +326,7 @@ module Gitlab
       #   repo.branch_names_contains('master')
       #
       def branch_names_contains(commit)
-        branches_contains(commit).map { |c| c.target_id }
+        branches_contains(commit).map { |c| c.name }
       end
 
       # Returns branch collection that contains the special commit(SHA1 or name)
@@ -813,33 +813,33 @@ module Gitlab
         prefix = File.basename(name)
         extension = Pathname.new(file_path).extname
 
-        if extension == ".zip"
+        if extension == '.zip'
           create_zip_archive(ref_name, file_path, prefix)
         else
-          # Create a tarfile in memory
-          tarfile = tar_string_io(ref_name, prefix)
-
-          if extension == ".tar"
-            File.new(file_path, "wb").write(tarfile.read)
-          else
-            compress_tar(tarfile, file_path, pipe_cmd)
+          rd_pipe, rw_pipe = IO.pipe
+          tar_pid = fork do
+            # Send the tar file to the write pipe
+            rd_pipe.close
+            Gem::Package::TarWriter.new(rw_pipe) do |tar|
+              tar.mkdir(prefix, 33261)
+
+              populated_index(ref_name).each do |entry|
+                add_archive_entry(tar, prefix, entry)
+              end
+            end
+            rw_pipe.close
           end
-        end
-      end
 
-      # Return a StringIO with the contents of the repo's tar file
-      def tar_string_io(ref_name, prefix)
-        tarfile = StringIO.new
-        Gem::Package::TarWriter.new(tarfile) do |tar|
-          tar.mkdir(prefix, 33261)
+          # Use the other end of the pipe to compress with bzip2 or gzip
+          FileUtils.mkdir_p(Pathname.new(file_path).dirname)
+          archive_file = File.new(file_path, 'wb')
+          rw_pipe.close
+          system(*pipe_cmd, in: rd_pipe, out: archive_file)
 
-          populated_index(ref_name).each do |entry|
-            add_archive_entry(tar, prefix, entry)
-          end
+          Process.waitpid(tar_pid)
+          rd_pipe.close
+          archive_file.close
         end
-
-        tarfile.rewind
-        tarfile
       end
 
       # Create a zip file with the contents of the repo
@@ -854,22 +854,26 @@ module Gitlab
       # Add a file or directory from the index to the given tar or zip file
       def add_archive_entry(archive, prefix, entry)
         prefixed_path = File.join(prefix, entry[:path])
-        content = rugged.lookup(entry[:oid]).content unless submodule?(entry)
 
-        # Create a file in the archive for each index entry
-        if archive.is_a?(Zip::File)
-          unless submodule?(entry)
+        if submodule?(entry)
+          # Create an empty directory for submodules
+          mask = case archive
+                 when Zip::File then 0755
+                 else '100755'.to_i(8)
+                 end
+          archive.mkdir(prefixed_path, mask)
+        else
+          blob = rugged.lookup(entry[:oid])
+          content = blob.content
+
+          # Write the blob contents to the archive
+          if archive.is_a?(Zip::File)
             archive.get_output_stream(prefixed_path) do |os|
               os.write(content)
             end
-          end
-        else
-          if submodule?(entry)
-            # Create directories for submodules
-            archive.mkdir(prefixed_path, 33261)
           else
-            # Write the blob contents to the file
-            archive.add_file(prefixed_path, entry[:mode]) do |tf|
+            archive.add_file_simple(prefixed_path,
+                                    entry[:mode], blob.size) do |tf|
               tf.write(content)
             end
           end
@@ -882,31 +886,6 @@ module Gitlab
         index_entry[:mode] == 57344
       end
 
-      # Send the +tar_string+ StringIO to +pipe_cmd+ for bzip2 or gzip
-      # compression.
-      def compress_tar(tar_string, file_path, pipe_cmd)
-        # Write the in-memory tarfile to a pipe
-        rd_pipe, rw_pipe = IO.pipe
-        tar_pid = fork do
-          rd_pipe.close
-          rw_pipe.write(tar_string.read)
-          rw_pipe.close
-        end
-
-        # Use the other end of the pipe to compress with bzip2 or gzip
-        FileUtils.mkdir_p(Pathname.new(file_path).dirname)
-        archive_file = File.new(file_path, "wb")
-        rw_pipe.close
-        compress_pid = spawn(*pipe_cmd, in: rd_pipe, out: archive_file)
-        rd_pipe.close
-
-        Process.waitpid(tar_pid)
-        Process.waitpid(compress_pid)
-
-        archive_file.close
-        tar_string.close
-      end
-
       # Return a Rugged::Index that has read from the tree at +ref_name+
       def populated_index(ref_name)
         tree = rugged.lookup(rugged.rev_parse_oid(ref_name)).tree

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_git
 version: !ruby/object:Gem::Version
-  version: 7.0.0.rc3
+  version: 7.0.0.rc4
 platform: ruby
 authors:
 - Dmitriy Zaporozhets
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-26 00:00:00.000000000 Z
+date: 2014-09-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gitlab-linguist