-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
treewide: fix procd service inactive #19617
Conversation
Exit directly will result procd service inactive and uci configuration changes are no longer monitored. Reported-by: Lvc Revincx <revincx233@gmail.com> Signed-off-by: Tianling Shen <cnsztl@immortalwrt.org>
That is…interesting. Given that AUTORELEASE doesn’t really work with treewide, I think it makes more sense to make multiple commits with each package. wonder how many other packages are affected… |
https://github.com/openwrt/packages/actions/runs/3265617495/jobs/5368113799#step:8:54
Yes, that's what I actually did before.
adguardhome and hs20 at least. EDIT:
|
Thanks to @1715173329 I've just fixed simple-adblock. PR pending. |
* make PKG_VERSION of the init script readonly to remove shellcheck exception * replace exit with return in the the procd scripts per: openwrt#19617 * remove custom boot() function as it prevented creation of procd firewall object on start on boot Signed-off-by: Stan Grishin <stangri@melmac.ca>
* make PKG_VERSION of the init script readonly to remove shellcheck exception * replace exit with return in the the procd scripts per: openwrt#19617 * remove custom boot() function as it prevented creation of procd firewall object on start on boot Signed-off-by: Stan Grishin <stangri@melmac.ca> (cherry picked from commit d691408)
* make PKG_VERSION of the init script readonly to remove shellcheck exception * replace exit with return in the the procd scripts per: openwrt#19617 * remove custom boot() function as it prevented creation of procd firewall object on start on boot * improve performance of allowing domains code Signed-off-by: Stan Grishin <stangri@melmac.ca>
* make PKG_VERSION of the init script readonly to remove shellcheck exception * replace exit with return in the the procd scripts per: openwrt#19617 * remove custom boot() function as it prevented creation of procd firewall object on start on boot * improve performance of allowing domains code Signed-off-by: Stan Grishin <stangri@melmac.ca> (cherry picked from commit 3ba5455)
* make PKG_VERSION of the init script readonly to remove shellcheck exception * replace exit with return in the the procd scripts per: openwrt#19617 * remove custom boot() function as it prevented creation of procd firewall object on start on boot * improve performance of allowing domains code Signed-off-by: Stan Grishin <stangri@melmac.ca>
* make PKG_VERSION of the init script readonly to remove shellcheck exception * replace exit with return in the the procd scripts per: openwrt#19617 * remove custom boot() function as it prevented creation of procd firewall object on start on boot * improve performance of allowing domains code Signed-off-by: Stan Grishin <stangri@melmac.ca> (cherry picked from commit 3ba5455)
Maintainer: me
Compile tested: N/A
Run tested: NanoPi R2S
Description:
Exit directly will result procd service inactive and uci configuration changes are no longer monitored.