On 09/21/11 - 05:05:45PM, Jan Provaznik wrote:
On 09/21/2011 05:00 PM, Jan Provaznik wrote:
> On 09/21/2011 02:25 PM, Chris Lalancette wrote:
>> As far as I can tell (and from what jayg has told me),
>> this is no longer necessary.
>>
>> Signed-off-by: Chris Lalancette<clalance(a)redhat.com>
>> ---
>> aeolus-conductor.spec.in | 1 -
>> src/Gemfile | 1 -
>> src/Gemfile.lock | 2 --
>> src/config/application.rb | 1 -
>> 4 files changed, 0 insertions(+), 5 deletions(-)
>>
>> diff --git a/aeolus-conductor.spec.in b/aeolus-conductor.spec.in
>> index cb5e4d8..4ec3445 100644
>> --- a/aeolus-conductor.spec.in
>> +++ b/aeolus-conductor.spec.in
>> @@ -26,7 +26,6 @@ Requires: rubygem(will_paginate)>= 3.0
>> Requires: rubygem(parseconfig)
>> Requires: rubygem(deltacloud-client)>= 0.4.0
>> Requires: rubygem(simple-navigation)
>> -Requires: rubygem(typhoeus)
>> Requires: rubygem(rest-client)>= 1.6.1
>> Requires: rubygem(builder)
>> Requires: rubygem(json)
>> diff --git a/src/Gemfile b/src/Gemfile
>> index d063047..67295bd 100644
>> --- a/src/Gemfile
>> +++ b/src/Gemfile
>> @@ -10,7 +10,6 @@ gem 'haml'
>> gem 'will_paginate', '>= 3.0.pre1'
>> gem 'nokogiri'
>> gem 'simple-navigation'
>> -gem 'typhoeus'
>> gem 'rest-client'
>> gem 'rack-restful_submit'
>> gem 'uuidtools'
>> diff --git a/src/Gemfile.lock b/src/Gemfile.lock
>> index 0f3912b..5a55b25 100644
>> --- a/src/Gemfile.lock
>> +++ b/src/Gemfile.lock
>> @@ -138,7 +138,6 @@ GEM
>> timecop (0.3.5)
>> treetop (1.4.9)
>> polyglot (>= 0.2.5)
>> - typhoeus (0.2.0)
>> tzinfo (0.3.26)
>> uuidtools (2.1.1)
>> vcr (1.10.0)
>> @@ -179,7 +178,6 @@ DEPENDENCIES
>> sqlite3
>> thin
>> timecop
>> - typhoeus
>> uuidtools
>> vcr
>> webmock
>> diff --git a/src/config/application.rb b/src/config/application.rb
>> index 41d12e9..ded491d 100644
>> --- a/src/config/application.rb
>> +++ b/src/config/application.rb
>> @@ -17,7 +17,6 @@ else
>> require 'will_paginate'
>> require 'nokogiri'
>> require 'simple-navigation'
>> - require 'typhoeus'
>> require 'rest-client'
>> require 'rack-restful_submit'
>> require 'uuidtools'
>
> ACK
>
> All works for me with all 3 dep-removal patches applied (tests, rpm
> build, app itself), also didn't find any occurance of usage of these gems.
>
actually one test is failing, but it's probably not related to this
patch as it's failing for me even on master branch.
Oh, it is probably the condormatic test. I saw that earlier and forgot about
it when I pushed the "remove condor" patch. We'll have to fix that.
--
Chris Lalancette