Commit Briefs

0b1c1d9c3d Thomas Adam

got_repo_pack_fds_close: don't close(0)

got_repo_pack_fds_close will try to close all of the GOT_PACK_NUM_TEMPFILES fds, even the ones that didn't manage to get a value from got_opentempfd(), which would result in a close(0) if we do not initialize to -1 here.


46cd746291 Thomas Adam

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


512073980d Thomas Adam

portable: MacOS: export env from configure

Compiling -portable on MacOS directly would never have worked, as the environment was only being set when run under CI. Move these checks into configure.ac so that they propagate out to make, etc. Noticed by cjones on IRC.


6ddaab3f7a Thomas Adam

portable: remove stray .orig files

These shouldn't have been commited, and probably predate when the .gitignore file was updated to filter out .orig files.


732c70f595 Thomas Adam

portable: gotwebd: add headers to EXTRA_DIST

Add the header files under gotwebd to EXTRA_DIST so that they're included in the distribution tarball.


ce0dfd3bb6 Thomas Adam

portable: update READMEs with release info

Document how to release a new `-portable` version.


585fb4308d Thomas Adam

portable: explain how to sync/update -portable

Add a section to the README files explaining how -portable is maintained.


0dd633fe8a Thomas Adam

portable: maintscript: add GH remote

Ensure that the remote GH repo is fetched from.


3bfa5a0662 Thomas Adam

Prevent use-after-free of packed_refs_path in error path.

Found by llvm's scan-build. OK stsp


f2c54bde77 Thomas Adam

Prevent memory leak when asprintf fails.

Found by llvm's scan-build. OK stsp


cfc8922bd7 Thomas Adam

Make sure got_repo_pack_fds_close() frees a malloc'ed pointer.

Found by llvm's scan-build (bad free). OK stsp


dee47c8728 Thomas Adam

If the first readdir() returns NULL err is uninitialized.

This can't happen in practice, but llvm's scan-build doesn't know this. OK stsp


d9cf08d99b Thomas Adam

We don't care about the length of the read line.

Found by llvm's scan-build (dead store). OK stsp


17c93e37ad Thomas Adam

Do not ignore error from got_pathlist_append.

Found by llvm's scan-build (dead store). OK stsp


ef944b8b4a Thomas Adam

Do not ignore error from format_author().

Found by llvm's scan-build (dead store). OK stsp


c80a2e6597 Thomas Adam

Dot not ignore error from got_object_id_str().

Found by llvm's scan-build (dead store). OK stsp


9e39ca7723 Thomas Adam

path_got is unused and never assigned, no need to free it

OK stsp


3521b46649 Thomas Adam

Do not ignore I/O errors.

Found by llvm's scan-build (dead store). OK stsp


ebb42948df Thomas Adam

Don't chug along if repo format version is unsupported.

Found by llvm's scan-build (dead store). OK stsp


399ea8e46a Thomas Adam

gotwebd: fix potential type mismatch between format specifier and argument

ok tracey


eaeaa612f1 Thomas Adam

tog: blame view keymap to open log of annotated line

As per stsp's TODO item: open a log view of the selected line with 'L'. helped by and ok stsp@


4ec2ae6313 Thomas Adam

regress: add missing return in error path


4ff74dcfee Thomas Adam

document pull request workflow in got.1 man page

With permission from Jelmer Vernooij to use my pull request config for Dulwich, and Jelmer's name, as a real-life example. ok tracey, jamsek


b0af9d3914 Thomas Adam

typo in cat_commit

otherwise prints the author instead of the committer


777aae211a Thomas Adam

tog: alias J and K to > and < keybindings for diff view

Use J and K to move to the next and previous commits, respectively, when the diff is loaded from the log or blame views. Patch by Mikhail ok stsp@