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

Jump to bottom

Cleanups and pack file improvements #109

Merged
merged 27 commits into from Jul 31, 2015
Merged

Cleanups and pack file improvements #109

merged 27 commits into from Jul 31, 2015

Conversation

samoht
Copy link
Member

@samoht samoht commented Jul 31, 2015

Need to check that it works fine on 32b architectures before merging.

samoht added 16 commits July 30, 2015 17:27
@samoht
Simpler and more efficient pack file API
b5cd819 
Multiple goals:
- refactor the pack index code to integrate @codinuum more deeply
- speed-up the unpacking of the pack file to rebuild to index on `ogit pull`
- make 'ogit push' works more reliably (not very well tested)
- should work on 32b systems

status: the library compiles, need to make the tests compile again and
fix regressions, and add new tests for the new API calls (especially
SHA.Array) and all the position-indenpendant pack objects.
@samoht
Rename Git.SHA.length into Git.SHA.hex_length to avoid confusions
4fd2959
@samoht
Avoid looping forever in Git.SHA.Array.linear_search
8660d77
@samoht
Explore all the cells in Git.SHA.Array.binary_search
b60e5eb
@samoht
hex_lenght++
2bf5799
@samoht
Explicitely specify (in the type) that using Git.Pack.Raw.add is a ba…
da01722 
…d idea
@samoht
Fix the sending pack files
bf26858
@samoht
Add unit-tests for Git.SHA.Array
0e6a988
@samoht
Fix compilation for older versions of OCaml
0dddf88
@samoht
Do not break backward compatiblity in SHA.of_hex
555bf6e
@samoht
Use the new SHA.of_short_hex functions in ogit.ml
917d6aa 
Plus some minor style cleanups
@samoht
More ogit.ml cleanups
ab8821f
@samoht
Sort the SHA1 by keys when pretty-printing a pack index
68ef004
@samoht
Fix reading of SHA1 in pack index
2555445
@samoht
Allow to output progress on long-running actions (mainly sync)
b5b4452 
This is parametrized by a `progress` function (which by default ignore its
argument).
@samoht
Display progress reports on stdout when using ogit
cecbb7e
@samoht
Copy link
Member Author

samoht commented Jul 31, 2015

I managed to ogit clone https://github.com/ocaml/opam-repository from a 32b ubuntu, with a pack-file of 17.46 MiB. So I'll merge that branch when Travis is green and cut a release. @gregtatcam you are welcome to test that branch a bit more on 32b if yo want.

samoht added a commit that referenced this pull request Jul 31, 2015
@samoht
Merge pull request #109 from samoht/pack
9351fe0 
Cleanups and pack file improvements
@samoht samoht merged commit 9351fe0 into mirage:master Jul 31, 2015
@samoht samoht deleted the pack branch July 31, 2015 21:42
@gregtatcam
Copy link

Thanks!

On Jul 31, 2015, at 1:31 PM, Thomas Gazagnaire notifications@github.com wrote:

I managed to ogit clone https://github.com/ocaml/opam-repository from a 32b ubuntu, with a pack-file of 17.46 MiB. So I'll merge that branch when Travis is green and cut a release. @gregtatcam https://github.com/gregtatcam you are welcome to test that branch a bit more on 32b if yo want.

—
Reply to this email directly or view it on GitHub #109 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@samoht @gregtatcam