Re: [PATCH 4/4] git-imap-send: Add method to convert from LF to CRLF

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



(2010年02月10日 01:15), Jakub Narebski wrote:
Hitoshi Mitake<mitake@xxxxxxxxxxxxxxxxxxxxx>  writes:

Some strict IMAP servers (e.g. Cyrus) don't
allow "bare newlines ('\n')" in messages.
So I added new boolean option "lf-to-crlf" to imap section.
If this option enabled, git-imap-send converts LF to CRLF("\r\n").

If you want to use it, add line:
	lf-to-crlf
to [imap] section of your .gitconfig .

This patch also adds description to Documentation/git-imap-send.txt .

+imap.lf-to-crlf::
+	If you use strict IMAP server (e.g. Cyrus),
+	"bare newlines ('\n')" in messages are not allowed.
+	If this option enabled, git-imap-send converts LF to CRLF("\r\n").
+

If you take a look at Documentation/config.txt at the names of other
config variables, you would see that they have

   core.fileMode::
   core.ignoreCygwinFSTricks::
   core.quotepath::
   core.safecrlf::

names, i.e. either camelCase or allsmallcase, and not

   imap.lf-to-crlf::

with '-' to separate parts.  Values can and do use this syntax, like
e.g. `blank-at-eol` for core.whitespace.

The only outlier is add.ignore-errors (not add.ignoreerrors or
add.ignoreErrors).


The same is true for the other config variable you propsed in 3/4
patch.

Thanks for your review, I'll rename these values
based on custom of Git.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]