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

commands (jsreport-studio-build, jsreport-studio-start) not working on OSX #1

Merged
merged 1 commit into from
Jul 2, 2016

Conversation

bjrmatos
Copy link
Collaborator

issue encountered while trying to publish jsreport-jade

command-bug

seems like a problem with the space in the hashbang, with this change i have been able to build normally

@pofider pofider merged commit 1f3eb8d into jsreport:master Jul 2, 2016
@bjrmatos
Copy link
Collaborator Author

bjrmatos commented Jul 2, 2016

same issue :(

issue

seems like the problem is windows CRLF line endings \r, changing CRLF to LF has solved the problem, seems like you will need to ensure that the CLI file is published with unix line endings.

solution

@pofider
Copy link
Contributor

pofider commented Jul 2, 2016

Ah, sorry, didn't know about this. It would be nice - npm/npm#12371

I'll change to LF, publish again, test it on ubuntu VM and let you know

@bjrmatos
Copy link
Collaborator Author

bjrmatos commented Jul 2, 2016

thnk you!

@pofider
Copy link
Contributor

pofider commented Jul 3, 2016

Release in 1.0.2, I've tested it on ubuntu and it seems to be working.

It would take me ages to figure out this line endings issue. Thank you very much for pointing it out.

@bjrmatos
Copy link
Collaborator Author

bjrmatos commented Jul 3, 2016

thnks! happy to help 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.

2 participants