-
Notifications
You must be signed in to change notification settings - Fork 24
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
all files in this repository (unintentionally?) have CRLF line endings #239
Comments
This is correct. Not intentional. What problems exactly did it cause on Linux? Some files in particular? |
It's not causing "problems" per se. cargo / rustc can still read all the files. We just need to do some normalization to remove |
Yes, seems reasonable. Thanks for taking the time to drop by :) |
This happens most of the time during development on windows machines since windows defaults to CRLF. You might be able to prevent this by enforcing LF line endings through the configuration of e.g. EditorConfig. This will also help if some local testing or Development is transmitted to some linux based VM or Container. Additionally some As of why this might be really important: I know this bug very well from the Docker / virtualization point of view. Windows can read LF just fine, however Linux often demands LF and if not provided runs into arbitrary and unexpected errors which might cost some developers a lot of time to fix 😅. |
When packaging this crate for Fedora Linux as a dependency for the
image
crate, we found that all files that are not auto-generated by cargo duringcargo publish
carry CRLF line endings. This is rather unusual for Rust crates, and there is no config file to configure git to explicitly rewrite files to have CRLF line endings, so I assume this is not intentional.The text was updated successfully, but these errors were encountered: