Commits
- Commit:
f292b27176775cc4061f3da0be7fe0efcaf0325c
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
simplify previous; avoid temporary string
blind ok stsp@
- Commit:
9472af9550ec85323ab6c004d71f8a671a6b014d
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
show reference labels next to commit messages in tog log view
requested by mpi@
ok op@
- Commit:
52c5094beddc1ccecba6d468c80d35bdce5bcc67
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog regress: TAB instruction + test for diff J keymap
Add the TAB instruction to the test harness, and a test case for the previous
commit: scroll down the log from the diff view with the J keymap when on the
last loaded commit.
ok stsp@
- Commit:
e0bcabc59053f9a9eb0fe046fcdc49d36eb46273
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: resize log view if toggling fullscreen from child view
We call resize_log_view() when toggling 'F'ullscreen from the log view, but
forgot to do this when toggling fullscreen from a child view if its parent
is the log view. This fixes a bug that prevents scrolling down the log view
from the diff view with the J keymap if already on the last loaded commit.
Regress test in the following commit.
ok stsp@
- Commit:
87a675e0d63fd0aa7e3a413cf19bc4ad53409d94
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: mark HSPLIT_SCALE as float; unbreak regress on i386
On i386 view_split_begin_y() computed a value slightly different
than on other arches, breaking the regress. Casting HSPLIT_SCALE
as float unbreaks it.
Diff from jamsek, committing on his behalf. Thanks!
- Commit:
5b3a801d85b1502e5f24373e4df4a1ad289b2acd
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: open screendump file early during regress
we can't open arbitrary files after unveil(), and that's a feature!
So, open it early and truncate it before use just in case screendump()
ends up being called more than once.
ok/tweak stsp, ok jamsek
- Commit:
5ffbe221c75789cd2cdb37e1b20f1a575a20447f
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: don't check whether stdin is a tty during regress
ok stsp, jamsek
- Commit:
f4086c716b2f27f2f0e34157dcd64960589a3c9f
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: don't open /dev/tty during regress
as it might not be there (for e.g. if ran under cron). Reuse instead
/dev/null since it's not expected to get input from stdin.
ok jamsek
- Commit:
88e8b64f39ed07952b8245c2e77070c4f4a1aea3
- From:
- Christian Weisgerber <naddy@mips.inka.de>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: always use alternate charset for vertical/horizontal line
ncurses(3) will use the line drawing characters from the VT100
alternate graphics character set if the terminal supports this as
indicated by termcap/terminfo, otherwise it will fall back to ASCII
replacement characters.
ok jamsek stsp
- Commit:
a184c764f00351d113f3c015e7b794e48e162e50
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: teach test harness to count and basic tree tests
Add count instruction to the test harness to simulate count modifier
compound keys (e.g., 11j), and add basic tests for the tree view.
ok stsp@
- Commit:
13bc6832b964af6431e0b8c6c8586405474e3e71
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog regress: zap needless defines and use "vt220" TERM
Also, overwrite - and | unconditionally when capturing screen dumps,
and unset TOG_COLORS and TOG_DIFF_ALGORITHM in regress setup.
All tweaks suggested by naddy@
ok stsp@
- Commit:
1134ebde7e9791e46cbbb9356aa0abf4617125da
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog regress: use pthread conditions for wait instruction
As suggested by stsp, rather than busy wait in the view loop, use pthread
conditions to wait for the blame thread to signal completion before drawing
the screen. We can add more conditions for other views when needed, but by
keeping the general WAIT_FOR_UI instruction and flag, this should make their
use context-dependent so we won't need more test harness WAIT_* instructions.
ok stsp@
- Commit:
9d9cab5e3aa27db5354b22d7d638230bbc4e92e7
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: lock mutex if leaving view_loop() on a script read error
ok stsp@
- Commit:
4334634cb7af0badfce4bc8e1ea2db71cc2705c0
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: tweak blame tog_view deallocation fix from 2ca2f982
Both view_close() and close_blame_view() are called from view_loop()
irrespective of error, so only call these from cmd_blame() to ensure
the tog_view structure is not leaked when open_blame_view() errors.
ok stsp@
- Commit:
8e778ade0370b5cabdf946ca7d9d6ac9338790b0
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: blame regress and new test framework wait instruction
Basic blame view test required adding the WAIT_FOR_UI instruction to delay
screen dumps till the ui has been completely rendered. This will also apply
to future test cases such as limited log views and further blame test cases.
ok stsp@
- Commit:
fa9bb690d2fe11cc48b603cb872a8a39a240d29d
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: rename regress test script envvar
As per stsp's idea: use a more descriptive name, and use the
TOG_TEST_SCRIPT envvar both in tog.c and the test scripts.
ok stsp@
- Commit:
c282f7879bc381d455df8a2fbe57446aba59c985
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
document the S key in tog's online help screen
- Commit:
51b7e1c33f37154713bea6490586e5403b86e306
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: plug tog_view structure leak on open_*_view() error
ok stsp@
- Commit:
1d98034bec23a98dae48660d895840b6f6403d93
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
release tog_io regress mock term resources in main()
ok stsp@
- Commit:
557d33657977f5752a77f1ac50dc51aaab381f41
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
make tog regress run with pledge active
ok jamsek
- Commit:
b85a34968b050b10c36277d23e4cbb4a3e903a36
- From:
- Mark Jamsek <mark@jamsek.dev>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: implement automated test harness
Add framework for scripting tog tests by capturing screen dumps to compare
with expected output. Includes some preliminary log view tests but lots more
tests are needed to cover diff, blame, tree, ref, and help views.
ok stsp@
- Commit:
7303e8c811ca9c78e98cb71ca412092c4f71387a
- From:
- Mikhail <mp39590@gmail.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
tog: resume blame and diff search from the first line
In diff and blame views, if the user scrolls up during a search, resume
search from the first displayed line--not the current match. This makes
sense in these views as the "cursor" or current line is the first line.
Patch by Mikhail.
ok op@
- Commit:
4fccd2fe0ffbbc668b66abe63614470635f92f1b
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: configure: split out dependencies
Rather than assume all dependencies are required for all programs, split
them out.
This will make packaging easier, as well as splitting the code to use
subprojects.
Note that due to the use of config.h semantics, in most cases the
got_compat.h header file is now at the top of the .c file it is included
in, so that it can handle the system header inclusion properly.
- Commit:
98670ba726486c39efff220ab1e074c62023aae7
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: rework SHA detection
Simply the SHA detection by not predicating on libcrypto, but instead
checking individual header files.
- Commit:
4680f704353811c8bb6ce65eac3714d1bd200c26
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: remove sha1.h; found portably
Remove sha1.h as this is found portably across systems.