Hi RP,
On 06/04/2018 03:37 PM, Richard Purdie wrote:
On Mon, 2018-06-04 at 10:30 +0800, Robert Yang wrote:
On 06/03/2018 11:59 PM, Richard Purdie wrote:
On Wed, 2018-05-30 at 17:03 +0800, Robert Yang wrote:
From: Shan Hai <[email protected]>
The chkconfig fails to list the xinetd service because the xinetd
lacks Init Script Actions in it, add the actions to fix it.
Signed-off-by: Shan Hai <[email protected]>
Signed-off-by: Mingli Yu <[email protected]>
---
meta/recipes-extended/xinetd/xinetd/xinetd.init | 12
+++++++++++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/meta/recipes-extended/xinetd/xinetd/xinetd.init
b/meta/recipes-extended/xinetd/xinetd/xinetd.init
index 777c2c8..a021410 100644
--- a/meta/recipes-extended/xinetd/xinetd/xinetd.init
+++ b/meta/recipes-extended/xinetd/xinetd/xinetd.init
@@ -1,5 +1,15 @@
#!/bin/sh
-#
+### BEGIN INIT INFO
+# Provides: inetd
+# Required-Start: $local_fs $remote_fs
+# Required-Stop: $local_fs $remote_fs
+# Should-Start: $syslog
+# Should-Stop: $syslog
+# Default-Start: 2 3 4 5
+# Default-Stop: 0 1 6
+# Short-Description: Start or stop the xinetd daemon.
+### END INIT INFO
+
Have we some kind of policy which says all init scripts have to
have a
header like this? I'm wondering why this is just getting applied
for
xinetd when there are probably other init scripts which don't have
this. Are we planning to add this everywhere?
We added this is because customer requires, and only for this one. We
don't have any plan to add more.
I don't see the need to have this in OE-Core, it just duplicates
information from the recipe which can easily become outdated...
Yes, they may become outdated, but these info are required by lsb:
http://refspecs.linuxbase.org/LSB_3.1.0/LSB-Core-generic/LSB-Core-generic/initscrcomconv.html
The OE-Core also has other similar init scripts which have similar info:
$ find meta/recipes-* -name '*.init' -exec grep -H '# Provides:' {} \;
meta/recipes-core/dbus/dbus/dbus-1.init:# Provides: dbus
meta/recipes-core/sysvinit/sysvinit/bootlogd.init:# Provides: bootlogd
meta/recipes-devtools/tcf-agent/tcf-agent/tcf-agent.init:# Provides:
tcf-agent
meta/recipes-extended/watchdog/watchdog/wd_keepalive.init:# Provides:
wd_keepalive
meta/recipes-extended/cronie/cronie/crond.init:# Provides: crond crontab
And xinetd/xinetd.init is owned by oe-core, it seems that this is the
only way to fix it.
// Robert
Cheers,
Richard
--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core
