gandalf/secubox-openwrt
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(secubox): make postinst script non-failing and add auto-recovery

Browse Source 
The postinst script was failing installation (exit 1) when the appstore
catalog wasn't found, which prevented package upgrades from completing.

Changes:
1. Removed exit 1 - installation now continues even if file is missing
2. Added auto-recovery: tries to restore from /rom overlay if available
3. Provides clear status feedback (✓ or ✗) for appstore catalog
4. Gives helpful error message with recovery instructions
5. Always exits with 0 to allow installation to complete

This fixes upgrades from 0.7.0-r2 to 0.7.0-r4 where the directory
structure changed from .appstore to appstore.

Version: 0.7.0-4

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
CyberMind-FR 2025-12-31 13:17:07 +01:00
parent c3cc47bb40
commit 43568ef41b
1 changed files with 22 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
luci-app-secubox/Makefile
View File

@ -2,7 +2,7 @@ include $(TOPDIR)/rules.mk
PKG_NAME:=luci-app-secubox
PKG_VERSION:=0.7.0
PKG_RELEASE:=3
PKG_RELEASE:=4
PKG_LICENSE:=Apache-2.0
PKG_MAINTAINER:=CyberMind <contact@cybermind.fr>
@ -46,14 +46,24 @@ define Package/$(PKG_NAME)/postinst
# Ensures appstore catalog and permissions are correct
[ -n "$${IPKG_INSTROOT}" ] || {
# Verify appstore catalog exists
# Check if appstore catalog exists
if [ ! -f /usr/share/secubox/appstore/apps.json ]; then
echo "WARNING: SecuBox appstore catalog not found!"
echo "Expected: /usr/share/secubox/appstore/apps.json"
exit 1
echo "WARNING: SecuBox appstore catalog not found at /usr/share/secubox/appstore/apps.json"
# Try to restore from package ROM overlay
if [ -f /rom/usr/share/secubox/appstore/apps.json ]; then
echo " Restoring from package overlay..."
mkdir -p /usr/share/secubox/appstore
cp /rom/usr/share/secubox/appstore/apps.json /usr/share/secubox/appstore/apps.json
echo " ✓ Restored appstore catalog"
else
echo " ERROR: Appstore catalog not found in package!"
echo " The app store will be empty until this is fixed."
echo " Try: opkg remove luci-app-secubox && opkg install luci-app-secubox"
fi
fi
# Set proper permissions
# Set proper permissions (safe even if files don't exist)
chmod 755 /usr/share/secubox 2>/dev/null || true
chmod 755 /usr/share/secubox/appstore 2>/dev/null || true
chmod 644 /usr/share/secubox/appstore/apps.json 2>/dev/null || true
@ -63,8 +73,12 @@ define Package/$(PKG_NAME)/postinst
/etc/init.d/rpcd reload 2>/dev/null || true
echo "SecuBox: Installation complete"
echo " - Appstore catalog: /usr/share/secubox/appstore/apps.json"
echo " - RPCD methods available via: ubus call luci.secubox"
if [ -f /usr/share/secubox/appstore/apps.json ]; then
echo " ✓ Appstore catalog: /usr/share/secubox/appstore/apps.json"
else
echo " ✗ Appstore catalog: MISSING"
fi
echo " ✓ RPCD methods available via: ubus call luci.secubox"
}
exit 0
endef