arvados 1.3.1.20190320201707 → 1.3.2.20181129194931

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d4bd9fd87d67c3d5d1dd096ba8418b17c7b0515e
-  data.tar.gz: 393a7a3a34e799a0dc290587714f32dec1e46dd4
+  metadata.gz: 4cb797cbaf0ec49d8eb4e780d31decb08cccb955
+  data.tar.gz: 8e009b72e581fa2023c9e0ec3b91646a45dec5e9
 SHA512:
-  metadata.gz: fb762c3560b7082a0bccdd257a7a8b39eb65aa79fcc7bdaaca88056a9a54b00e16c2b38ab7d348765bba6270cf4724f450aaafe234ddb2a225b4394f459a7094
-  data.tar.gz: 48c3cc63b6cee3542eaf9251cabcf5235d270c72f1e0a685e810facc16a66f69cf3d86be8b8a5d738c47152e78214d8af800be11cf1d8c18bc83970b516d77bc
+  metadata.gz: 47c9ab2ee660d4a3ec31bf82ee227bf4176b0fd12b7a38020f275c2b7c5a4d0bd1cdd09510809e6a90ed627de5eb1638529d236a03de18efa1b093a0cc9c899d
+  data.tar.gz: 3c6ae59e4bfdaa4a2175549b87028c59e19665d378288af696060685c825f4a55c112e7aaa7083a0cfe2226cf4652e965e07c12fb0d2be7518eb8c94a3b64127

data/lib/arvados/collection.rb

@@ -369,11 +369,7 @@ module Arv
       end
 
       def add_copy(src_item, key)
-        if key == "."
-          self[key] = src_item.copy_named("#{path}")
-        else
-          self[key] = src_item.copy_named("#{path}/#{key}")
-        end
+        self[key] = src_item.copy_named("#{path}/#{key}")
       end
 
       def merge(src_item, key)
@@ -461,10 +457,6 @@ module Arv
         items["."] = CollectionStream.new(".")
       end
 
-      def add_copy(src_item, key)
-        items["."].add_copy(src_item, key)
-      end
-
       def raise_root_write_error(key)
         raise ArgumentError.new("can't write to %p at collection root" % key)
       end

data/lib/arvados/keep.rb

@@ -101,14 +101,8 @@ module Keep
   end
 
   class Manifest
-    STREAM_TOKEN_REGEXP = /^([^\000-\040\\]|\\[0-3][0-7][0-7])+$/
-    STREAM_NAME_REGEXP = /^(\.)(\/[^\/]+)*$/
-
-    EMPTY_DIR_TOKEN_REGEXP = /^0:0:\.$/ # The exception when a file can have '.' as a name
-    FILE_TOKEN_REGEXP = /^[[:digit:]]+:[[:digit:]]+:([^\000-\040\\]|\\[0-3][0-7][0-7])+$/
-    FILE_NAME_REGEXP = /^[[:digit:]]+:[[:digit:]]+:([^\/]+(\/[^\/]+)*)$/
-
-    NON_8BIT_ENCODED_CHAR = /[^\\]\\[4-7][0-7][0-7]/
+    STRICT_STREAM_TOKEN_REGEXP = /^(\.)(\/[^\/\s]+)*$/
+    STRICT_FILE_TOKEN_REGEXP = /^[[:digit:]]+:[[:digit:]]+:([^\s\/]+(\/[^\s\/]+)*)$/
 
     # Class to parse a manifest text and provide common views of that data.
     def initialize(manifest_text)
@@ -137,9 +131,7 @@ module Keep
       end
     end
 
-    def self.unescape(s)
-      return nil if s.nil?
-
+    def unescape(s)
       # Parse backslash escapes in a Keep manifest stream or file name.
       s.gsub(/\\(\\|[0-7]{3})/) do |_|
         case $1
@@ -151,10 +143,6 @@ module Keep
       end
     end
 
-    def unescape(s)
-      self.class.unescape(s)
-    end
-
     def split_file_token token
       start_pos, filesize, filename = token.split(':', 3)
       if filename.nil?
