From: Junio C Hamano <gitster@pobox.com>
To: "Sidney San Martín" <s@sidneysm.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Support wrapping commit messages when you read them
Date: Mon, 13 Feb 2012 14:25:29 -0800 [thread overview]
Message-ID: <7vzkcmbcbq.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: <46957CEB-5E48-4C11-8428-9A88C3810548@sidneysm.com> ("Sidney San Martín"'s message of "Mon, 13 Feb 2012 16:26:24 -0500")
Sidney San Martín <s@sidneysm.com> writes:
>> After all, SCM is merely a method to help communication between
>> developers, and sticking to the common denominator is a proven good way to
>> make sure everybody involved in the project can use what is recorded in
>> the repository. This is not limited only to the log message, but equally
>> applies to filenames (e.g. don't create xt_tcpmss.c and xt_TCPMSS.c in the
>> same directory if you want your project extractable on case insensitive
>> filesystems) and even to the sources.
>>
>> You need to justify the cause a bit better. Why is such a new logic
>> justified?
>
> You’re right, that sentence doesn't say anything.
>
> I agree that projects need to have standards for their commit messages,
> but I also think that line wrapping should be taken care of by the
> computer so that the humans can think about the content of their commit
> messages. It's easier for everyone.
I just typed M-q to wrap the above paragraph from you to make it readable.
"Computers are good at automating" is true, and that is why real editors
give an easy way to auto-wrap long prose in a paragraph while composing.
But "computers are good at automating" is not a convincing justification
to let the composer leave unreasonably long lines in the commit log object
and force the reader side to line-wrap the mess only to fix it up.
next prev parent reply other threads:[~2012-02-13 22:25 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-12-25 5:05 [PATCH] Support wrapping commit messages when you read them Sidney San Martín
2011-12-25 9:57 ` Junio C Hamano
2012-02-13 21:26 ` Sidney San Martín
2012-02-13 22:25 ` Junio C Hamano [this message]
2012-02-14 12:10 ` Holger Hellmuth
2012-02-20 21:09 ` Sidney San Martín
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=7vzkcmbcbq.fsf@alter.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=s@sidneysm.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