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

#310 upgraded to Takes 0.20.2 #316

Merged
merged 2 commits into from
Jun 8, 2015
Merged

#310 upgraded to Takes 0.20.2 #316

merged 2 commits into from
Jun 8, 2015

Conversation

dmzaytsev
Copy link
Contributor

For #310

upgraded to Takes 0.20.2
fixed the file corrupt issue,
I did compare the file hash before the upload and after the download
on windows
certutil -hashfile test_movie_before.mp4
certutil -hashfile test_movie_after.mp4

@dmzaytsev dmzaytsev changed the title #310 upgraged to Takes 0.20.1 #310 upgraded to Takes 0.20.1 Jun 8, 2015
@yegor256
Copy link
Contributor

yegor256 commented Jun 8, 2015

@dmzaytsev we have to use 0.20.2, due to this rather serious issue in 0.20.1: yegor256/takes#336

@dmzaytsev
Copy link
Contributor Author

@yegor256 got it, will do

@dmzaytsev dmzaytsev changed the title #310 upgraded to Takes 0.20.1 #310 upgraded to Takes 0.20.2 Jun 8, 2015
@yegor256
Copy link
Contributor

yegor256 commented Jun 8, 2015

@rultor merge

@rultor
Copy link
Contributor

rultor commented Jun 8, 2015

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit 8fe26c8 into libreio:master Jun 8, 2015
@rultor
Copy link
Contributor

rultor commented Jun 8, 2015

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 10min)

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.

3 participants