luci-app-adblock-fast: prepare migration to APK

[stangri]

Maintainer: me
Compile tested: x86_64, Sophos XG-135r3, OpenWrt 23.05.2

[aparcar]

Is there a reason you don't set it to PKG_RELEASE=x?

[stangri]

Is there a reason you don't set it to PKG_RELEASE=x?

Sorry, missed your comment! Thank you -- first time it's been brought up regarding my luci apps, even when others contributed, so I was not aware of that variable being available for luci apps. I'll update all of the luci apps in my repo now and the fix be merged next time I create a feature PR.

[stangri]

Is there a reason you don't set it to PKG_RELEASE=x?

@aparcar I've changed all my luci packages to include PKG_RELEASE and now I remembered why I didn't do it originally -- the produced IPK file for luci applications doesn't seem to include the PKG_RELEASE in the filename, at least with 23.05 SDK.

So with the same

PKG_VERSION:=1.1.1
PKG_RELEASE:=r7

the produced IPKs are:

adblock-fast_1.1.1-r7_all.ipk
luci-app-adblock-fast_1.1.1_all.ipk

Since you're working on the include files for the APK prep, any chance you could have a look at the luci.mk (or wherever the version number for the resulting IPK file is set) to fix that?

[aparcar]

I have no strong opinion here except that I'd prefer to not complicate things. Your current solution seems fine, not needing to use PKG_RELEASE. I wasn't aware that it's ignored by luci.mk.