Thread (60 messages) flat view 60 messages, 6 authors, 2017-08-17

Re: [PATCH 3/5] interpret-trailers: add an option to show only existing trailers

From: Stefan Beller <hidden>
Date: 2017-08-10 17:38:52

On Thu, Aug 10, 2017 at 10:33 AM, Jeff King [off-list ref] wrote:
On Thu, Aug 10, 2017 at 10:27:19AM -0700, Stefan Beller wrote:
quoted
quoted
I'm not fond of that, as it's vague about which exact trailers we're
talking about. I also thought of something like --verbatim, but I'd
worry that would seem to conflict with --normalize.

I dunno. All of the names seem not quite descriptive enough to me.
I meant 'exact' as in 'exactly from the patch/commit, no external
influence such as config', so maybe '--from-patch' or '--from-commit'
(which says the same as --no-config just the other way round.
Having --no- in config options as the standard is a UX disaster
IMHO as then we have to forbid the --no-no-X or reintroduce X
and flip the default)
Yes, that was definitely the other reason I didn't want to call it
"--no-config".  :)

It's not always from a patch or commit. The most accurate along those
lines is "--from-input".
quoted
Maybe --genuine ?
But in the greater context I think that's vague again; we don't know
which part of the command's operation is "genuine".
The input of course. ;) --genuine-input.
Perhaps "--exact-input" hits all of those. Or maybe "--only-input" to
match the other "--only".

I think I like that last one the best. It makes it clear that we are
looking just at the input, and not anything else. Which is exactly what
the feature does.
Makes sense to me,

Thanks,
Stefan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help