Skip to content
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

Pin openwrt release to 24.10.0-rc5 to work around APK/OPKG incompatibility #211

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

knazarov
Copy link

@knazarov knazarov commented Jan 9, 2025

See #204. The latest snapshots of OpenWrt have transitioned to using APK instead of OPKG for package management, and this installer relies on OPKG. So the quick workaround is to explicitly pin the latest version of OpenWrt that still used OPKG.

Eventually, it will make sense to port this installer to APK, but at the moment there are problems with post-install scripts of some packages that we depend on, namely openwrt/openwrt#17439

So for the time being, the best strategy to keep this repo working is to just pin the version.

…ility

See dangowrt#204. The latest snapshots of OpenWrt have transitioned to using APK
instead of OPKG for package management, and this installer relies on
OPKG. So the quick workaround is to explicitly pin the latest version
of OpenWrt that still used OPKG.

Eventually, it will make sense to port this installer to APK, but at
the moment there are problems with post-install scripts of some packages
that we depend on, namely openwrt/openwrt#17439

So for the time being, the best strategy to keep this repo working is
to just pin the version.
@knazarov knazarov changed the title Pin openwrt release to 24.10.0-rc5 to work around APK/OPKG incompatib… Pin openwrt release to 24.10.0-rc5 to work around APK/OPKG incompatibility Jan 9, 2025
@KA2107
Copy link

KA2107 commented Jan 23, 2025

build_installer.sh is failing for OpenWrt 24.10.0-rc6.

Removing package kmod-mt7622-firmware from root...
Removing package kmod-mt7915-firmware from root...
Removing package kmod-mt7615e from root...
Removing package kmod-mt7615-common from root...
Removing package kmod-mt7915e from root...
Removing package kmod-mt76-connac from root...
Removing package kmod-mt76-core from root...
Removing package odhcp6c from root...
Removing package odhcpd-ipv6only from root...
Removing package ppp from root...
Removing package ppp-mod-pppoe from root...
Removing package wpad-basic-mbedtls from root...
Collected errors:
 * print_dependents_warning: Package firewall4 is depended upon by packages:
 * print_dependents_warning: 	luci-app-firewall
 * print_dependents_warning: These might cease to work if package firewall4 is removed.

 * print_dependents_warning: Force removal of this package with --force-depends.
 * print_dependents_warning: Force removal of this package and its dependents
 * print_dependents_warning: with --force-removal-of-dependent-packages.
 * print_dependents_warning: Package kmod-nft-offload is depended upon by packages:
 * print_dependents_warning: 	firewall4
 * print_dependents_warning: These might cease to work if package kmod-nft-offload is removed.

 * print_dependents_warning: Force removal of this package with --force-depends.
 * print_dependents_warning: Force removal of this package and its dependents
 * print_dependents_warning: with --force-removal-of-dependent-packages.

Seems to be due to this line

OPENWRT_REMOVE_PACKAGES=(firewall4 kmod-mt7622-firmware kmod-mt7915-firmware kmod-mt7615e kmod-mt7615-common kmod-mt7915e kmod-mt76-connac kmod-mt76-core kmod-firewall4 nftables kmod-nft-offload odhcp6c odhcpd-ipv6only ppp ppp-mod-pppoe wpad-basic-mbedtls)

@dangowrt Ping.

@dangowrt
Copy link
Owner

Got to add luci-ssl, luci-app-firewall and kmod-nft-offload to the list of packages to remove from the installer image. Or no longer remove firewall4. It's all cosmetics in the end to safe a few kB on the installer image...

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.

3 participants