Re: [PATCH 4/4] git-imap-send: Add method to convert from LF to CRLF
- From
- Hitoshi Mitake <mitake@dcl.info.waseda.ac.jp>
- Date
- Feb 11, 2010, 14:37 UTC
- Message-ID
- <4B741643.1000607@dcl.info.waseda.ac.jp>
- In-Reply-To
- <m31vgu9yjk.fsf@localhost.localdomain>
(2010年02月10日 01:15), Jakub Narebski wrote:
Show 40 quoted lines
> Hitoshi Mitake<mitake@dcl.info.waseda.ac.jp> 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.