Thread (178 messages) 178 messages, 10 authors, 2025-08-16

Re: [GSoC PATCH v5 0/5] repo: add new command for retrieving repository info

From: Eric Sunshine <hidden>
Date: 2025-07-27 20:12:09

On Sun, Jul 27, 2025 at 1:51 PM Lucas Seiki Oshiro
[off-list ref] wrote:
This v6 contains small fixes pointed in v5.

In v5, Patrick and Junio were discussing about quoting the values in the
key=value format using `quote_c_style` [1]. Given that it wouldn't affect
this patchset and it's a simple change that can be done later, I didn't
change it and I'll leave for further discussion when I start to deal
with paths.
The counterargument to this stance is that if you employ
`quote_c_style` from the onset and document it, then if a future
version of Git does start outputting values containing "funny"
characters for properties which previously did not emit such values
(this isn't limited to paths), then consumers who heeded the
documentation won't find their tooling breaking suddenly. Tooling may
break for consumers who did not heed the documentation, but that will
be due to their own shortsightedness, not due to the Git project
failing to care about compatibility between versions. For this reason,
I'm of the opinion that `quote_c_style` should be used and documented
even at this very early stage.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help