php8: must build all modules or build fails
Open
Nobody has claimed this yet.
bug
release/24.10
- Dominant language
- Makefile
- Stars
- 4.6k
- Forks
- 4k
- Avg merge
- 3d 12h
- Merged PRs (30d)
- 134
Description
Package Name
php8
Maintainer
@mhei
OpenWrt Version
24.10.6
OpenWrt Target/Subtarget
mediatek/filogic
Steps to Reproduce
- Extract SDK for 24.10-SNAPSHOT
- `./scripts/feeds update -a && ./scripts/feeds -install -p packages -f zabbix
- make menuconfig
- Select all Zabbix, and only top-level php8
- Select zabbix frontend as well
- make download && make
- PHP8 build fails with missing library
- Add the library and build
- PHP8 build fails with another missing library
- Add all modules and build
- PHP8 succeeds as does the rest of the build
Actual Behaviour
The dependencies for the modules appear to be needed by the main compile, even if the modules that require them are not selected for build (e.g. php8-mod-gmp, php8-mod-sodium, etc).
Confirmation Checklist
- The package is maintained in this repository.
- I understand that issues related to the base OpenWrt repository or LuCI repository will be closed.
- I am reporting an issue for OpenWrt, not an unsupported fork.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reproducing the reported php8 build in the OpenWrt 24.10 SDK with only top-level php8 and the selected Zabbix packages, then inspect the php8 package and module dependency declarations alongside the build errors. Done means php8 and the selected packages build successfully without selecting every php8 module or manually adding missing libraries.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- build-system
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100