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

Build fails on Windows if you cloned with autocrlf=input, eol=lf #10398

Open
Eulbobo opened this issue Jul 26, 2024 · 2 comments · May be fixed by #10463
Open

Build fails on Windows if you cloned with autocrlf=input, eol=lf #10398

Eulbobo opened this issue Jul 26, 2024 · 2 comments · May be fixed by #10463
Labels

Comments

@Eulbobo
Copy link
Contributor

Eulbobo commented Jul 26, 2024

Some implementations relies on System.lineSeparator() to split configuration file lines

If you are running on Windows, your default lineSeparator is \n\r, but, if you cloned the project with LF end of lines, the split can't be done anymore and build fails
image

Issue #477 provided an improved way to deal with end of lines, so it may be the solution

Maybe I'm the only one using this setup, but I'll try and fix it anyway

@Eulbobo
Copy link
Contributor Author

Eulbobo commented Jul 31, 2024

image
Shouldn't take much time until I get something good

Eulbobo added a commit to Eulbobo/jhipster-lite that referenced this issue Jul 31, 2024
Changing end of file and properties String manipulation to use Properties class

close jhipster#10398
Eulbobo added a commit to Eulbobo/jhipster-lite that referenced this issue Jul 31, 2024
Changing end of file and properties String manipulation to use Properties class

close jhipster#10398
Eulbobo added a commit to Eulbobo/jhipster-lite that referenced this issue Jul 31, 2024
Changing end of file and properties String manipulation to use Properties class

close jhipster#10398
@Eulbobo Eulbobo linked a pull request Jul 31, 2024 that will close this issue
@renanfranca
Copy link
Contributor

Maybe I'm the only one using this setup, but I'll try and fix it anyway

I know your pain! It's good to have someone with this setup to ensure the project is working on it. I started using WSL on Windows to avoid Windows-only issues 😅, and to use the Bash scripts more easily on Linux. In my tests, WSL was faster. I talked about it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants