Commits
- Commit:
01d78e9e349d8547e36bfe47e4c53acfcef2973d
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
make 'got send' detect connections that are unexpectedly closed by the server
ok op@
- Commit:
3d1b16d126400e28a2cbab66b17dacd47a01763c
- From:
- Josh Rickmar <jrick@zettaport.com>
- Date:
Begin cvg
- Commit:
bafea5babab6072bfa7528ad7c767bde76a446a3
- From:
- Josh Rickmar <jrick@zettaport.com>
- Date:
plug a leak in insert_sendable_ref error path
from op@
- Commit:
27555e8e6053ab0cc846d201757b588d5a79293f
- From:
- Josh Rickmar <jrick@zettaport.com>
- Date:
simplify lib/send.c reference handling
Reorganize the reference validation and pathlist generation by
removing the reflist and building a pathlist directly. The pathlist
entries record the object id in their extra data pointer, which also
allows several redundant reference lookups to be skipped.
This will eventually simplify sending target reference names that do
not match the local repo by adding another parameter to
insert_sendable_ref for a remote reference. This remote name will be
added to the pathlist, but validation and object id lookups will
continue to be performed with the local reference.
ok jamsek
- Commit:
e166551f658768ab6f4f75386d7da7d6c67b7ee3
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
fix sending merge commits; with a regression test by James Cook
- Commit:
1eb3899277955a87d80eef900b6ea458ff73bd6c
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
significantly reduce the amount of code linked into gitwrapper
By moving got_serve_parse_command() from lib/serve.c into lib/dial.c
as got_dial_parse_command(), we can significantly reduce the amount
of symbols gitwrapper depends on indirectly.
As a downside, gotsh now needs to link to dial.c. But it only uses the
same parsing routine, and any other routines in dial.c would likely
cause pledge violations in gotsh if used.
No functional change.
- Commit:
53bf0b541977b66862040d4b633fb6b5d3a3c6c8
- From:
- Omar Polo <op@omarpolo.com>
- Date:
rename lib/sha1.c to lib/hash.c
It will soon grow functions to deal with sha256 too. stsp@ agrees.
- Commit:
5822e79e39a8777984e08b5d89a8c55d6aa0356d
- From:
- Omar Polo <op@omarpolo.com>
- Date:
include sha2.h too where sha1.h is included
In preparation for wide sha256 support; stsp@ agrees. Change done
mechanically with
find . -iname \*.[cy] -exec sam {} +
X ,x/<sha1\.h>/i/\n#include <sha2.h>
- Commit:
bd56d7e1de5fb58f775afab6d908dbea7a2833de
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
unbreak send.sh regress from recent ref-delta changes
fix argument order typo in flags passed to got_pack_create()
- Commit:
c7a4fcc8bebad9dc0bd1183fec3bd405dd4ebc57
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotadmin pack: add a -D flag to force using ref-delta
Intended mostly for the regress suite, so we'll be able to test also the
ref-delta code paths.
ok stsp@
- Commit:
d8bacb933720b8819f0c4e76b004775aa1885b9b
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
got: minor refactor of got_pathlist_free() API
Accept flag parameter to optionally specify which pointers to free. This saves
callers looping through the list to free pointers.
ok + fix stsp@
- Commit:
6242c45bf7571868b35d644cd0655d3b007d60c9
- From:
- Omar Polo <op@omarpolo.com>
- Date:
got send: show server error
Print the error message reported by the remote server when failing to
update a branch (for e.g. because of a server-side check.) Reported by
gonzalo@, with help and ok stsp@.
- Commit:
cae60ab8f2a261b006b3ccbded2d53dccbd6f300
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
let callers of got_pack_create() configure rate-limiting of progress reporting
Needed by future gotd(8), where progress reports will be sent to a network
socket, rather than a local terminal.
- Commit:
a32780aad7a4bc8539de9e7fd39b371f2fe8c346
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
move got_opentempfd() call out of got_pack_create()
Future gotd(8) needs to run got_pack_create() in a chroot environment,
so we can no longer open new temporary files inside got_pack_create().
ok op@
- Commit:
894e4711ffca9e21b24e567a610d387bd0c31817
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
make got_pack_create() write to a file descriptor instead of a stdio FILE
The old code required a seekable output file. This conflicts with requirements
of future gotd(8), which will write pack file data to network sockets.
ok op@
- Commit:
c46eda88bcbbcf577ca7a805746b44d63c0e4c0a
- From:
- Florian Obser <florian@narrans.de>
- Date:
Correctly track if we sent more data since the last time we looked.
With the narrow scope of bytes_sent_cur it would always be zero, and
we would always update progress.
Found by llvm's scan-build (Dead store).
OK stsp
- Commit:
b8af7c0693fbaccaecf7b904aaf237066269f222
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
print additional progress information while packing
ok op@
- Commit:
924100791157ab7f6842e7c7b5bd225c00ad5aab
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
fix regression where 'got send -T' failed if same tag already exists on server
Problem reported and fix tested by Omar Polo.
- Commit:
f8b19efda6453501f8bc4ce868f953b703dca576
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
use RB_TREE instead of STAILQ to manage packindex bloom filters; much faster
- Commit:
4e91ef1519233965edd1bbd83f6699f669c8f9d6
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
make it possible to merge vendor branches with 'got merge'
- Commit:
c2105d002a73e60c6f3194dd96a1a0b09eb34e08
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
fix another instance of 'got send' sending branches the server already has
- Commit:
d65a88a2f0db40d2c2ac34ee34e8aff8ac629d52
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
move duplicated dial_ssh() and dial_git() functions into a common file
These functions are used by 'got send' and 'got fetch' in order to
open network connections to a server. Move them into new file lib/dial.c
and declare relevant functions in got_dial.h and lib/got_lib_dial.h.
No functional change.
- Commit:
535e07c7d678cfc4a2b6ad61f72c36e0a46e5111
- From:
- Christian Weisgerber <naddy@mips.inka.de>
- Date:
make realloc_ids() malloc-like and do not overallocate
Let realloc_ids() take as argument the number of entries to allocate.
Do not allocate an extra chunk.
ok stsp
- Commit:
8f2a2f4f1cea3d391fbf62e7507dd233cad10d99
- From:
- Christian Weisgerber <naddy@mips.inka.de>
- Date:
drop realloc_ids() initialization calls, one of which had a typo
realloc_ids() is called again in the loops that assign to the ID arrays,
before the first assignment.
ok stsp
- Commit:
27b75514d9b1c8e9f188ef9c483760647a8c1b72
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
do not send a pack file when 'got send' is only deleting branches
The git protocol spec says the client MUST NOT send a pack file
if the only command used is 'delete'.
Fixes 'got send -d' against Github's server which closed the session
upon receiving the empty pack file we sent. This problem wasn't caught
by regression tests since git-daemon does accept an empty pack file.
Problem reported by jrick.