Commit Briefs

68bdcdc2f5 Stefan Sperling

addblk() may seek in its input file; reposition the file pointer afterwards

diff | patch | tree

a893025fd2 Stefan Sperling

addblk: iterate over the correct number of entries after growing the array

ok naddy

diff | patch | tree

e89540a95a Stefan Sperling

addblk: be more careful about expanding the blocks array when we outgrow it

fixes + ok naddy

diff | patch | tree

51a494da48 Stefan Sperling

check a block's hash as well as its length before expensive comparisons

suggested by + ok naddy, and Ori agrees

diff | patch | tree

282f42e5d1 Stefan Sperling

remove stray debug print which was accidentally committed

diff | patch | tree

0c9eeee211 Stefan Sperling

avoid uninitialized error being returned from got_repo_get_packfile_info()

diff | patch | tree

3b9e6fcfb6 Stefan Sperling

simplify deflate checksumming code; fixes errors with upcoming 'gotadmin pack'

tweak + ok naddy

diff | patch | tree

54d5be0776 Stefan Sperling

fix unrelated changes being merged by got cherrypick/backout/rebase/histedit

This was a long-standing and very annoying bug. The two xfail tests in the cherrypick test suite are passing now.

diff | patch | tree

1af628f4bf Stefan Sperling

switch merge_file_cb() to using merge_file() instead of merge_blob()

diff | patch | tree

eec2f5a957 Stefan Sperling

decouple merge_file() parameters for 2nd derived version and merge target file

diff | patch | tree

dc082d4a7a Stefan Sperling

in merge_file(), remove allocation of a base_path which is unused

diff | patch | tree

07bb0f933a Stefan Sperling

move merge_file()'s f_deriv parameter next to the f_orig parameter

diff | patch | tree

db59069162 Stefan Sperling

make merge_file() accept FILEs instead of paths for orig and deriv inputs

diff | patch | tree

67a666476f Stefan Sperling

refactor merge_file() such that it no longer requires a blob object parameter

diff | patch | tree

3cd22b2144 Stefan Sperling

add an xfail test for a cherrypick bug where unrelated changes get merged

The test case I am using here is a bit large but I could not yet find a way to make it smaller. If someone knows a smaller test case, please let me know. naddy caught a small bug in the new test code I had written, thanks!

diff | patch | tree

f270548637 Christian Weisgerber

dial_git(): fix memory leak and simplify

Simplify dial_git() by formatting the initial Git protocol packet with dprintf, suggested by millert, and stop leaking an asprintf buffer. ok stsp

diff | patch | tree

57c1819858 Stefan Sperling

validate reference names in open_ref()

This catches invalid reference names passed to 'got ref -l' and will also be needed to validate reference names passed to a future 'gotadmin pack' command. ok naddy@

diff | patch | tree

91b40e30e0 Stefan Sperling

add checksum support to got_deflate_to_file()

This will eventually be used by 'gotadmin pack'. Checksum init and finalization will need to be done by the caller since many objects will be written out in compressed form while we are computing checksums across the entire pack file. ok millert, naddy

diff | patch | tree

af8a5c4afb Stefan Sperling

plug a memory leak in got_ref_list_free()

Code which inserts a reference into a reflist passes ownership of this reference to the list. However, got_ref_list_free() never closed such references, which resulted in a memory leak. ok tracey

diff | patch | tree

9069347b69 Stefan Sperling

improve error reporting when delta application fails

diff | patch | tree

e5ad73651d Stefan Sperling

fix file descriptor leak when got-read-object returns a raw object

diff | patch | tree

dbbf4a5f0c Stefan Sperling

allow got_deltify_free(NULL); will be needed by 'gotadmin pack'

diff | patch | tree

932dbee786 Stefan Sperling

fix wrong function name in error message; pointed out by Mikolaj Kucharski

diff | patch | tree

a486b62b84 Stefan Sperling

ignore SIGWINCH while polling in the main process

Avoids an error where tog(1) would exit with "poll: Interrupted system call" while the terminal window was being resized. ok millert

diff | patch | tree

a3af57b852 Stefan Sperling

remove unused worktree-related includes from lib/repository.c

diff | patch | tree