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

DOS-like line endings in release executables #318

Closed
williamboman opened this issue Dec 10, 2021 · 6 comments
Closed

DOS-like line endings in release executables #318

williamboman opened this issue Dec 10, 2021 · 6 comments
Labels
bug Something isn't working

Comments

@williamboman
Copy link

Describe the Bug

Starting in the 1.3.0 release, it seems like the bundled executables come with DOS-like line endings. This causes some issues when running on POSIX systems, for example interpreting the shebang incorrectly:

$ ./puppet-languageserver
env: ruby\r: No such file or directory

Expected Behavior

Unix-like line endings!

Steps to Reproduce

Steps to reproduce the behavior:

  1. Go to https://github.com/puppetlabs/puppet-editor-services/releases
  2. Download the either the .zip or .tar.gz from the 1.3.0 release.
  3. Unpack the archive.
  4. Review the line endings of the puppet-* scripts.

Environment

  • Version: 1.3.0
  • Platform: macOS
@williamboman williamboman added the bug Something isn't working label Dec 10, 2021
anhpt379 added a commit to anhpt379/dotfiles that referenced this issue Feb 3, 2022
The latest release has a line-ending problem, and it hasn't been
resolved yet:
puppetlabs/puppet-editor-services#318
anhpt379 added a commit to anhpt379/dotfiles that referenced this issue Feb 7, 2022
The latest release has a line-ending problem, and it hasn't been
resolved yet:
puppetlabs/puppet-editor-services#318
@dpmmb
Copy link

dpmmb commented Sep 14, 2022

As a work-around you can run dos2unix or similar on puppet-languageserver.

@acsezen
Copy link

acsezen commented Sep 30, 2022

Hello, dos2unix just convert the problem to this one;

.../vim/lsp/rpc.lua:420 "rpc" "puppet-languageserver" "stderr" "<internal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb>:85:in require': cannot load such file -- puppet_languageserver (LoadError)\n\tfrom internal:/usr/lib/ruby/vendor_ruby/rubygems/core_ext/kernel_require.rb:85:in require'\n\tfrom /home/asezen/.local/share/nvim/mason/bin/puppet-languageserver:11:in <main>\n

I installed puppet-languageserver with mason.

Thanks

@madelaney
Copy link

I have the same problem.

@bit0rez
Copy link
Contributor

bit0rez commented Jan 18, 2023

I have the same problem.
Looks like a bug in $LOAD_PATH for dependencies.
This is reproduced when I using a link to puppet-languageserver instead of starting it from the installation directory.

bit0rez added a commit to bit0rez/puppet-editor-services that referenced this issue Jan 18, 2023
bit0rez added a commit to bit0rez/puppet-editor-services that referenced this issue Jan 19, 2023
bit0rez added a commit to bit0rez/puppet-editor-services that referenced this issue Jan 19, 2023
bit0rez added a commit to bit0rez/puppet-editor-services that referenced this issue Jan 31, 2023
Using the "latest" tag is a bad idea.
In this case images was updated and dependencies was broken.
bit0rez added a commit to bit0rez/puppet-editor-services that referenced this issue Feb 14, 2023
williamboman added a commit to mason-org/mason-registry that referenced this issue Apr 26, 2023
williamboman added a commit to mason-org/mason-registry that referenced this issue Apr 26, 2023
@LukasAud
Copy link
Contributor

LukasAud commented Aug 21, 2023

Hi @williamboman, it looks like there was a Pull Request (#326) merged a couple of months ago that was described as a fix for this issue. Can you confirm whether this issue is still active or has been addressed?

@williamboman
Copy link
Author

Seems fine now 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants