Commits
- Commit:
0deb960709e3e94b2b79beb88bafc443d6d117f1
- From:
- Omar Polo <op@omarpolo.com>
- Date:
show full URL during got clone/fetch/send
discussed with and ok jamsek, stsp
- Commit:
6242c45bf7571868b35d644cd0655d3b007d60c9
- From:
- Omar Polo <op@omarpolo.com>
- Date:
got send: show server error
Print the error message reported by the remote server when failing to
update a branch (for e.g. because of a server-side check.) Reported by
gonzalo@, with help and ok stsp@.
- Commit:
c9f1ac46710e3ac672c769862ae51b838c3c8e46
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
make 'got clone -b' work for repositories which lack a HEAD reference
ok op@
- Commit:
b90054ed55f30ebe28115abf5ad9cecc2b925713
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
use mkstemps(3) instead of mkstemp(3) for opening named temporary files
Allows 'got commit' to use a ".diff" suffix for temporary diff files.
ok op@
- Commit:
2a47b1e5852390eadc730c1dd2dd7caae011adfa
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
provide a diff of changes in a temp file while editing a commit log message
ok landry@
- Commit:
21f077265038a0c30c857035d2ab286f293573b1
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
require space between commit author name and email, for Git compatibility
Allowing such author fields breaks 'got send' towards Github for affected
commits because git-index-pack --strict will error out on the server:
$ git index-pack --strict pack-de791fb6a3a1961e44ac5d98d72fd533bf9277c8.pack
error: object 5d6bde9eaaf27f41ae8fa7112bb45e489d3c16b9: missingSpaceBeforeEmail:
invalid author/committer line - missing space before email
fatal: fsck error in packed object
problem encountered by landry@
ok op@
- Commit:
6f3190632709528909c1dc45046a2a85921584aa
- From:
- Josiah Frentsos <jfrent@tilde.team>
- Via:
- Stefan Sperling <stsp@stsp.name>
- Date:
sort getopt() option lists and switch statements; patch by Josiah Frentsos
- Commit:
9cda65e55e3f6d565e83377712e1b81177639cda
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
simplify the way 'got patch' opens a tempfile when reading from stdin
Also add basic test coverage for reading patches from stdin, while here.
ok op@
- Commit:
0e51642e8ca7b6c1024426347999853df4cc21e7
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
the my_refs reflist in delete_missing_refs() must be freed on exit
- Commit:
6a0a1bd407b4d06a6d003428daaa50cc2d92c6f9
- From:
- Omar Polo <op@omarpolo.com>
- Date:
sync usage with reality
The -h and -V/--version flags must be given before the command; with
those flags the command is actually optional, but don't over-complicate
the usage string with these nitpicks.
ok/improvements stsp@
- Commit:
e02b422b6f7736de0e851b3f8fad812e77e9c6b4
- From:
- Josh Rickmar <jrick@zettaport.com>
- Date:
disallow integrating into references outside refs/heads/
Spotted by stsp@ while considering a feature request for
cmd_integrate.
ok stsp@
- Commit:
c87842d5aed77e4f15bab3980e8cb22f18e954f8
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
show file mode for new added files in work tree diffs
Make work tree diffs of local changes on disk display the file mode of new
added files like other versions of `got diff` currently do.
ok stsp@
- Commit:
6f04a73d17761092a3a7ab07632cd1a89d4bfd21
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
add gotadmin init -b <branch> to specify repo head ref
Similar to `git init -b`. Includes a change to `got import` behaviour such that
"main" is no longer hardcoded by default; instead, we import to the branch
resolved via the repository's HEAD reference unless `got import -b` is used,
and only if HEAD cannot be resolved to a branch do we fallback to "main".
includes fix plus ok from stsp@
- Commit:
d9787ed86ecec0bda7a570181d86c44ba80bd583
- From:
- Omar Polo <op@omarpolo.com>
- Date:
make got_commit_graph_iter_next use caller-provided storage for the id
and adjust the callers. discussed with and ok stsp@
- Commit:
e15c42decfa8a80fb91cc1e19b467efc34a8c05d
- From:
- Omar Polo <op@omarpolo.com>
- Date:
plug leak in the commit graph iterator
We fail to release the memory for the nodes. To fix it however, we some
consumer of the commit graph iterator need to be corrected: the returned
pointer is safe to be used only up until the next iter_next call;
save a copy it if it's needed afterwards too.
ok stsp@
- Commit:
04666d1a54c25c8be7e39bc628b4a80f3376c127
- From:
- Omar Polo <op@omarpolo.com>
- Date:
got: fflush(stdout) after asking questions
Not everywhere getline does an implicit fflush, so we might end up not
showing the prompt to the user. Spotted on alpine with muslc.
ok Thomas Adams
- Commit:
442ede73eadb025cdc45bede186bf31aee869dad
- From:
- Stefan Sperling <stsp@stsp.name>
- Date:
forbid rebase of references outside the refs/heads/ namespace
ok jrick
- Commit:
8d4a8ca19cca1354c4d99d5df888c3c9b56e075c
- From:
- Josh Rickmar <jrick@zettaport.com>
- Date:
Do not segfault verifying "lightweight" tags
ok stsp@
- Commit:
5e91dae4dd43b8024731223110ab18f505f453ac
- From:
- Josiah Frentsos <jfrent@tilde.team>
- Via:
- Stefan Sperling <stsp@stsp.name>
- Date:
remove trailing whitespace; patch by Josiah Frentsos
- Commit:
360f3aea9198de6c93e8a185f87641b59c96dbfe
- From:
- Mark Jamsek <mark@jamsek.dev>
- Date:
got: fix typo in argc check when diffing two blobs
We already have two arguments when diffing two blobs, so check for more than
two like we do when diffing two objects to check for invalid path args.
Includes new regress test by op for diffing two blobs.
ok stsp@ and ok plus regress test from op@
- Commit:
85f2c2e0132ed34974446382474602b11d336f3a
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix overflow in blame callback
spotted by noticing gotwebd crashing on some blame requests.
Diff from stsp@ with a fix from tracey@, I'm committing it only because
he is short on time.
ok stsp@
- Commit:
438d0cc30baf8e97fd870f5e620833e0a45401aa
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix snprintf error handling
follow the "proper secure idiom" described in the CAVEATS section of
printf(3). reminded by tb@ and millert@
- Commit:
827a167b34401bd6541b199377e4e6c68d643151
- From:
- Josiah Frentsos <jfrent@tilde.team>
- Via:
- Stefan Sperling <stsp@stsp.name>
- Date:
Group options in accordance with style(9)
patch by Josiah Frentsos
- Commit:
d30ab56fff40b038af410d963b66e866a8e4e528
- From:
- Christian Weisgerber <naddy@mips.inka.de>
- Date:
fix unused variable, function definition warnings for clang 15
- Commit:
8fb6814c7fc92dbee5d95b1c66383d00bfa60f49
- From:
- Omar Polo <op@omarpolo.com>
- Date:
avoid allocating too many errors in cmd_info
got_error_path uses get_custom_err to get a statically allocated error
allocated on a buffer that's used a ring. This is nice and simple, but
the drawback is that thees custom errors shouldn't be used "for too
long" because they might get overwritten later.
cmd_info is one offender: change it to store a "simple" error and later
re-use it to construct the per-path error.
OK and tweaks stsp@