From: Will Palmer <wmpalmer@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] make description of "core.autocrlf" less ambiguous
Date: Wed, 14 Apr 2010 15:49:34 +0100 [thread overview]
Message-ID: <m2p5b9751661004140749kf97d84f8ufee365390e1c57f5@mail.gmail.com> (raw)
In-Reply-To: <7vk4saqguf.fsf@alter.siamese.dyndns.org>
On Wed, Apr 14, 2010 at 2:55 PM, Junio C Hamano <gitster@pobox.com> wrote:
> Very low level plumbing commands deliberately omit the conversion in order
> to show the raw data (e.g. cat-file), so it is not correct to reword it to
> "when output" as in your version.
Blame poor testing on my part, then. Yesterday, my tests showed
"cat-file blob HEAD:a-crlf-file" outputting crlf lines, but
today (with a script, rather than my typing commands in by-hand) that
seems not to be the case.
I agree that verbose-and-anal is not the right way to go, but I still
think the phrases reading from / writing to "the filesystem"
sound very ambiguous, especially when related to a command which
effects the way git stores things in its internal filesystem.
Most other uses of the term "filesystem" in the manpage use wording
such as: "...filesystems like NFS..",
"..filesystems like FAT..", "traditional UNIX filesystems", etc. The
only non-explicit uses of the term talk about
"slow filesystems", which are clearly talking about something other
than git. The autocrlf mention is the only use of the
term "the filesystem".
Though at the time I thought I wasn't being anal enough, perhaps the
correct move would be to go the opposite direction:
technically not the-real-truth, but "good enough": maybe both
references to "the filesystem" should just be replaced with
"the work tree", which is the term used in the safecrlf section anyway?
prev parent reply other threads:[~2010-04-14 14:50 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-04-13 23:23 [PATCH] make description of "core.autocrlf" less ambiguous Will Palmer
2010-04-14 13:55 ` Junio C Hamano
2010-04-14 14:49 ` Will Palmer [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m2p5b9751661004140749kf97d84f8ufee365390e1c57f5@mail.gmail.com \
--to=wmpalmer@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).