post-install script is failing about easywsl HOT 10 CLOSED

Okay, I see. Still, thanks for your PR @scadu!
We basically have to do some code cleaning up soon... we will do it with the next release (while also adding other features, stay tuned).

from easywsl.

@mikenelson-io Indeed, it's a correct file. I'll provide a fix in another PR.

from easywsl.

@scadu hi, sorry for the late reply.
I've just added build instructions to README in 1aa5825

from easywsl.

This is still an issue. The new post-install.sh file is also erroring.
ArchLinux:

from easywsl.

Looks like there's another post-install.sh in the Release directory which as I can see was generated during the build process, committed to the repository, and you actually use the version from the last build.
I guess it's time to think about a CI/CD pipeline 😄.
cc @unrooted

from easywsl.

I copy/pasted the raw content of the post-install.sh that is located in the root of easyWSL/easyWSL/. That was the one recently modified. It has the addition of #!/usr/bin/env sh in line 1. Is that not the correct file?

from easywsl.

@mikenelson-io I checked the script once again, and it works correctly. For some reason, I ignored that you copied the script focusing on the mysterious No such file or directory for /usr/bin/env.
While copying, line endings have changed from LF (*nix) to CRLF (Windows) resulting in a file that's not interpreted correctly by the *nix shell. If you copy the script and save the file with the LF line endings, it should be better (VS Code, Notepad++ provides an option to change the file line endings easily).

from easywsl.

I just slapped myself for not seeing that 🤦
It does indeed work with LF. Thank you for clarifying!

from easywsl.

Thanks, @scadu, for helping us with the post install script. We've cleaned up the project and the corrected the line endings too. You can build the master branch and see if it works as intended. We are planning to make a new release soon, but before that we want to test everything pretty extensively.
If that's doable, can you please guys check whether it works right now, after the latest update when you build from the source code yourselves?

from easywsl.

@unrooted Is it possible to add a build procedure to the readme? I wouldn't like to guess trying to work out a correct setup.

from easywsl.