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

Update WSL troubleshooting in README.md #585

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

halter73
Copy link

This instructs WSL users they need to install /usr/bin/ape in order to use cosmo-binfmt.service.

I'm not an expert on αcτµαlly pδrταblε εxεcµταblε's or binfmt_misc, but I needed to install the APE loader to /usr/bin/ape as suggested in the Linux troubleshooting section in order to get cosmo-binfmt.service to work on my WSL 2 Ubuntu installation.

I also added the --now flag to the systemctl enable command for convenience.

The Linux troubleshooting section adds entries with both MZqFpD and jartsr as magic strings, so it's tempting to do the same in cosmo-binfmt.service for consistency, but I didn't need the jartsr entry for the Llama-3.2-3B-Instruct.F16.llamafile I tested this with, so I'm guessing that's only necessary for legacy executables and left it out.

@jart Does this change make sense to you? I could submit a PR to update the https://github.com/jart/cosmopolitan README to suggest adding the cosmo-binfmt.service too. It's nice to be able to support both APE and Windows binaries on WSL.

Instruct WSL users to install /usr/bin/ape to use binfmt service
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant