Skip to content

fix: PassWall/PassWall2 install deps check and i18n lazy download#28

Open
HouJia wants to merge 3 commits into
slobys:mainfrom
HouJia:fix/passwall2-install-deps-lang-v2
Open

fix: PassWall/PassWall2 install deps check and i18n lazy download#28
HouJia wants to merge 3 commits into
slobys:mainfrom
HouJia:fix/passwall2-install-deps-lang-v2

Conversation

@HouJia

@HouJia HouJia commented May 7, 2026

Copy link
Copy Markdown

Summary

Fixes installation failures on OpenWrt/iStoreOS 24.10 where opkg cannot resolve PassWall / PassWall2 dependencies from default feeds alone.

Changes

  • passwall2.sh: Install tcping and geoview from passwall_packages only when missing; log explicit skip/install steps.
  • passwall.sh: Install chinadns-ng and dns2socks only when missing (same pattern).
  • Both: Install main LuCI package first; handle zh-cn i18n after checking bundled resources / existing package; keep LANG_IPK="" with lazy assignment to avoid wasted downloads and file clashes.

Testing

  • Verified on iStoreOS 24.10.6 (aarch64_generic): PassWall and PassWall2 installers complete successfully with expected terminal logs.

Notes

Fork branch: fix/passwall2-install-deps-lang-v2.

HouJia and others added 3 commits May 8, 2026 01:37
Install tcping/geoview before luci-app-passwall2 and skip standalone zh-cn package when the main package already bundles Chinese resources, preventing dependency failures and i18n file conflicts seen on iStoreOS 24.10.

Co-authored-by: Cursor <cursoragent@cursor.com>
Check dependency packages before installing, print explicit install/skip status during execution, and avoid unnecessary i18n installs when language packages are already bundled or present.

Co-authored-by: Cursor <cursoragent@cursor.com>
Restore explicit LANG_IPK declaration for readability while preserving on-demand language package download logic.

Co-authored-by: Cursor <cursoragent@cursor.com>
@slobys
slobys force-pushed the main branch 7 times, most recently from acd3dcd to 18760bf Compare May 13, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant