Actions
Bug #18247
closed[arv] faraday gem warnings on Ruby 2.7+
Start date:
10/05/2021
Due date:
% Done:
100%
Estimated time:
(Total: 0.00 h)
Story points:
-
Release:
Release relationship:
Auto
Description
e.g. on Ubuntu 20.04 (Focal Fossa) with system Ruby packages:
$ arv /var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead /var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead /var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead /var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead /var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead ARVADOS_API_HOST and ARVADOS_API_TOKEN need to be defined as environment variables. $ ruby -v ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-linux-gnu] $ gem list |grep arv arvados (2.2.2) arvados-cli (2.2.2) arvados-google-api-client (0.8.7.4)
Updated by Ward Vandewege over 3 years ago
- Status changed from New to In Progress
Updated by Ward Vandewege over 3 years ago
Hmm. We have faraday 0.15.4 because of the old arvados-google-api-client gem we use, according to a comment in arvados-cli.gemspec:
# arvados-google-api-client 0.8.7.2 is incompatible with faraday 0.16.2 s.add_dependency('faraday', '< 0.16')
This was done in #15673, where we also pushed 0.8.7.3 for arvados-google-api-client. So.... the logical solution then may be to depend on that version of arvados-google-api-client and remove this faraday pin. Maybe. Looking into that...
Updated by Ward Vandewege over 3 years ago
- Related to Bug #15673: [SDKs][Ruby] Update faraday usage added
Updated by Ward Vandewege over 3 years ago
Ready for review in ebf9bc1e9ac39c4c0b4ea36fd3bd5d9411c73a6b on branch 18247-faraday-version
Updated by Lucas Di Pentima over 3 years ago
Running tests at: developer-run-tests: #2724
Updated by Ward Vandewege over 3 years ago
- Status changed from In Progress to Resolved
Updated by Ward Vandewege over 3 years ago
- Related to Bug #18309: [gem] faraday dependency difference between arvados and arvados-login-sync added
Actions