Commit Briefs

b6b86fd1b9 Thomas Adam

remove trailing whitespace; patch by Josiah Frentsos


d54d17c383 Thomas Adam

portable: remove imsg.h

Allow imsg.h to be checked for portably.


60b94e7d2a Thomas Adam

mark got_error_fmt as printf-like and fix the arisen errors

ok stsp@


8b925c6ccd Thomas Adam

portable: add back sys/queue.h

Now that the handling of including sys/queue.h is better, there's no need to remove those lines from the source. Copy the location of those original sys/queue.h lines from upstream at the same line number, so as to avoid any conflicts in the future.


ec2b23c5d2 Thomas Adam

implement support for commit coloring in got-read-pack for speed

ok op, tracey


0c6f49baf2 Thomas Adam

build with -Wwrite-strings

Throwing this into the mix for a while, we can always get rid of it again if it becomes annoying. No objections from stsp@



ef20f54204 Thomas Adam

build with -Wmissing-prototypes

ok stsp@


e71f1e62dd Thomas Adam

let got-read-pack be explicit about whether it could enumerate all objects

This allows the main process to avoid looping over all object IDs again in case the pack file used for enumeration is complete. ok op@



63915ee56b Thomas Adam

Bring back object enumeration inside got-read-pack as a fast path.

The problem that was found in the earlier version has been fixed. ok op@


146e62ff57 Thomas Adam

portable: remove endian.h

This is included portably.


3dfecf3ec1 Thomas Adam

revert object enumeration in got-read-pack for now; needs more work

This implementation marked commits and trees as enumerated before all trees which they depend on were enumerated. This behaviour leads to incomplete pack files when a tree is only partially packed and got-read-pack hits a missing tree entry as a result. The algorithm must be reworked such that packed leave nodes are marked enumerated first, then bubble-up. Found by op@


96b72a8e59 Thomas Adam

free id and path in load_packed_tree_ids() on error, else they would leak

pointed out by op@


132807502f Thomas Adam

implement object enumeration support in got-read-pack

ok op@


265bd9c38d Thomas Adam

map delta cache file into memory if possible while writing a pack file

with a fix from + ok op@


13eb256cde Thomas Adam

fix a bug in findwixt() which caused pack files with missing parent commits

The 'nskip' variable is supposed to reflect commits which are waiting on the queue and have the 'skip' color. Only increment 'nskip' when adding such commits to the queue. Problem observed with got send -T and a tag pointing to a deleted branch. Test to reproduce the bug written by op@.


cc524d369b Thomas Adam

use random seeds for murmurhash2

change the three hardcoded seeds to fresh ones generated on demand via arc4random. Suggested/fixed by and ok stsp@


036966bae3 Thomas Adam

include header


6c0788afee Thomas Adam

shrink struct got_pack_meta a bit by removing the have_reused_delta flag

This flag can be expressed as m->reused_delta_offset != 0 because all deltas in valid pack files will be written at a non-zero offset. We allocate a huge number of these structs during packing, so every little bit helps.


1569c56d2b Thomas Adam

reduce the amount of memory used for caching deltas during deltification

With files sorted properly for deltification we produce better deltas but end up consuming more memory and risk running into OpenBSD ulimits during packing. To compensate, reduce the threshold for the amount of delta data we store in memory, spooling more deltas into the cache file. ok op@


d23a6c95c5 Thomas Adam

store a path hash instead of a verbatim path in pack meta data

This reduces memory use by gotadmin pack. The goal is to sort files which share a path next to each other for deltification. A hash of the path is good enough for this purpose and consumes less memory than a verbatim copy of the path. Git does something similar. ok op@


eee114b41c Thomas Adam

fix paths stored in pack meta data, improving file deltification

The old code was broken and stored an empty path or filenames, instead of a repository-relative path. Which means we didn't sort files for deltification as was intended. Fixing this provides much better deltas in large pack files written by gotadmin pack -a. In my test case, pack size changed from 2GB to 1.5GB. ok op@


20c6bdb6df Thomas Adam

plug a small memleak on error in got_pack_create()


d93dda9a3b Thomas Adam

map delta cache file into memory if possible while writing a pack file

with a fix from + ok op@