Please first read the commit msg below, which is, I hope, self-explanatory, but I want to note a few things.

When searching to see how this issue was addressed in the past, I saw the recommendation was to set git global autocrlf to false. That's inadvisable because it affects checkout/clones of all other repos that the Windows user has. Setting repo local autocrlf to false is preferable but any local edits with CRLF will be merged unchanged.

So the safe alternative to this patch is to set local autocrlf to input for Windows users and document that clearly in both Git How-to and ffmpeg-web.git/src/download. gitattributes will have to track any deliberate CRLF files.

Regards,
Gyan
From e273754f53f2ad937e2ea05304dd554747fdc792 Mon Sep 17 00:00:00 2001
From: Gyan Doshi <gyando...@gmail.com>
Date: Sat, 7 Apr 2018 13:31:55 +0530
Subject: [PATCH] gitattributes - force LF for some FATE srt and webvtt refs

The SRT and WebVTT muxers only write out LF line endings.
On Windows, the refs for a few related FATE tests are
checked out, by default, as CRLF, causing these tests to fail.

Patch forces these files to be checked out as LF.
---
 .gitattributes | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/.gitattributes b/.gitattributes
index 5a19b963b6..59e20ffc92 100644
--- a/.gitattributes
+++ b/.gitattributes
@@ -1,2 +1,6 @@
 *.pnm -diff -text
 tests/ref/fate/sub-scc eol=crlf
+tests/ref/fate/sub-srt-empty-events eol=lf
+tests/ref/fate/sub-srt-madness-timeshift eol=lf
+tests/ref/fate/sub-srt-rrn-remux eol=lf
+tests/ref/fate/sub-webvttenc eol=lf
-- 
2.12.2.windows.2
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to