Commit Briefs

Stefan Sperling

allow start_pack_privsep_child() to be called from outside lib/object.c

diff | patch | tree

Stefan Sperling

open temporary files needed for delta application in got_repo_open()

This prepares for callers of got_repo_open() that cannot afford to open files in /tmp, such as gotwebd. In a follow-up change, we could ask such callers to pass in the required amount of open temporary files. One consequence is that got_repo_open() now requires the "cpath" pledge promise. Add the "cpath" promise to affected callers and remove it once the repository has been opened. ok tracey

diff | patch | tree

Stefan Sperling

avoid get_delta_chain_max_size() in dump_delta_chain_to_mem()

diff | patch | tree

Stefan Sperling

avoid get_delta_chain_max_size() in dump_delta_chain_to_file()

diff | patch | tree

Stefan Sperling

store deltas in compressed form while packing, both in memory and cache file

This reduces memory and disk space consumption during packing. with tweaks + memleak on error fix from op@ ok op@

diff | patch | tree

Stefan Sperling

inline struct got_object_id in struct got_object_qid

Saves us from doing a malloc/free call for every item on the list. ok op@

diff | patch | tree

Stefan Sperling

remove needless header inclusion; lib/pack.c does not need got_opentemp.h

diff | patch | tree

Omar Polo

factorize imsg_clear calls after imsg_flush failures

imsg_clear frees and closes resources allocated as part of enqueueing imsgs so it's a no-op after reads. discussed with and ok stsp@

diff | patch | tree

Stefan Sperling

reuse existing deltas when creating pack files

tested by thomas, naddy, and myself

diff | patch | tree

Stefan Sperling

fix double-free and double-close issues in error paths of got_packidx_open()

diff | patch | tree

Stefan Sperling

fix file corruption regression in 'got checkout' caused by commit db696021

got-read-pack must rewind its files temporary files before reusing them. Problem reported by naddy ok tracey naddy

diff | patch | tree

Stefan Sperling

add missing checks for reads beyond the mapped memory area of a pack file

diff | patch | tree

Stefan Sperling

fix bogus "object not found" errors when object ID begins with 00

ok thomas_adam

diff | patch | tree

Stefan Sperling

wrap overlong lines

diff | patch | tree

Stefan Sperling

add O_CLOEXEC (close-on-exec) to openat(2) calls

suggested by millert ok thomas_adam

diff | patch | tree

Stefan Sperling

don't scan pack index offsets for large values if pack file is < 2GB

This saves an iteration over the entire h->offsets array when opening a pack index which should not contain large offsets in the first place. ok millert@

diff | patch | tree

Stefan Sperling

handle pack index files which lack a corresponding pack file

ok millert

diff | patch | tree

Stefan Sperling

make got_packidx_get_packfile_path() usable for callers who only have a path

diff | patch | tree

Christian Weisgerber

switch from SIMPLEQ to equivalent STAILQ macros

The singly-linked tail queue macros were added to OpenBSD 6.9 and are more widely available on other systems. ok stsp

diff | patch | tree

Stefan Sperling

expose got_pack_parse_ref_delta() for library-internal use

This will be needed by a future 'gotadmin listpack' command.

diff | patch | tree

Stefan Sperling

expose got_packidx_get_object_offset() for library-internal use

This will be needed by a future 'gotadmin listpack' command.

diff | patch | tree

Stefan Sperling

mark got_pack_stop_privsep_child() static; it is only used inside pack.c

diff | patch | tree

Tracey Emery

fix a leaky ibuf. ok stsp

diff | patch | tree

Stefan Sperling

add got_packidx_get_packfile_path() for library-internal use

diff | patch | tree

Stefan Sperling

when reading a pack index byte-swap fields at compile-time where possible

suggested by mpi@ ages ago

diff | patch | tree