Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Files created on Windows should not use CRLF #405

Closed
pombredanne opened this issue Mar 8, 2015 · 7 comments
Closed

Files created on Windows should not use CRLF #405

pombredanne opened this issue Mar 8, 2015 · 7 comments
Assignees
Labels
bug This issue/PR relates to a bug. feature This issue/PR relates to major feature request. windows This issue/PR related to windows systems only
Milestone

Comments

@pombredanne
Copy link
Contributor

When running cookiecutter on Windows the output is different from running it on posix.
In particular even though templates use LF line endings, the results is files with CRLF line endings.
CRLF is bad in general in my book, but here converting to CRLF is not so nice.
Since Jinja uses LF by default, I wonder if this is not because Git would be trying to convert things, though my Git settings on Windows are set to make no conversions at all

@pombredanne
Copy link
Contributor Author

@audreyr this is a failing test for a start. cloning is not the culprit. The issue happens somewhere during file generation

@pombredanne
Copy link
Contributor Author

The commits in #406 shoud fix it.

@pydanny pydanny mentioned this issue Mar 15, 2015
@hackebrot hackebrot added the windows This issue/PR related to windows systems only label Jul 30, 2015
pombredanne added a commit to pombredanne/cookiecutter that referenced this issue Aug 10, 2017
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@themanifold
Copy link

Is anyone looking at this ticket? I've come across this and would be interested in a solution.

@pombredanne
Copy link
Contributor Author

@themanifold please go for it!

@themanifold
Copy link

themanifold commented Feb 13, 2019 via email

@franciscomoma
Copy link
Contributor

@pombredanne I've fixed it in this PR applying that @themanifold suggest. Is it solving your problems too?

@insspb insspb added the 1.8.0 label Dec 22, 2019
@insspb insspb added this to To do in 1.8.0 Release via automation Dec 22, 2019
@ssbarnea ssbarnea removed the 1.8.0 label Mar 30, 2020
@insspb insspb added bug This issue/PR relates to a bug. feature This issue/PR relates to major feature request. labels May 27, 2020
@insspb insspb self-assigned this May 27, 2020
@insspb insspb added this to the 2.0.0 milestone May 27, 2020
@insspb insspb removed this from To do in 1.8.0 Release May 27, 2020
@insspb insspb added this to To do in 2.0.0 Release via automation May 27, 2020
@insspb
Copy link
Member

insspb commented May 29, 2020

@pombredanne @themanifold @franciscomoma Fixed in cookiecuter 2.0 by #1407
You are welcome to do test of this fix, via using current master branch.

@insspb insspb closed this as completed May 29, 2020
2.0.0 Release automation moved this from To do to Done May 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug. feature This issue/PR relates to major feature request. windows This issue/PR related to windows systems only
Projects
2.0.0 Release
  
Done
Development

No branches or pull requests

6 participants