Commits
- Commit:
088f4d22ce3dcd3090bd440308e0b8b4fc8529aa
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
make new log message modication check more robust; fixes regress fallout
ok jamsek
- Commit:
64a300dd8d0fd181638cad5dfdeba7582cb92093
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
garbage-collect redundant read_logmsg() parameter
As identified and suggested by stsp, with the previous commit,
'strip_comments' is now always set thus no longer needed.
ok stsp@
- Commit:
83389425cc9afb887c5428f2db3299a1b049d71e
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
got: use timestamp and emptiness to validate log message
As suggested by naddy: consider commit log messages valid provided the temp
file time stamp has changed and the file is not empty. This heuristic
provides the desired behaviour (i.e., reusing cherrypicked/backed-out log
messages) that's currently provided but is simpler to grok.
Improved by and ok stsp@
- Commit:
1ff8274840d12a105064c246d5602b0e93957be5
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
convert two files to utf8
ok stsp
- Commit:
e2d314ead910671af065da16bf254aaffc2193a1
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
add a TODO item for gotadmin pack command
- Commit:
5b289346a9a33da176f47bfd6e879cb8ed6ab41b
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
open_blob: copy whole id not just the sha1
- Commit:
38bdc99fc83d59d7c507555551823c6e31c3cde0
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
remove recv_disconnect() from repo_write.c; missed in previous commit
- Commit:
92db09ff823dd814cbde563bb4455af59ba66415
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
do not expect to see a DISCONNECT message in repo processes
The parent no longer sends this message. Perform related cleanup
in the shutdown path instead.
ok op@
- Commit:
4301059192a1614d1ae81c9c61820487bd15f9bb
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
add functions to extract object ids from fileindex entries
ok stsp
- Commit:
52939b686ae906897e9b7956b61d15c255539e6d
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
fix gotd segfault in libevent while disconnecting clients
Sending a DISCONNECT message to the repo process and then killing
it and freeing its structure won't work. The message ends up on
internal libevent queues and will soon crash because we have freed
the memory for the event structre.
Sending that message is a leftover from the times when the repo
procs were persistent.
- Commit:
43c08a530e595df40d8133b9a3f77318c2a04b44
- From:
- Tracey Emery <tracey@traceyemery.net>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
rm unused boolean grammar. ok stsp@
- Commit:
75a8c854e063f97e5a2605f5256a1ce15ffb2f24
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
make edits made to comments count as a log message modification
This supports use of cherrypick/backout without requiring the user
to modify the log message of the original commit.
ok jamsek, op
- Commit:
96d694ac3bc159099a9ad93aa49ec7ac1764b18d
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
fix gotd and gotsh usage() function declaration
"function declaration isn't a prototype"
should fix the error seen by tracey@ on sparc64
- Commit:
b7acbe65b9c3861892ccd85dce82d78aeb285f54
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
make gotd parent dispatch handlers more robust during teardown
We have observed gotd exiting after "cannot find client for fd N" errors.
If this occurs then we are likely in the process of disconnecting a client
session while processing an event from a child process. Treat the above
error as non-fatal and stop processing more events from the child process.
- Commit:
f54d892e4b41b214bfbc655c9d9b72d8b2bf0f07
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: reset 1 second refresh rate after starting search
This fixes an issue reported by Mikhail where CPU usage continuously
increases if search is started after the refresh rate has dropped from
100ms to 1s by resetting the 1s refresh rate if search is started after
fast initialisation has finished. This is not needed in limit_log_view()
because the 1s refresh rate is reset in draw_commits() once the screen
is redrawn (i.e., commits_needed == 0).
Tested by and based on initial patch from Mikhail.
ok stsp@
- Commit:
5330ab762909f51f51f564f8b8af9eac9c8f4354
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
drop double process name from some gotd logs
i.e. "listen: listen: shutting down" -> "listen: shutting down"
the procname is already prepended by vlog()
ok jamsek
- Commit:
b29d689bdb0acd6ef758897f0ed1db91172f931a
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: make bison's location configurable
Some systems have bison (yacc) in a non-standard location. Since
Autotolls is setting the YACC environment variable based on the
AC_PROG_YACC macro, let the user override this if YACC is set in the
environment already.
This issue was found by a user of MacPorts. I did not get any feedback
from them as to whether this worked, but my own testing indicates this
should work.
- Commit:
9efb7226be6de6927ff8e03da3c82cd71cf960df
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: cross-compile pages.c
When building the template files on the host (which is cross-compiled),
do so just by handling pages.c appropriately.
Based on a patch by op@, and modified by me.
- Commit:
4c63d3a5b747a6acb4786af9c9e24d422feff00b
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
only fetch the work tree's branch by default if it is inside "refs/heads/"
ok jamsek
- Commit:
84073f62fdfec793493251a7adbb339cd265a45b
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
got: use intermediate pointers to plug leak on realloc
And save worktree_branch_len for reuse.
ok stsp@ and op@
- Commit:
a6d87ea2fa484657855fbde3ed06fe6428d42a81
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
nix unnecessary colon in gotd log output
ok op@
- Commit:
9d0a7ee35139cbfd755c1647d0feeaa665e1a2a2
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
fix interaction of 'got fetch -b', got.conf, and work tree
Without branches in got.conf for a remote, and without -b/-R options,
the fallback to HEAD would only work when not invoked in a work tree.
With this fix 'got fetch' should behave as described in the man page.
The -b option now overrides both got.conf and the fallback to the work
tree's branch.
And fallback to HEAD works as expected when invoked in a repository.
Also, do not strictly require remote repositories to provide a branch
from the refs/heads/ namespace. In such cases users should be able to
use -R to select something to fetch.
ok jamsek
- Commit:
3c8e3a64b01bf1814cebd087e17d52ef79b0d668
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
hide unexpected diff output from xfail test
- Commit:
8682db42e0ef4ce144b7c6f934125877c33e5602
- From:
- Lucas <lucas@sexy.is>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
add xfail test for 'got status' ignore patterns with a trailing slash
Patch by Lucas
- Commit:
e012cf94a2ba4c30e3b5842aacd715b5f5e420f9
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
make 'got import' -I option match directory names with a trailing slash
reported by Lucas on IRC, who sent a patch which this commit was based on
ok jamsek