Sverre Rabbelier [off-list ref] writes:
On Tue, Sep 8, 2009 at 09:15, Junio C Hamano[off-list ref] wrote:
quoted
+ /* would_overwrite */
+ "Your local changes to '%s' will be clobbered by merge. Aborting.",
Still scary, shouldn't that be s/will be/would be/ ?
Thanks, very true indeed. "It would be clobbered if we were to continue
hence we abort." is how we want to explain our behaviour, so "would" is
definitely better here.
quoted
+ /* not_uptodate_dir */
+ "Updating '%s' would lose untracked files in it. Aborting.",
This is "merge would resolve to have a file X, but you have a directory X
in your work tree and it is not empty" case.
I'll leave the exact wording up to other people. My primary focus was to
end all of these messages with "Aborting."
This turns out to be a continuation of an older discussion thread back in
May 2008, and I do not know if anybody took up the challenge back then. I
wouldn't be surprised if "checkout", which was the topic of the old
thread, has some other scary plumbing message still seeping through to the
UI layer. Perhaps there are some other commands that needs similar kind
of love.
On Tue, Sep 08, 2009 at 12:48:23AM -0700, Junio C Hamano wrote:
Thanks, very true indeed. "It would be clobbered if we were to continue
hence we abort." is how we want to explain our behaviour, so "would" is
definitely better here.
While we're picking apart your wording, is "clobbered" the word we want
to use? Everywhere else that is user-facing we tend to use the term
"overwritten".
-Peff
2009/9/8 Junio C Hamano [off-list ref]:
I'll leave the exact wording up to other people. My primary focus was to
end all of these messages with "Aborting."
This turns out to be a continuation of an older discussion thread back in
May 2008, and I do not know if anybody took up the challenge back then. I
wouldn't be surprised if "checkout", which was the topic of the old
thread, has some other scary plumbing message still seeping through to the
UI layer. Perhaps there are some other commands that needs similar kind
of love.
Just a note that Documentation/git-checkout.txt references this
message in an example and should be kept in step with the final
wording change. It would be ideal to be able to regression-test the
examples in the documentation somehow but that might involve abusing
the asciidoc markup somewhat.
2009/9/8 Jeff King [off-list ref]:
While we're picking apart your wording, is "clobbered" the word we want
to use?
If we're debating 'clobbered', then maybe the non-word 'uptodate' is
fair game too? 8-)
Mike Ralphson [off-list ref] writes:
2009/9/8 Junio C Hamano [off-list ref]:
quoted
I'll leave the exact wording up to other people. My primary focus was to
end all of these messages with "Aborting."
This turns out to be a continuation of an older discussion thread back in
May 2008, and I do not know if anybody took up the challenge back then. I
wouldn't be surprised if "checkout", which was the topic of the old
thread, has some other scary plumbing message still seeping through to the
UI layer. Perhaps there are some other commands that needs similar kind
of love.
Just a note that Documentation/git-checkout.txt references this
message in an example and should be kept in step with the final
wording change. It would be ideal to be able to regression-test the
examples in the documentation somehow but that might involve abusing
the asciidoc markup somewhat.
Thanks.
2009/9/8 Jeff King [off-list ref]:
quoted
While we're picking apart your wording, is "clobbered" the word we want
to use?
If we're debating 'clobbered', then maybe the non-word 'uptodate' is
fair game too? 8-)
I think you added smiley because you already knew the answer, but if that
is not the case, please see
http://thread.gmane.org/gmane.comp.version-control.git/81100/focus=82358