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: add .gitattributes to force bash scripts to have LF line endings #35

Merged
merged 1 commit into from
Jul 26, 2024

Conversation

NameOfTheDragon
Copy link
Sponsor Contributor

@NameOfTheDragon NameOfTheDragon commented Jul 26, 2024

Fixes #34

In the latest commit, the start.sh file has Windows line endings and that messes with Bash under Linux so the container will not start.

This PR fixes this by:

  1. Correcting the line endings to LF in the start.sh file
  2. Adding a .gitattributes file that forces LF line endings on all bash scripts.

This should ensure that even if the files are edited in a Windows editor (such as VS Code) that line endings will be correctly preserved.

@dw-0 dw-0 changed the title Added .gitattributes to force bash scripts to have LF line endings fix: added .gitattributes to force bash scripts to have LF line endings Jul 26, 2024
@dw-0 dw-0 changed the title fix: added .gitattributes to force bash scripts to have LF line endings fix: add .gitattributes to force bash scripts to have LF line endings Jul 26, 2024
@dw-0 dw-0 merged commit 1ef1f1b into mainsail-crew:master Jul 26, 2024
1 check passed
@dw-0
Copy link
Member

dw-0 commented Jul 26, 2024

Thank you!

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

Successfully merging this pull request may close these issues.

Docker container starts then immediately stops with error
2 participants