-
Notifications
You must be signed in to change notification settings - Fork 71
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
Modify Laucher and Executable names #75
Comments
same symptoms as for #74 and probably a good proposal for a solution |
Same issue as in #73 and the proposal seems like a solution. |
sanbee
pushed a commit
to sanbee/exodus_sbfork
that referenced
this issue
Jul 3, 2023
Seems to fix the problem reported in intoli#73.
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
--version
option in exodus)/usr/bin/kmod
readelf -a /usr/bin/kmod
output = readelf-kmod.txtI created an exodus bundle for kmod program using the following command:
kmod and its symlinks failed to run, terminating with 254 status code. kmod verifies its
argv[0]
usingprogram_invocation_short_name()
to determine what it will run as, kind of like busybox. To fix this, I renamed launcher scripts inexodus/bundles/3d77fd2.../<launcher>
to add a.sh
suffix. Then I renamedexodus/bundles/3d77fd2.../<progname>-x
to remove-x
suffix, hence regaining their original names. I edited the launcher scripts to remove the-x
suffix accordingly. This is why I used--shell-launchers
instead of binary ones. I then re-symlinked launcher scripts fromexodus/bundles/3d77fd2.../<launcher>.sh
toexodus/bin/<progname>
so they would point to they newly renamed launchers. kmod worked just fine after that.My suggestion is that launcher files in
exodus/bundles/<hash>/<launcher>
have a.sh
suffix if shell-based or.bin
suffix if binary. This way the ported programs inexodus/bundles/<hash>/<progname>
will have their original names, incase they use it to determine what to run as. The symlinks to launchers inexodus/bin/<progname>
will also have the same original names as the programs ported, so user experience will not be impacted. Also, a--version
option would be great in exodus.The text was updated successfully, but these errors were encountered: