Skip to content

Binary does not work with WSL #11

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

Closed
DiegoBM opened this issue Mar 5, 2019 · 12 comments
Closed

Binary does not work with WSL #11

DiegoBM opened this issue Mar 5, 2019 · 12 comments

Comments

@DiegoBM
Copy link

DiegoBM commented Mar 5, 2019

  • vscode-remote version: 1.31.0-20
  • OS Version: Windows 10 (rel 1809) - WSL Ubuntu 16.04.4 LTS

Steps to Reproduce

  1. Run WSL and try to run the executable
  2. Error: -bash: ./code-server: cannot execute binary file: Exec format error
@kylecarbs
Copy link
Member

Will look into this. Thanks for the report.

@nikeinikei
Copy link

nikeinikei commented Mar 5, 2019

can confirm, didn't work for me either, with same error message

@kylecarbs
Copy link
Member

Seems like binaries compressed with UPX do not work in WSL.

microsoft/WSL#2121 (comment)

We'll create a version in the CI that isn't compressed. You can try decompressing it, might work.

@DiegoBM
Copy link
Author

DiegoBM commented Mar 5, 2019

@kylecarbs running the upx -d command on the binary it says that the binary was not packed by upx

# upx -d code-server
                       Ultimate Packer for eXecutables
                          Copyright (C) 1996 - 2013
UPX 3.91        Markus Oberhumer, Laszlo Molnar & John Reiser   Sep 30th 2013

        File size         Ratio      Format      Name
   --------------------   ------   -----------   -----------
upx: code-server: NotPackedException: not packed by UPX

Unpacked 0 files.

@kylecarbs
Copy link
Member

kylecarbs commented Mar 5, 2019 via email

@nhooyr nhooyr added the ci label Mar 8, 2019
@chintanp
Copy link

chintanp commented Mar 12, 2019

Running from docker on WSL would be good. Right now I use Cloud9 local install through https://github.com/c9/core for Python development. Wonder if Coder can provide better intellisense.

@DiegoBM
Copy link
Author

DiegoBM commented Mar 14, 2019

Any chance to disable that UPX compression in the next release?

@SuperSandro2000
Copy link
Contributor

Any chance to disable that UPX compression in the next release?

The plan was to add an additional target and not remove an optimization.

@DiegoBM
Copy link
Author

DiegoBM commented Mar 14, 2019

The plan was to add an additional target and not remove an optimization.

@SuperSandro2000 who's saying otherwise? It was already mentioned by @kylecarbs that it would be an additional target

@code-asher
Copy link
Member

code-asher commented Mar 15, 2019

We ended up skipping compression altogether (from 1.32.0-245 onward): 8bab787.

@DiegoBM
Copy link
Author

DiegoBM commented Mar 15, 2019

@coderasher, was compression causing other unexpected problems? Out of curiosity

@DiegoBM
Copy link
Author

DiegoBM commented Mar 15, 2019

Now it works flawlessly on WSL, thank you! Closing as fixed

@DiegoBM DiegoBM closed this as completed Mar 15, 2019
code-asher pushed a commit that referenced this issue Jun 19, 2019
* Create initial server layout

* Adjust command name to entry

* Add @oclif/config as dependency

* Implement build process for outputting single binary

* Add init message

* Remove unused import, add tsconfig.json to .gitignore

* Accidently pushed wacky change to output host FS files

* Add options to createApp
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

No branches or pull requests

7 participants