@@ -174,15 +162,15 @@ module Keep
           elsif in_file_tokens or not Locator.valid? token
             in_file_tokens = true
 
-            start_pos, file_size, file_name = split_file_token(token)
+            file_tokens = split_file_token(token)
             stream_name_adjuster = ''
-            if file_name.include?('/')                # '/' in filename
-              dirname, sep, basename = file_name.rpartition('/')
-              stream_name_adjuster = sep + dirname   # /dir_parts
-              file_name = basename
+            if file_tokens[2].include?('/')                # '/' in filename
+              parts = file_tokens[2].rpartition('/')
+              stream_name_adjuster = parts[1] + parts[0]   # /dir_parts
+              file_tokens[2] = parts[2]
             end
 
-            yield [stream_name + stream_name_adjuster, start_pos, file_size, file_name]
+            yield [stream_name + stream_name_adjuster] + file_tokens
           end
         end
       end
@@ -209,13 +197,10 @@ module Keep
       # files.  This can help you avoid parsing the entire manifest if you
       # just want to check if a small number of files are specified.
       if stop_after.nil? or not @files.nil?
-        # Avoid counting empty dir placeholders
-        return files.reject{|_, name, size| name == '.' and size == 0}.size
+        return files.size
       end
       seen_files = {}
-      each_file_spec do |streamname, _, filesize, filename|
-        # Avoid counting empty dir placeholders
-        next if filename == "." and filesize == 0
+      each_file_spec do |streamname, _, _, filename|
         seen_files[[streamname, filename]] = true
         return stop_after if (seen_files.size >= stop_after)
       end
@@ -265,9 +250,7 @@ module Keep
         count = 0
 
         word = words.shift
-        raise ArgumentError.new "Manifest invalid for stream #{line_count}: >8-bit encoded chars not allowed on stream token #{word.inspect}" if word =~ NON_8BIT_ENCODED_CHAR
-        unescaped_word = unescape(word)
-        count += 1 if word =~ STREAM_TOKEN_REGEXP and unescaped_word =~ STREAM_NAME_REGEXP and unescaped_word !~ /\/\.\.?(\/|$)/
+        count += 1 if word =~ STRICT_STREAM_TOKEN_REGEXP and word !~ /\/\.\.?(\/|$)/
         raise ArgumentError.new "Manifest invalid for stream #{line_count}: missing or invalid stream name #{word.inspect if word}" if count != 1
 
         count = 0
@@ -279,9 +262,7 @@ module Keep
         raise ArgumentError.new "Manifest invalid for stream #{line_count}: missing or invalid locator #{word.inspect if word}" if count == 0
 
         count = 0
-        raise ArgumentError.new "Manifest invalid for stream #{line_count}: >8-bit encoded chars not allowed on file token #{word.inspect}" if word =~ NON_8BIT_ENCODED_CHAR
-        while unescape(word) =~ EMPTY_DIR_TOKEN_REGEXP or
-          (word =~ FILE_TOKEN_REGEXP and unescape(word) =~ FILE_NAME_REGEXP and ($~[1].split('/') & ['..', '.']).empty?)
+        while word =~ STRICT_FILE_TOKEN_REGEXP and ($~[1].split('/') & ['..','.']).empty?
           word = words.shift
           count += 1
         end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: arvados
 version: !ruby/object:Gem::Version
-  version: 1.3.1.20190320201707
+  version: 1.3.2.20181129194931
 platform: ruby
 authors:
 - Arvados Authors
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-03-20 00:00:00.000000000 Z
+date: 2018-11-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -45,7 +45,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 1.3.3
 - !ruby/object:Gem::Dependency
-  name: arvados-google-api-client
+  name: cure-google-api-client
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -118,7 +118,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.1.5
-description: Arvados client library, git commit a07891c7ac7e90ebdf35ae1812ec03c818fe2a67
+description: Arvados client library, git commit 0d22de74ea069003b49a9d52878d5f1ac04d71bb
 email: gem-dev@curoverse.com
 executables: []
 extensions: []