SecLists icon indicating copy to clipboard operation
SecLists copied to clipboard

CRLF Line Endings

Open kazkansouh opened this issue 4 years ago • 4 comments

Hello.

I have noticed that the line endings in 212 of the 5144 .txt files are CRLF. I guess this is by accident and not intended? I find it can make scripting a little more inconvenient when reading these files as there is one more item to think about.

Usually, the GIT approach would be to rely on core.autocrlf or .gitattributes to manage the translation. That is, store with LF endings, and allow git to add CRLF during checkout when needed, e.g. on Windows.

If its ok, I can submit a simple PR that adds a .gitattributes file (to help standardise future commits) and standardise the existing line endings by running dos2unix on them?

kazkansouh avatar Mar 16 '20 08:03 kazkansouh

Would be good to add a .gitattributes file

L1ghtn1ng avatar May 26 '20 21:05 L1ghtn1ng

Love a PR! <3

g0tmi1k avatar May 27 '20 08:05 g0tmi1k

I find it can make scripting a little more inconvenient when reading these files as there is one more item to think about.

How so? Any half decent library/tool that parses files should already have the capability of seamlessly parsing CRLF or LF endings. If you can point a specific example I think this could be fixed with a CI/CD pipeline using a Github Action.

ItsIgnacioPortal avatar Jun 24 '22 02:06 ItsIgnacioPortal

@g0tmi1k I think this can be closed, also #977 renormalised the line endings.

molangning avatar Feb 19 '24 07:02 molangning