-
-
Notifications
You must be signed in to change notification settings - Fork 151
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
_build/oils.sh requires bash, but should only require /bin/sh (build/common.sh ) #1900
Comments
Yeah sorry there are 2 tarballs which is confusing now I recommend installing the There is a single binary https://www.oilshell.org/blog/2023/03/rename.html#where-are-they-used It is rough now, so if this is still not clear, then I recommend waiting a few months ... we recently switched to I think @Melkor333 uses it with Termux though |
So, I had actually already downloaded and extracted the oils-for-unix tarball, so that is fortuitous and bodes well for the install. Configure worked. I then tried Actioning your subsequent advice, I read: 'README-native.txt'. I then executed: ❯ pwd
/data/data/com.termux/files/home/oils-for-unix-0.21.0
❯ vim README-native.txt
❯ sh _build/oils.sh
_build/oils.sh: 80: build/common.sh: Syntax error: Bad function name This is the context of line 80: ...
78 _compile_one "$compiler" "$variant" "" \
79 _gen/cpp/build_stamp.cc "_build/obj/$compiler-$variant-sh/_gen/cpp/build_stamp.o"
80
81 _compile_one "$compiler" "$variant" "" \
82 _gen/data_lang/nil8.asdl.cc "_build/obj/$compiler-$variant-sh/_gen/data_lang/nil8.asdl.o"
83
84 _compile_one "$compiler" "$variant" "" \
85 _gen/data_lang/pretty.asdl.cc "_build/obj/$compiler-$variant-sh/_gen/data_lang/pretty.asdl.o"
... Knowing what I should change, or edit, is a bit beyond my nooby pay grade. It is also a bit odd, as there is no code on line 80. |
FYI: I shot Samuel an email. |
Oh can you try If bash is on your machine, it should work
We should relax this -- thanks for testing! |
Oh, I totally misread that output, that it was about common.sh, instead of oils.sh. Well, I ventured execution of the script with bash, instead of sh (which is symlinked to dash on Termux, following Debian):
I then restarted Termux, and executed: |
I think you forgot to
After that it should work :) |
I wish to install your shell in Termux, but your documentation confuses me greatly. Is the shell called: osh, ysh, oils or oils-for-unix? This confuses me greatly. Also, I am unrooted.
BTW, I hope the name of the shell is
ysh
, as it sounds like the name of a Un*x/*NIX shell.Any assistance greatly appreciated!
Cheers
The text was updated successfully, but these errors were encountered: