Carlos Martín Nieto [off-list ref] writes:
The current message is too long and at too low a level for anybody to
understand it if they don't know about the configuration format
already.
Reformat it to show the commands a user would be expected to use,
instead of the contents of the configuration file.
---
Sounds like a change going in the right direction. I am unsure if it is a
good idea to remove "See git-config...", but otherwise I like the updated
text much better.
But of course I am not the target audience, so let's see what we hear from
others.
Thanks.
On Thu, 2012-02-23 at 12:28 -0800, Junio C Hamano wrote:
Carlos Martín Nieto [off-list ref] writes:
quoted
The current message is too long and at too low a level for anybody to
understand it if they don't know about the configuration format
already.
Reformat it to show the commands a user would be expected to use,
instead of the contents of the configuration file.
---
Sounds like a change going in the right direction. I am unsure if it is a
good idea to remove "See git-config...", but otherwise I like the updated
text much better.
We're already telling them to go look at the git-pull or git-rebase
manpages, where there's already a lot of information and adding another
RTFM seemed a bit unfriendly (and not that useful, as they'd be
manipulating the config via other commands and shouldn't need to care
about config that much)
But of course I am not the target audience, so let's see what we hear from
others.
Sure, I'll see if I can come up with some better wording and send a
signed-off patch.
cmn