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

"olv.sh" doesn't run in Debian GNU/Linux #635

Open
negora opened this issue May 12, 2022 · 3 comments · May be fixed by #637
Open

"olv.sh" doesn't run in Debian GNU/Linux #635

negora opened this issue May 12, 2022 · 3 comments · May be fixed by #637
Labels
Milestone

Comments

@negora
Copy link

negora commented May 12, 2022

Description.

When I try to run olv.sh in Debian GNU/Linux, I get this error:

/bin/bash^M: bad interpreter: No such file or directory

It has turned to be because the file has CRLF line endings (Windows) instead of LF ones (Linux).

Workaround.

I've run this Perl script to solve it:

perl -p -i -e 's/\r\n/\n/' olv.sh

Technical specifications.

  • Debian GNU/Linux 11.3.
  • OpenJDK Runtime Environment (build 17.0.3+7-Debian-1deb11u1)
  • OtrosLogViewer 1.14.18.

Thank you.

@svennissel svennissel added the bug label May 12, 2022
@svennissel svennissel modified the milestones: 1.4.19, 1.5.0 May 12, 2022
@svennissel
Copy link
Collaborator

The problem was my local window pc. It used \r\n and git do not used updated version from git with \n. Has someone a idea to fix it automatic with gradle build one building distribution zip?

@negora
Copy link
Author

negora commented May 22, 2022

Does it help? What is the easiest way convert line endings to unix style when creating tar file in Gradle?. The thread is about Tar but I guess it's applicable to Zip too. I don't use Gradle, so just a guess...

@svennissel
Copy link
Collaborator

I added a test case to check the files. It is not the best solution but better than nothing

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.

2 participants