Right now upstream supports openrc with a init.d file https://github.com/PurpleI2P/i2pd/tree/openssl/contrib/openrc and we install our own init.d & conf.d files: https://gitweb.gentoo.org/repo/gentoo.git/tree/net-vpn/i2pd/i2pd-2.56.0.ebuild#n60 I think merging our files together would benefit both projects. I am willing to do this work but before I do I would like someone to confirm that they will review my changes before I open up a PR upstream. That is also the case with systemd but since I do not use it I am not that eager to work on it.
Hi. It would be great if you merge the openrc files. I maintain i2pd in portage, but I don't really use openrc, so maintaining the openrc side is difficult. I'll be happy to look at the changes though. There's this bug which might be fixed along the way: https://bugs.gentoo.org/show_bug.cgi?id=949988
I have increased the importance as I think it improve the package significantly. Both 949988 & 808845 would have been avoided if we used upstream service files.
The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=b63d81cc7eee01fd502897d82767876ac52280fc commit b63d81cc7eee01fd502897d82767876ac52280fc Author: Filip Kobierski <fkobi@pm.me> AuthorDate: 2025-05-06 15:42:16 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2025-05-21 21:47:57 +0000 net-vpn/i2pd: add 2.56.0-r1 with improved openrc files Basically a rewrite at this point Added: - name - reload command - required_{dirs,files} - command_args_background - I2PD_LOGLEVEL - default USER & GROUP - confd file header Fixed: - pidfile usage - properly fix for #808845 - part of #949988 (bad shutdown) - not changing group Changed: - hardcode pidfile path to /run/i2pd/ - use command_progress instead of --progress - use command_user instead of --user - change `retry` timeouts to match upstream - always run with --daemon --service --log=file - rename OPTIONS to ARGS & add default value - rename LOG to LOGFILE & add default value - graceful command: - drop eval - use $retry - style - group subcommands stuff at the bottom - indent stuff - unquote known atomic constants - use " instead of ' for strings - confd: move unprefixed rc_ulimit to the bottom Removed: - use netmount - I2P_PID_DIR Bug: https://bugs.gentoo.org/808845 Bug: https://bugs.gentoo.org/949988 Bug: https://bugs.gentoo.org/955416 Signed-off-by: Filip Kobierski <fkobi@pm.me> Part-of: https://github.com/gentoo/gentoo/pull/41968 Closes: https://github.com/gentoo/gentoo/pull/41968 Signed-off-by: Sam James <sam@gentoo.org> net-vpn/i2pd/files/i2pd-2.56.0.confd | 16 ++++++++ net-vpn/i2pd/files/i2pd-2.56.0.initd | 53 +++++++++++++++++++++++++ net-vpn/i2pd/i2pd-2.56.0-r1.ebuild | 76 ++++++++++++++++++++++++++++++++++++ 3 files changed, 145 insertions(+)
