Commits
- Commit:
d60961d296583c78051ab30aeee8171fd9611aac
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: bubble up got_get_repo_tags
This bubbles up the call to got_get_repo_tags from gotweb_render_tags to
its caller, gotweb_process_request and gotweb_render_summary. It helps
simplifying gotweb_render_summary and making it easier to templateify,
but also helps in untangling a bit the code.
At this point gotweb_render_tags becomes just a wrapper to
gotweb_render_tags_tmpl so switch completely to it.
ok jamsek
- Commit:
3ab2c91424750aef75f929402e6a7581a704f36d
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: templateify gotweb_render_branches
no functional change intended. Bubble up the allocation of the reflist
from gotweb_render_branches to gotweb_render_summary (its only caller)
and rewrite it as a template.
ok tracey@
- Commit:
5fa52d6cdcc6de872ce3c7f6639cc638fd83eeea
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
got: style(9) and cleanup failure leaks lib/diff.c
Spotted while implementing diffstat plus one introduced with diffstat code.
Don't leak 'change' on got_pathlist_append() error. And don't leak modestr{1,2}
and l{1,2} char pointers in diff_blobs() and got_diff_tree(), respecitvely.
Regarding modestr leaks, rather than free at all error return points, use op's
suggestion to lift modestr vars to function scope.
ok op@
- Commit:
a7331d50e5949de9db777b05789d7180d1082fd4
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
got: set error on asprintf() failure
ok op@
- Commit:
00b3e9ae14f04a45f1ca7445bade6b41a6e8a1c5
- From:
- Omar Polo <op@omarpolo.com>
- Date:
replace malloc+memcpy with strndup. no functional change intended
ok stsp@
- Commit:
bfed0e200a721018cbc04abab4d86e4e700ad741
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
got.1: Add example which lists recently modified branches. I use this a lot.
- Commit:
f7a854cff3b036707a2322ebff8540f29561a8a5
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
use just one pointer variable to keep track of a client's repo process
ok op@
- Commit:
eac23c306591c21ddaa23930159d302f668ae34c
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
update client state tracking in the gotd parent process
The session process takes over the old state definitions under
a new name ("session state"). The parent only needs to keep
track of whether a client has been granted access, so it only
uses two states: NEW, and ACCCESS_GRANTED which is set as soon
as the auth process has granted repository access and before
the session and repo_read/repo_write children are started.
Because 'gotctl info' can no longer observe the session state
remove support code for printing it.
ok op@
- Commit:
e26970ccc755b7327924d761d1772a1bc5a5bf01
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotd: delete trailing blanks spotted while re-reading
- Commit:
6c3d326354af2183c210a5f1422d4c99a606ae66
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: show committer instead of author in BRIEFS
- Commit:
af800df5a11beb2e4b8f519f7b5171a41095afb6
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: use from/via in gotweb_render_commits too
like the previous, show also the committer if different from the author.
- Commit:
49632cd3b1c81b19229e3a4a523e411d29cd8827
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: make commit page more like `got log' / tog
- drop the "diff commit1 commit2" header; it's already showed in the
body of the diff.
- hide the tree id (not that useful to show.)
- show the committer if different from the author.
- use From/Via for Committer/Author.
discussed with jamsek, stsp, tracey
ok jamsek tracey
- Commit:
ee0b2e502f3be1d01835e96bdcd38afed67429f7
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
remove support for showing client capabilities in 'gotctl info'
The gotd parent process has lost access to client capabilities.
Take the easy way out and remove related code. If needed, client
capabilities can still be found in the debug log with 'gotd -v'.
ok op, jamsek
- Commit:
2e6043e87514d1a49e1c8a037d7c4e20869b07ad
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
tog: fix {first,last}_displayed_line dead store
Both are already assigned just above in open_diff_view()
ok op@
- Commit:
910d235d362fb7951e997ce08e3dfb4b8cb24c5d
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
typo: pass rejectfile--not outfile--to got_ferror()
ok stsp@
- Commit:
38ef2333dd2b85b1e6aed14ef36bca80cb8eb551
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
catch a few missed got_pathlist_append() error checks
ok stsp@
- Commit:
d0407b8677a44f0789c2f4cd238db49b3a28d6b1
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
typo: zap double semi-colon
- Commit:
2fc51af0411e9e5ab6f12a7cdb303fabd86571d7
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
do not forget to initialize delta_cache_fd in the session process
- Commit:
4da0355e93f589e30bffceac65bfb1a6f2395f36
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
remove now unused fields from struct gotd_client
- Commit:
587550a5041e8aba39cd1338a22e85b6405d879f
- From:
- Omar Polo <op@omarpolo.com>
- Date:
gotwebd: templateify gotweb_render_diff
ok tracey@; rebased after recent changes.
- Commit:
fd20239feff11b69919be913807aa1f371b06fb9
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
got: zap dead variable in add_branch()
ok stsp@ op@
- Commit:
ae7c1b785440d9b4e2289d935ca85725f45680f1
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
add a gotd session process, split off from the parent process
The new session process is able to manipulate files in the repository
and keeps track of the read/write client session state.
The parent process now restricts its view of the filesystem to the
absolute path stored in argv[0], and combines this with unveil "x"
on this path. As a result the parent process can only re-exec itself.
small tweaks + ok op@
- Commit:
75b17c2a7d14fc0476cba0375a5a031cf0c13a00
- From:
- Omar Polo <op@omarpolo.com>
- Date:
regress: strip trailing / from GOT_TEST_ROOT
otherwise most test fails because of double "/" ending in paths and
tricking some check into thinking that's failed.
ok jamsek stsp
- Commit:
34d80a0ff3357761ddb975a1a96d3ee795218e41
- From:
- Omar Polo <op@omarpolo.com>
- Date:
regress: use `local' for some variables
- Commit:
6b37f13aa48dd88d4cc6c8176104a3101d7a260c
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
add regress for 'got diff -d' and minor output fix
Ensure an actual file path is displayed in the 'got diff -d -c commit path ...'
case when one of the specified paths is a deleted file. Prior to this, the
unhelpful "/dev/null" label was shown instead. Includes some copypasta fixes
noted by op.
ok stsp@ op@