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.

Sign up for GitHub

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

Jump to bottom

antlr4ts bin script published with CRLF line endings #250

Closed
carlansley opened this issue Dec 13, 2016 · 3 comments
Closed

antlr4ts bin script published with CRLF line endings #250

carlansley opened this issue Dec 13, 2016 · 3 comments
Assignees
@sharwell
Labels
bug fixed
Milestone
0.4.0-alpha.2

Comments

@carlansley
Copy link

This causes issues on MacOS/Linux etc:

> ./node_modules/.bin/antlr4ts 
env: node\r: No such file or directory
@carlansley
Copy link
Author

Assuming antlr4ts-cli is being published from Windows, likely related to npm/npm#12371.

@thosakwe
Copy link

thosakwe commented Dec 16, 2016 •
edited
Loading

Having this error as well. For now, I think we will just have to build from source ourselves.

EDIT: That doesn't work, either.

@sharwell
Copy link
Member

@carlansley Thanks for identifying this and taking the time to file a bug. I'll get right on it!

@sharwell sharwell self-assigned this Dec 18, 2016
sharwell added a commit to sharwell/antlr4ts that referenced this issue Dec 18, 2016
@sharwell
Fix the antlr4ts script for use on Linux systems
744bdc3 
Mark the file as executable, and force the line endings to LF (even when
cloned on Windows).

Fixes tunnelvisionlabs#250
@sharwell sharwell mentioned this issue Dec 18, 2016
Merged
@sharwell sharwell added this to the 0.4.0-alpha.2 milestone Dec 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sharwell sharwell

Labels
bug fixed
Projects
None yet
Milestone
0.4.0-alpha.2
Development

No branches or pull requests
3 participants
@carlansley @sharwell @thosakwe