abbysmal / Canopy

A git-blogging unikernel written using MirageOS
ISC License
120 stars 27 forks source link

Fix inflator #87

Closed dinosaure closed 7 years ago

dinosaure commented 7 years ago

For curious men, the problem was about Mstruct and Decompress: with the previous inflator.ml, the inflate function shifts more than necessary the Mstruct.t and lost some informations used afterwards by ocaml-git (and it's why we have the reserved kind 5 then).

In this new version, I shift the Mstruct.t only what Decompress used.

dinosaure commented 7 years ago

Sorry again for the delay 👍 !

abbysmal commented 7 years ago

Thank you very much, It looks like it work, at least on Xen.

Unix is still crashing with this error:

2017-04-02 13:10:36 +00:00: INF [git.sync] Received a pack file of 341174 bytes.
2017-04-02 13:10:37 +00:00: WRN [canopy-store] failed pull Unix.Unix_error(Unix.EBADF, "check_descriptor", "")

For some reason. Will keep track of this issue somewhere else. In the meanwhile, merging, thank you again!