Thread (55 messages) 55 messages, 7 authors, 2021-03-23

Re: [PATCH v13] [GSOC] commit: add --trailer option

flat view

From: Christian Couder <hidden>
Date: 2021-03-22 21:56:33

On Mon, Mar 22, 2021 at 11:23 AM ZheNing Hu [off-list ref] wrote:
2.
`git interpret-trailers --in-place`  seem like work on git top-dir,
If I am in a sub-dir `b` and I want to change a file such as `d.c`,
then I must use `git interpret-trailers --in-place b/d.c` to add some
trailers.
What happens without --in-place? Are the input files read correctly?
I think the original intention of `--in-place` is to modify a file similar to
"$COMMIT_MSG_FILE", so make it run at top-dir, but this is not reflected
in the git documentation. This at least confuses people who use this
option for the first time. Is it worth modifying? Or is there something
wrong with the design of `--in-place`?
I haven't checked but there is perhaps a bug in
create_in_place_tempfile() in "trailer.c".
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help