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

Fix CRLF line endings #78

Closed
stweil opened this issue Jul 7, 2022 · 5 comments
Closed

Fix CRLF line endings #78

stweil opened this issue Jul 7, 2022 · 5 comments

Comments

@stweil
Copy link
Contributor

stweil commented Jul 7, 2022

v4/alto-4-2.xsd and v4/alto-4-3.xsd got the wrong line endings.

Is it still possible to replace CRLF by LF for those two files?

@cipriandinu
Copy link
Member

Hi, I think is not really "wrong" as I know CRLF is used on Windows based systems, LF in Unix world and CR on Mac world. Probably I am the guilty one since I am using Windows based machine, but I don't know exactly when and how this change appears since I haven't done anything explicit. I will propose this topic on the next meeting and we should come to a decision what should be the prefered format and then we should see how to ensure this is not changed accidentally.

@stweil
Copy link
Contributor Author

stweil commented Sep 16, 2022

Mac uses LF like Unix / Linux for many years now. And the convention for Git repositories is using LF for all kinds of text files. Then Windows users can choose whether their Git should convert that to CRLF.

@cipriandinu
Copy link
Member

There is any fast way to make this change directly in Git (for 4.2, 4.3 and the current "in progress" version 4.4)? Then since I am only editing directly in browser the files, the issue should not appear. I am not a real expert in using Git, some help would be great :)

@cipriandinu
Copy link
Member

Great, thank you! Probably we should do it again for 4.4 since the branch for adding is based on older 4.3 (containing CRLF).

@cipriandinu
Copy link
Member

@stweil I would propose to close this topic, since the issue was solved with your help. Also 4.4 proposal looks to be updated. Do you agree?

@stweil stweil closed this as completed Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants