-
Notifications
You must be signed in to change notification settings - Fork 7
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
Upgrade liberasurecode 1.5.0 -> 1.6.1 #14
base: master
Are you sure you want to change the base?
Conversation
Build cache images are too old. I'll delete them and re-run. |
018cc07
to
52bee40
Compare
52bee40
to
116f713
Compare
note: こちらも参照 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Note: we need to check if frugalos built with this change works correctly. |
On MacOS High Sierra (10.13.6):
On Linux (CentOS 7):
Seems ok 😁 |
Note: fragments made in the old version and in the new version differ, because crc32's implementation changed. |
Resolves #13.
Probably this PR needs to wait until #12 is merged.