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

ExactTimestampsSync.compare_time() explodes on super call #960

Closed
MichaelBlume opened this issue Oct 22, 2014 · 2 comments · Fixed by #964
Closed

ExactTimestampsSync.compare_time() explodes on super call #960

MichaelBlume opened this issue Oct 22, 2014 · 2 comments · Fixed by #964
Labels
bug This issue is a bug. s3sync

Comments

@MichaelBlume
Copy link

return super(SizeOnlySyncStrategy, self).compare_time(src_file, dest_file) references a class that isn't imported and doesn't appear to exist. Also, shouldn't a super call from ExactTimestampsSync be passing ExactTimestampsSync to super

@kyleknap
Copy link
Contributor

Thanks for looking into the issue. Yeah, you are correct that should not be there. I will work on fixing it.

@kyleknap kyleknap added bug This issue is a bug. confirmed s3 s3sync and removed s3 labels Oct 22, 2014
@MichaelBlume
Copy link
Author

Thanks =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. s3sync
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants