git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 1/1] chainlint: fix for core.autocrlf=true

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Aug 15, 2018, 16:32 UTC
Message-ID
<CAPig+cQkHiKDUhMJEdOewnEHTiTvdULwhZGd_h_94W0+zzGb0Q@mail.gmail.com>
In-Reply-To
<d0d1d49d6f3e741e9c048041a2f467742bce4bdc.1534343621.git.gitgitgadget@gmail.com>

On Wed, Aug 15, 2018 at 10:33 AM Johannes Schindelin via GitGitGadget <gitgitgadget@gmail.com> wrote:

Show 10 quoted lines
> The `chainlint` target compares actual output to expected output, where
> the actual output is generated from files that are specifically checked
> out with LF-only line endings. So the expected output needs to be
> checked out with LF-only line endings, too.
>
> Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> ---
> diff --git a/t/.gitattributes b/t/.gitattributes
> @@ -1,4 +1,5 @@
> +/chainlint/*.expect eol=lf

Make sense. I did touch t/.gitignore for the chainlint series but never thought to examine t/.gitattributes (and wasn't necessarily aware of its existence). Had I looked inside .gitattributes, perhaps I would have intuited the need for this change. Thanks for handling it.

Previous: Junio C Hamano
Message 4 of 4 in “Fix make -C t chainlint with DOS line endings”
  1. 0/1 Fix make -C t chainlint with DOS line endingsJohannes Schindelin via GitGitGadget, Aug 15, 2018
  2. 1/1 chainlint: fix for core.autocrlf=trueJohannes Schindelin via GitGitGadget, Aug 15, 2018
  3. Junio C HamanoAug 15, 2018
  4. Eric SunshineAug 15, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.