Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update to require digest/md5 instead of md5 #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

update to require digest/md5 instead of md5 #9

wants to merge 1 commit into from

Conversation

steveroot
Copy link

s3sync didn't work for me giving error of:
home/sroot/.rvm/rubies/ruby-1.9.3-p327/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:36:in require': cannot load such file -- md5 (LoadError)

It appears ruby 1.9 uses digest/md5 instead of md5
I suggest someone more knowledgeable checks this before accepting the pull request.

@chris3000
Copy link

I can confirm that ruby 1.9.3 uses digest/md5.

:~ chris$ irb
1.9.3p194 :001 > require 'md5'
LoadError: cannot load such file -- md5
    from /Users/chris/.rvm/rubies/ruby-1.9.3-p194/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
    from /Users/chris/.rvm/rubies/ruby-1.9.3-p194/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:36:in `require'
    from (irb):1
    from /Users/chris/.rvm/rubies/ruby-1.9.3-p194/bin/irb:16:in `<main>'
1.9.3p194 :002 > require 'digest/md5'
 => true 
1.9.3p194 :003 > 

I can also confirm that changing "require 'md5'" to "require digest/md5" on line 23 of s3sync.rb will fix the error

@igreg
Copy link

igreg commented Feb 8, 2013

+1 for me, is this gem no longer maintained?

@maedana
Copy link

maedana commented Mar 25, 2013

+1

@cprobert
Copy link

There are 2 forks that I know of that have fixed this issue: aproxacs-s3sync and frahugo-s3sync.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants