Commit Briefs


Stefan Sperling

typo fix in tog.1 man page


Stefan Sperling

remove now pointless 'check_disk' parameter of got_repo_map_path()

suggested by naddy


Stefan Sperling

make tog avoid got_repo_map_path() if a work tree is available

ok naddy


Christian Weisgerber

Call pthread_cond_destroy(cond) exactly once when closing a view.

This moves the pthread_cond_destroy(need_commits) from stop_log_thread(), which can be called twice, to close_log_view(), which is called once. It also destroys the commit_loaded condition variable, which is created in open_log_view() but was never destroyed. ok stsp


Stefan Sperling

highlight matched search terms in tog diff and tog blame views

ok naddy@




Stefan Sperling

plug two memory leaks in tog's draw_file()


Stefan Sperling

merge new diff implementation from the git.gameoftrees.org diff.git repository

This new diff implementation was started by Neels Hofmeyr during the u2k20 hackathon and now replaces diffreg.c code lifted from the OpenBSD base system. The integration of this code into Got was done by me. Got now uses the patience diff algorithm by default. The diff.git repository will remain the primary repository for the diff code, which already compiles and runs on other operating systems such as Linux. Any fixes and improvements for files inherited from the diff.git repository should be written against that repository and synced to got.git afterwards.


Stefan Sperling

bump version number


Stefan Sperling

typo fix in CHANGES


Stefan Sperling

fix 0.44 release date in CHANGES file


Stefan Sperling

changes for 0.44




Stefan Sperling

fix replacing symlinks with files

And add a test case which verifies that the inverse also works, i.e. a symlink being replaced with a regular file. problem reported and fix tested by jrick




Stefan Sperling

avoid got_repo_map_path() in 'got tree' if a work tree is available

Note that got_worktree_resolve_path() will return a path based on the current working directory if the argument is the empty string. This quirk keeps the behaviour of 'got tree' intact which depends on the current working directory if run in a work tree. ok naddy@


Stefan Sperling

test case for 'got integrate' failing if symlink changes to file; patch by jrick

This test verifies the current behaviour, even though it is not the most desirable behaviour.



Stefan Sperling

avoid got_repo_map_path() in 'got blame'

'got blame' does not need access to the work tree. So far the work tree was completely hidden with unveil(). We must now expose the work tree while resolving the path, so unveil() calls are shuffled around slightly. Failing realpath() calls would mess with path resolution otherwise. There's a bug in got_worktree_resolve_path() where it failed to canonicalize a path constructed from a user-specified path that does not exist on disk. Note that this path falls into strncmp() a few lines down. I am fixing this by adding canonicalization. Generally, joining paths with asprintf() and comparing paths with strncmp() is fragile. A more general solution might be needed but I am leaving that for another day. ok naddy



Christian Weisgerber

convert all remaining instances of chmod(2) to fchmod(2)

ok stsp