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

fix timestamps (but stays on Irmin 0.10.1) #48

Closed
wants to merge 9 commits into from

Conversation

hannesm
Copy link
Collaborator

@hannesm hannesm commented May 9, 2016

is on top of #43, cleaner code which also works with 0.10.1 (maybe too much work with 0.11.0 where the Topographical.fold seems to be sorted by date)

@hannesm
Copy link
Collaborator Author

hannesm commented May 10, 2016

doesn't seem to be good, on a git push to the repo, I get from canopy:

Raised at file "bytes.ml", line 220, characters 25-34
Called from file "src/pre_sexp.ml", line 86, characters 11-45
149.158 mirage       ERROR: Git_mirage.IO.read_exactly: expecting 4, got 0
Fail pull (Failure "Git_mirage.IO.read_exactly: expecting 4, got 0")

@Engil
Copy link
Owner

Engil commented May 10, 2016

It happens only on this revision ?
Because I have somewhere a nasty ocaml-git bug that happens on some specific repos, and this might be this one.

@hannesm
Copy link
Collaborator Author

hannesm commented May 12, 2016

not sure... also the first clone works fine, but subsequent pulls seem to not work properly...

@hannesm hannesm mentioned this pull request May 16, 2016
@hannesm hannesm closed this Jul 6, 2016
@hannesm hannesm deleted the irmin-0.10.1 branch November 2, 2016 19:22
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.

None yet

2 participants