[ARVADOS] updated: 4c4387de5857ef00899fcb760a7cfc5ad4b76268

Git user git at public.curoverse.com
Fri May 13 11:26:07 EDT 2016


Summary of changes:
 sdk/cli/arvados-cli.gemspec |  4 +++-
 sdk/cli/bin/arv             | 29 +++++++++++++++++++----------
 sdk/ruby/arvados.gemspec    |  4 +++-
 3 files changed, 25 insertions(+), 12 deletions(-)

       via  4c4387de5857ef00899fcb760a7cfc5ad4b76268 (commit)
       via  e0d2a1aa4e13da85575cef7ea4136e69b5423ba8 (commit)
       via  324d0d6d115488d0ac7fea7aa1a6cff3dd16be0e (commit)
       via  d2dad24530a065808487a3006ab20a98a67d8fa5 (commit)
      from  7f7ed63314bec77bf93edc233514c50a1030cac4 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 4c4387de5857ef00899fcb760a7cfc5ad4b76268
Merge: 7f7ed63 e0d2a1a
Author: Brett Smith <brett at curoverse.com>
Date:   Fri May 13 11:25:36 2016 -0400

    Merge branch '9213-fix-arv-gems-wip'
    
    Closes #9213, #9215.


commit e0d2a1aa4e13da85575cef7ea4136e69b5423ba8
Author: Brett Smith <brett at curoverse.com>
Date:   Thu May 12 16:48:43 2016 -0400

    9213: Update arv's `gem install` suggestions.
    
    This makes it match what it actually loads.

diff --git a/sdk/cli/bin/arv b/sdk/cli/bin/arv
index 301cadc..b377ffe 100755
--- a/sdk/cli/bin/arv
+++ b/sdk/cli/bin/arv
@@ -40,7 +40,7 @@ Error loading gems: #{error}
 
 Please install all required gems:
 
-  gem install activesupport andand curb google-api-client json oj trollop yaml
+  gem install arvados activesupport andand curb json oj trollop
 
   EOS
 end

commit 324d0d6d115488d0ac7fea7aa1a6cff3dd16be0e
Author: Brett Smith <brett at curoverse.com>
Date:   Thu May 12 16:40:37 2016 -0400

    9213: Improve gem loading in `arv`.
    
    * Include the exception string in the error message.
    * Separate stdlib loading problems from gem loading problems.
    * Load gems with more dependencies first, to avoid situations like
      this:
    
    irb(main):001:0> require 'active_support/inflector'
    => true
    irb(main):002:0> require 'arvados/google_api_client'
    Gem::LoadError: Unable to activate arvados-0.1.20160420143004, because activesupport-4.2.6 conflicts with activesupport (< 4.2.6, >= 3)

diff --git a/sdk/cli/bin/arv b/sdk/cli/bin/arv
index 541bcdc..301cadc 100755
--- a/sdk/cli/bin/arv
+++ b/sdk/cli/bin/arv
@@ -14,21 +14,30 @@ if RUBY_VERSION < '1.9.3' then
 end
 
 begin
-  require 'curb'
-  require 'rubygems'
-  require 'arvados/google_api_client'
   require 'json'
+  require 'net/http'
   require 'pp'
-  require 'trollop'
+  require 'tempfile'
+  require 'yaml'
+rescue LoadError => error
+  abort "Error loading libraries: #{error}\n"
+end
+
+begin
+  require 'rubygems'
+  # Load the gems with more requirements first, so we respect any version
+  # constraints they put on gems loaded later.
+  require 'arvados/google_api_client'
+  require 'active_support/inflector'
   require 'andand'
+  require 'curb'
   require 'oj'
-  require 'active_support/inflector'
-  require 'yaml'
-  require 'tempfile'
-  require 'net/http'
-rescue LoadError
+  require 'trollop'
+rescue LoadError => error
   abort <<-EOS
 
+Error loading gems: #{error}
+
 Please install all required gems:
 
   gem install activesupport andand curb google-api-client json oj trollop yaml

commit d2dad24530a065808487a3006ab20a98a67d8fa5
Author: Brett Smith <brett at curoverse.com>
Date:   Thu May 12 16:37:59 2016 -0400

    9213: Fix google-api-client dependency range in gemspecs.

diff --git a/sdk/cli/arvados-cli.gemspec b/sdk/cli/arvados-cli.gemspec
index 7755a5a..0e11f96 100644
--- a/sdk/cli/arvados-cli.gemspec
+++ b/sdk/cli/arvados-cli.gemspec
@@ -25,7 +25,9 @@ Gem::Specification.new do |s|
   s.executables << "arv-tag"
   s.required_ruby_version = '>= 2.1.0'
   s.add_runtime_dependency 'arvados', '~> 0.1', '>= 0.1.20150128223554'
-  s.add_runtime_dependency 'google-api-client', '~> 0.6', '>= 0.6.3', '<0.9'
+  # Our google-api-client dependency used to be < 0.9, but that could be
+  # satisfied by the buggy 0.9.pre*.  https://dev.arvados.org/issues/9213
+  s.add_runtime_dependency 'google-api-client', '~> 0.6', '>= 0.6.3', '<0.8.9'
   s.add_runtime_dependency 'activesupport', '~> 3.2', '>= 3.2.13'
   s.add_runtime_dependency 'json', '~> 1.7', '>= 1.7.7'
   s.add_runtime_dependency 'trollop', '~> 2.0'
diff --git a/sdk/ruby/arvados.gemspec b/sdk/ruby/arvados.gemspec
index 355ed5d..2c4e60e 100644
--- a/sdk/ruby/arvados.gemspec
+++ b/sdk/ruby/arvados.gemspec
@@ -22,7 +22,9 @@ Gem::Specification.new do |s|
   # activesupport <4.2.6 only because https://dev.arvados.org/issues/8222
   s.add_dependency('activesupport', '>= 3', '< 4.2.6')
   s.add_dependency('andand', '~> 1.3', '>= 1.3.3')
-  s.add_dependency('google-api-client', '>= 0.7', '< 0.9')
+  # Our google-api-client dependency used to be < 0.9, but that could be
+  # satisfied by the buggy 0.9.pre*.  https://dev.arvados.org/issues/9213
+  s.add_dependency('google-api-client', '>= 0.7', '< 0.8.9')
   # work around undeclared dependency on i18n in some activesupport 3.x.x:
   s.add_dependency('i18n', '~> 0')
   s.add_dependency('json', '~> 1.7', '>= 1.7.7')

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list