-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
fix(scripts/setup-termux): install bison, not byacc
#24977
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
Conversation
- Partial fix (one part of the fix for) termux#21658 - Partially reverts termux@cd151f2 - The Ubuntu termux-package-builder docker container contains `bison`, **not** `byacc` https://github.com/termux/termux-packages/blob/dfb980969307f853d1ce6f3eab4b49358623526d/scripts/setup-ubuntu.sh#L33 - Many Termux packages, for one example `mesa`, cannot be built without having `bison` installed, rather than `byacc`
|
Thank you! I will merge it in 24 hours from now, and after this, if the package that requires |
thunder-coding
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, I don't do ondevice builds anymore at all now so can't test
- Fixes this error when building `nethack` on-device: `make: lex: No such file or directory` [no ci]
|
There are basically also some other mismatches between In the repository, there are only two packages that reference |
Partial fix (one part of the fix for) [Bug]: mesa & mesa-vulkan-icd-freedreno-dri3 build error #21658
Partially reverts cd151f2
The Ubuntu termux-package-builder docker container contains
bison, notbyacctermux-packages/scripts/setup-ubuntu.sh
Line 33 in dfb9809
mesa, cannot be built without havingbisoninstalled, rather thanbyacc