* [PATCH V3 0/2] Split the functions script into a separate package in initscripts @ 2014-01-06 7:27 Chen Qi 2014-01-06 7:27 ` [PATCH V3 1/2] initscripts: split the functions script into a separate package Chen Qi 2014-01-06 7:27 ` [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions Chen Qi 0 siblings, 2 replies; 9+ messages in thread From: Chen Qi @ 2014-01-06 7:27 UTC (permalink / raw) To: openembedded-core Changes since V2: 1. Fix the FILES for the initscripts-functions package 2. Rebase against the latest master //Chen Qi The following changes since commit f33c990db392c6b208efecea0f941449c4d25cec: maintainers.inc: reassign e2fsprogs (2014-01-03 15:16:42 +0000) are available in the git repository at: git://git.pokylinux.org/poky-contrib ChenQi/initscripts-functions http://git.pokylinux.org/cgit.cgi/poky-contrib/log/?h=ChenQi/initscripts-functions Chen Qi (2): initscripts: split the functions script into a separate package Add missing RDEPENDS of initscripts-functions meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + .../irda-utils/irda-utils_0.9.18.bb | 2 ++ .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + meta/recipes-core/dbus/dbus.inc | 2 +- meta/recipes-core/initscripts/initscripts_1.0.bb | 8 ++++++-- meta/recipes-core/systemd/systemd_208.bb | 2 +- meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- meta/recipes-extended/at/at_3.1.14.bb | 3 +-- meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + .../oprofile/oprofileui-server_git.bb | 2 +- 18 files changed, 28 insertions(+), 13 deletions(-) -- 1.7.9.5 ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH V3 1/2] initscripts: split the functions script into a separate package 2014-01-06 7:27 [PATCH V3 0/2] Split the functions script into a separate package in initscripts Chen Qi @ 2014-01-06 7:27 ` Chen Qi 2014-01-06 7:27 ` [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions Chen Qi 1 sibling, 0 replies; 9+ messages in thread From: Chen Qi @ 2014-01-06 7:27 UTC (permalink / raw) To: openembedded-core Many SysV init scripts need the /etc/init.d/functions script. But this script is part of the initscripts package. As a result, the initscripts package should always be installed into the system to avoid errors when starting daemons. However, it makes no sense to install the initscripts package into a systemd based image, because what the init scripts provide has already been provided by the systemd. On the other hand, the functions script might be still needed in a systemd based image because other init scripts such as distcc might need it. The above situation leads to a natural separation of the functions script from the initscripts package. And this patch does so. It separates the functions script into initscripts-functions packages. Signed-off-by: Chen Qi <Qi.Chen@windriver.com> --- meta/recipes-core/initscripts/initscripts_1.0.bb | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/meta/recipes-core/initscripts/initscripts_1.0.bb b/meta/recipes-core/initscripts/initscripts_1.0.bb index 5951e0c..d1644a3 100644 --- a/meta/recipes-core/initscripts/initscripts_1.0.bb +++ b/meta/recipes-core/initscripts/initscripts_1.0.bb @@ -42,8 +42,12 @@ KERNEL_VERSION = "" inherit update-alternatives DEPENDS_append = " update-rc.d-native" -ALTERNATIVE_PRIORITY = "90" -ALTERNATIVE_${PN} = "functions" +PACKAGES =+ "${PN}-functions" +RDEPENDS_${PN} = "${PN}-functions" +FILES_${PN}-functions = "${sysconfdir}/init.d/functions*" + +ALTERNATIVE_PRIORITY_${PN}-functions = "90" +ALTERNATIVE_${PN}-functions = "functions" ALTERNATIVE_LINK_NAME[functions] = "${sysconfdir}/init.d/functions" HALTARGS ?= "-d -f" -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions 2014-01-06 7:27 [PATCH V3 0/2] Split the functions script into a separate package in initscripts Chen Qi 2014-01-06 7:27 ` [PATCH V3 1/2] initscripts: split the functions script into a separate package Chen Qi @ 2014-01-06 7:27 ` Chen Qi 2014-01-17 14:11 ` Paul Eggleton 1 sibling, 1 reply; 9+ messages in thread From: Chen Qi @ 2014-01-06 7:27 UTC (permalink / raw) To: openembedded-core Now that the initscripts-functions has been packaged separately, packages which may use the functions script should have a runtime dependency on it. Signed-off-by: Chen Qi <Qi.Chen@windriver.com> --- meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + .../irda-utils/irda-utils_0.9.18.bb | 2 ++ .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + meta/recipes-core/dbus/dbus.inc | 2 +- meta/recipes-core/systemd/systemd_208.bb | 2 +- meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- meta/recipes-extended/at/at_3.1.14.bb | 3 +-- meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + .../oprofile/oprofileui-server_git.bb | 2 +- 17 files changed, 22 insertions(+), 11 deletions(-) diff --git a/meta/recipes-bsp/alsa-state/alsa-state.bb b/meta/recipes-bsp/alsa-state/alsa-state.bb index 552fcfc..4be74aa 100644 --- a/meta/recipes-bsp/alsa-state/alsa-state.bb +++ b/meta/recipes-bsp/alsa-state/alsa-state.bb @@ -37,7 +37,7 @@ PACKAGES += "alsa-states" RRECOMMENDS_alsa-state = "alsa-states" -RDEPENDS_${PN} = "alsa-utils-alsactl" +RDEPENDS_${PN} = "alsa-utils-alsactl initscripts-functions" FILES_${PN} = "${sysconfdir}/init.d ${sysconfdir}/asound.conf" CONFFILES_${PN} = "${sysconfdir}/asound.conf" diff --git a/meta/recipes-bsp/apmd/apmd_3.2.2-14.bb b/meta/recipes-bsp/apmd/apmd_3.2.2-14.bb index 8c4b75e..d5ddc17 100644 --- a/meta/recipes-bsp/apmd/apmd_3.2.2-14.bb +++ b/meta/recipes-bsp/apmd/apmd_3.2.2-14.bb @@ -8,6 +8,7 @@ LICENSE = "GPLv2+" LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f \ file://apm.h;beginline=6;endline=18;md5=7d4acc1250910a89f84ce3cc6557c4c2" DEPENDS = "libtool-cross" +RDEPENDS_${PN} = "initscripts-functions" PR = "r2" SRC_URI = "${DEBIAN_MIRROR}/main/a/apmd/apmd_3.2.2.orig.tar.gz;name=tarball \ diff --git a/meta/recipes-connectivity/irda-utils/irda-utils_0.9.18.bb b/meta/recipes-connectivity/irda-utils/irda-utils_0.9.18.bb index 7f216ac..59c9b6a 100644 --- a/meta/recipes-connectivity/irda-utils/irda-utils_0.9.18.bb +++ b/meta/recipes-connectivity/irda-utils/irda-utils_0.9.18.bb @@ -24,6 +24,8 @@ inherit autotools update-rc.d INITSCRIPT_NAME = "irattach" INITSCRIPT_PARAMS = "defaults 20" +RDEPENDS_${PN} += "initscripts-functions" + do_compile () { oe_runmake -e -C irattach oe_runmake -e -C irdaping diff --git a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.9.bb b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.9.bb index 45eacd9..fd44ea1 100644 --- a/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.9.bb +++ b/meta/recipes-connectivity/nfs-utils/nfs-utils_1.2.9.bb @@ -9,8 +9,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=95f3a93a5c3c7888de623b46ea085a84" # util-linux for libblkid DEPENDS = "libcap libnfsidmap libevent util-linux sqlite3" -RDEPENDS_${PN}-client = "rpcbind bash" -RDEPENDS_${PN} = "${PN}-client bash" +RDEPENDS_${PN}-client = "rpcbind bash initscripts-functions" +RDEPENDS_${PN} = "${PN}-client bash initscripts-functions" RRECOMMENDS_${PN} = "kernel-module-nfsd" inherit useradd diff --git a/meta/recipes-connectivity/openssh/openssh_6.4p1.bb b/meta/recipes-connectivity/openssh/openssh_6.4p1.bb index 4f8d70a..c61d16f 100644 --- a/meta/recipes-connectivity/openssh/openssh_6.4p1.bb +++ b/meta/recipes-connectivity/openssh/openssh_6.4p1.bb @@ -122,7 +122,7 @@ FILES_${PN}-misc = "${bindir}/ssh* ${libexecdir}/ssh*" FILES_${PN}-keygen = "${bindir}/ssh-keygen" RDEPENDS_${PN} += "${PN}-scp ${PN}-ssh ${PN}-sshd ${PN}-keygen" -RDEPENDS_${PN}-sshd += "${PN}-keygen" +RDEPENDS_${PN}-sshd += "${PN}-keygen initscripts-functions" CONFFILES_${PN}-sshd = "${sysconfdir}/ssh/sshd_config" CONFFILES_${PN}-ssh = "${sysconfdir}/ssh/ssh_config" diff --git a/meta/recipes-connectivity/ppp/ppp_2.4.5.bb b/meta/recipes-connectivity/ppp/ppp_2.4.5.bb index 52bab4f..ed1a391 100644 --- a/meta/recipes-connectivity/ppp/ppp_2.4.5.bb +++ b/meta/recipes-connectivity/ppp/ppp_2.4.5.bb @@ -5,6 +5,7 @@ SECTION = "console/network" HOMEPAGE = "http://samba.org/ppp/" BUGTRACKER = "http://ppp.samba.org/cgi-bin/ppp-bugs" DEPENDS = "libpcap" +RDEPENDS_${PN} = "initscripts-functions" LICENSE = "BSD & GPLv2+ & LGPLv2+ & PD" LIC_FILES_CHKSUM = "file://pppd/ccp.c;beginline=1;endline=29;md5=e2c43fe6e81ff77d87dc9c290a424dea \ file://pppd/plugins/passprompt.c;beginline=1;endline=10;md5=3bcbcdbf0e369c9a3e0b8c8275b065d8 \ diff --git a/meta/recipes-core/dbus/dbus.inc b/meta/recipes-core/dbus/dbus.inc index ae80cb0..9bdb489 100644 --- a/meta/recipes-core/dbus/dbus.inc +++ b/meta/recipes-core/dbus/dbus.inc @@ -6,7 +6,7 @@ LICENSE = "AFL-2 | GPLv2+" LIC_FILES_CHKSUM = "file://COPYING;md5=10dded3b58148f3f1fd804b26354af3e \ file://dbus/dbus.h;beginline=6;endline=20;md5=7755c9d7abccd5dbd25a6a974538bb3c" DEPENDS = "expat virtual/libintl" -RDEPENDS_dbus = "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest-ptest', '', d)}" +RDEPENDS_dbus = "${@base_contains('DISTRO_FEATURES', 'ptest', 'dbus-ptest-ptest', '', d)} initscripts-functions" RDEPENDS_dbus_class-native = "" RDEPENDS_dbus_class-nativesdk = "" diff --git a/meta/recipes-core/systemd/systemd_208.bb b/meta/recipes-core/systemd/systemd_208.bb index dcf68dc..6590235 100644 --- a/meta/recipes-core/systemd/systemd_208.bb +++ b/meta/recipes-core/systemd/systemd_208.bb @@ -229,7 +229,7 @@ PACKAGES =+ "udev-dbg udev udev-utils udev-hwdb" FILES_udev-dbg += "/lib/udev/.debug" -RDEPENDS_udev += "udev-utils" +RDEPENDS_udev += "udev-utils initscripts-functions" RPROVIDES_udev = "hotplug" RRECOMMENDS_udev += "udev-hwdb" diff --git a/meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb b/meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb index dd25f71..a84448a 100644 --- a/meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb +++ b/meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb @@ -7,7 +7,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \ file://COPYRIGHT;endline=15;md5=349c872e0066155e1818b786938876a4" PR = "r11" -RDEPENDS_${PN} = "${PN}-inittab" +RDEPENDS_${PN} = "${PN}-inittab initscripts-functions" SRC_URI = "http://download.savannah.gnu.org/releases-noredirect/sysvinit/sysvinit-${PV}.tar.bz2 \ file://install.patch \ diff --git a/meta/recipes-devtools/distcc/distcc_3.1.bb b/meta/recipes-devtools/distcc/distcc_3.1.bb index 5c889fa..e351a58 100644 --- a/meta/recipes-devtools/distcc/distcc_3.1.bb +++ b/meta/recipes-devtools/distcc/distcc_3.1.bb @@ -6,6 +6,7 @@ LICENSE = "GPLv2" LIC_FILES_CHKSUM = "file://COPYING;md5=94d55d512a9ba36caa9b7df079bae19f" DEPENDS = "avahi" +RDEPENDS_${PN} = "initscripts-functions" GTKCONFIG = "gtk" GTKCONFIG_libc-uclibc = "" diff --git a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb index 1eb0021..6d0fbb6 100644 --- a/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb +++ b/meta/recipes-devtools/tcf-agent/tcf-agent_git.bb @@ -15,7 +15,7 @@ SRC_URI = "git://git.eclipse.org/gitroot/tcf/org.eclipse.tcf.agent.git \ " DEPENDS = "util-linux openssl" -RDEPENDS_${PN} = "bash" +RDEPENDS_${PN} = "bash initscripts-functions" S = "${WORKDIR}/git" diff --git a/meta/recipes-extended/at/at_3.1.14.bb b/meta/recipes-extended/at/at_3.1.14.bb index 1eb18b3..0a13157 100644 --- a/meta/recipes-extended/at/at_3.1.14.bb +++ b/meta/recipes-extended/at/at_3.1.14.bb @@ -7,9 +7,8 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=4325afd396febcb659c36b49533135d4" DEPENDS = "flex flex-native \ ${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}" -VIRTUAL-RUNTIME_initscripts ?= "initscripts" RDEPENDS_${PN} = "${@base_contains('DISTRO_FEATURES', 'pam', '${PAM_DEPS}', '', d)} \ - ${VIRTUAL-RUNTIME_initscripts} \ + initscripts-functions \ " PAM_DEPS = "libpam libpam-runtime pam-plugin-env pam-plugin-limits" diff --git a/meta/recipes-extended/cronie/cronie_1.4.11.bb b/meta/recipes-extended/cronie/cronie_1.4.11.bb index d98d0d9..c698181 100644 --- a/meta/recipes-extended/cronie/cronie_1.4.11.bb +++ b/meta/recipes-extended/cronie/cronie_1.4.11.bb @@ -39,6 +39,8 @@ PACKAGECONFIG[pam] = "--with-pam,--without-pam,libpam,${PAM_DEPS}" INITSCRIPT_NAME = "crond" INITSCRIPT_PARAMS = "start 90 2 3 4 5 . stop 60 0 1 6 ." +RDEPENDS_${PN} = "initscripts-functions" + USERADD_PACKAGES = "${PN}" GROUPADD_PARAM_${PN} = "--system crontab" diff --git a/meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb b/meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb index 6043501..c1c3403 100644 --- a/meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb +++ b/meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb @@ -33,6 +33,8 @@ PACKAGECONFIG[tcp-wrappers] = "--enable-libwrap,--disable-libwrap,tcp-wrappers" INITSCRIPT_NAME = "rpcbind" INITSCRIPT_PARAMS = "start 12 2 3 4 5 . stop 60 0 1 6 ." +RDEPENDS_${PN} = "initscripts-functions" + SYSTEMD_SERVICE_${PN} = "rpcbind.service" SYSTEMD_AUTO_ENABLE = "disable" diff --git a/meta/recipes-extended/sysklogd/sysklogd.inc b/meta/recipes-extended/sysklogd/sysklogd.inc index 3a51810..ef70044 100644 --- a/meta/recipes-extended/sysklogd/sysklogd.inc +++ b/meta/recipes-extended/sysklogd/sysklogd.inc @@ -24,6 +24,8 @@ SRC_URI_append_e500v2 = " file://no-vectorization.patch" INITSCRIPT_NAME = "syslog" CONFFILES_${PN} = "${sysconfdir}/syslog.conf.${BPN}" +RDEPENDS_${PN} = "initscripts-functions" + CFLAGS_append = " -DSYSV" do_install () { diff --git a/meta/recipes-extended/xinetd/xinetd_2.3.15.bb b/meta/recipes-extended/xinetd/xinetd_2.3.15.bb index 1928949..c43892c 100644 --- a/meta/recipes-extended/xinetd/xinetd_2.3.15.bb +++ b/meta/recipes-extended/xinetd/xinetd_2.3.15.bb @@ -7,6 +7,7 @@ LICENSE = "BSD" LIC_FILES_CHKSUM = "file://COPYRIGHT;md5=8ad8615198542444f84d28a6cf226dd8" DEPENDS = "" +RDEPENDS_${PN} = "initscripts-functions" PR = "r2" SRC_URI = "http://www.xinetd.org/xinetd-${PV}.tar.gz \ diff --git a/meta/recipes-kernel/oprofile/oprofileui-server_git.bb b/meta/recipes-kernel/oprofile/oprofileui-server_git.bb index dbb5599..63f33e2 100644 --- a/meta/recipes-kernel/oprofile/oprofileui-server_git.bb +++ b/meta/recipes-kernel/oprofile/oprofileui-server_git.bb @@ -11,7 +11,7 @@ SRC_URI = "git://git.yoctoproject.org/oprofileui \ EXTRA_OECONF += "--disable-client --enable-server" -RDEPENDS_${PN} = "oprofile" +RDEPENDS_${PN} = "oprofile initscripts-functions" do_install_append() { install -d ${D}${sysconfdir}/init.d -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions 2014-01-06 7:27 ` [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions Chen Qi @ 2014-01-17 14:11 ` Paul Eggleton 2014-01-17 15:37 ` Martin Jansa 2014-01-20 2:12 ` ChenQi 0 siblings, 2 replies; 9+ messages in thread From: Paul Eggleton @ 2014-01-17 14:11 UTC (permalink / raw) To: Chen Qi; +Cc: openembedded-core Hi Qi, On Monday 06 January 2014 15:27:35 Chen Qi wrote: > Now that the initscripts-functions has been packaged separately, > packages which may use the functions script should have a runtime > dependency on it. > > Signed-off-by: Chen Qi <Qi.Chen@windriver.com> > --- > meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- > meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + > .../irda-utils/irda-utils_0.9.18.bb | 2 ++ > .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- > meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- > meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + > meta/recipes-core/dbus/dbus.inc | 2 +- > meta/recipes-core/systemd/systemd_208.bb | 2 +- > meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- > meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + > meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- > meta/recipes-extended/at/at_3.1.14.bb | 3 +-- > meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ > meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ > meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ > meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + > .../oprofile/oprofileui-server_git.bb | 2 +- > 17 files changed, 22 insertions(+), 11 deletions(-) Sorry to come back on this patchset again, but could we not detect installation of an initscript requiring these functions in update-rc.d.bbclass and just add the RDEPENDS automatically? Otherwise it's just another thing people can get wrong when writing a recipe, not to mention all of the existing recipes outside OE-Core that install initscripts. Cheers, Paul -- Paul Eggleton Intel Open Source Technology Centre ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions 2014-01-17 14:11 ` Paul Eggleton @ 2014-01-17 15:37 ` Martin Jansa 2014-01-20 2:15 ` ChenQi 2014-01-20 2:12 ` ChenQi 1 sibling, 1 reply; 9+ messages in thread From: Martin Jansa @ 2014-01-17 15:37 UTC (permalink / raw) To: Paul Eggleton; +Cc: openembedded-core [-- Attachment #1: Type: text/plain, Size: 2461 bytes --] On Fri, Jan 17, 2014 at 02:11:39PM +0000, Paul Eggleton wrote: > Hi Qi, > > On Monday 06 January 2014 15:27:35 Chen Qi wrote: > > Now that the initscripts-functions has been packaged separately, > > packages which may use the functions script should have a runtime > > dependency on it. > > > > Signed-off-by: Chen Qi <Qi.Chen@windriver.com> > > --- > > meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- > > meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + > > .../irda-utils/irda-utils_0.9.18.bb | 2 ++ > > .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- > > meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- > > meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + > > meta/recipes-core/dbus/dbus.inc | 2 +- > > meta/recipes-core/systemd/systemd_208.bb | 2 +- > > meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- > > meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + > > meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- > > meta/recipes-extended/at/at_3.1.14.bb | 3 +-- > > meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ > > meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ > > meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ > > meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + > > .../oprofile/oprofileui-server_git.bb | 2 +- > > 17 files changed, 22 insertions(+), 11 deletions(-) > > Sorry to come back on this patchset again, but could we not detect > installation of an initscript requiring these functions in update-rc.d.bbclass > and just add the RDEPENDS automatically? Otherwise it's just another thing > people can get wrong when writing a recipe, not to mention all of the existing > recipes outside OE-Core that install initscripts. Agreed, my current list of components where I've reverted this (to get my images building again) is: dbus, systemd, apmd, rpcbind, nfs-utils, alsa-state, openssh, ppp, cronie, irda-utils To be fair, it's possible that some of these need it and I just don't use their script in runtime (e.g. irda-utils is included only in image on "spitz" MACHINE and I've never used irda here - it's pulled through some packagegroup because MACHINE_FEATURES have irda) -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 205 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions 2014-01-17 15:37 ` Martin Jansa @ 2014-01-20 2:15 ` ChenQi 2014-01-20 10:18 ` Martin Jansa 0 siblings, 1 reply; 9+ messages in thread From: ChenQi @ 2014-01-20 2:15 UTC (permalink / raw) To: Martin Jansa, Paul Eggleton; +Cc: openembedded-core On 01/17/2014 11:37 PM, Martin Jansa wrote: > On Fri, Jan 17, 2014 at 02:11:39PM +0000, Paul Eggleton wrote: >> Hi Qi, >> >> On Monday 06 January 2014 15:27:35 Chen Qi wrote: >>> Now that the initscripts-functions has been packaged separately, >>> packages which may use the functions script should have a runtime >>> dependency on it. >>> >>> Signed-off-by: Chen Qi <Qi.Chen@windriver.com> >>> --- >>> meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- >>> meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + >>> .../irda-utils/irda-utils_0.9.18.bb | 2 ++ >>> .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- >>> meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- >>> meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + >>> meta/recipes-core/dbus/dbus.inc | 2 +- >>> meta/recipes-core/systemd/systemd_208.bb | 2 +- >>> meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- >>> meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + >>> meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- >>> meta/recipes-extended/at/at_3.1.14.bb | 3 +-- >>> meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ >>> meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ >>> meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ >>> meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + >>> .../oprofile/oprofileui-server_git.bb | 2 +- >>> 17 files changed, 22 insertions(+), 11 deletions(-) >> Sorry to come back on this patchset again, but could we not detect >> installation of an initscript requiring these functions in update-rc.d.bbclass >> and just add the RDEPENDS automatically? Otherwise it's just another thing >> people can get wrong when writing a recipe, not to mention all of the existing >> recipes outside OE-Core that install initscripts. > Agreed, my current list of components where I've reverted this (to get > my images building again) is: > > dbus, systemd, apmd, rpcbind, nfs-utils, alsa-state, openssh, ppp, > cronie, irda-utils Hi Martin, Could you please give me some more information why to revert this? I really don't want to mess things up. So I really wish I could be aware of some potential problems before I start working on this. Best Regards, Chen Qi > To be fair, it's possible that some of these need it and I just don't > use their script in runtime (e.g. irda-utils is included only in image > on "spitz" MACHINE and I've never used irda here - it's pulled through > some packagegroup because MACHINE_FEATURES have irda) > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions 2014-01-20 2:15 ` ChenQi @ 2014-01-20 10:18 ` Martin Jansa 2014-01-23 12:14 ` ChenQi 0 siblings, 1 reply; 9+ messages in thread From: Martin Jansa @ 2014-01-20 10:18 UTC (permalink / raw) To: ChenQi; +Cc: Paul Eggleton, openembedded-core [-- Attachment #1: Type: text/plain, Size: 3632 bytes --] On Mon, Jan 20, 2014 at 10:15:41AM +0800, ChenQi wrote: > On 01/17/2014 11:37 PM, Martin Jansa wrote: > > On Fri, Jan 17, 2014 at 02:11:39PM +0000, Paul Eggleton wrote: > >> Hi Qi, > >> > >> On Monday 06 January 2014 15:27:35 Chen Qi wrote: > >>> Now that the initscripts-functions has been packaged separately, > >>> packages which may use the functions script should have a runtime > >>> dependency on it. > >>> > >>> Signed-off-by: Chen Qi <Qi.Chen@windriver.com> > >>> --- > >>> meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- > >>> meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + > >>> .../irda-utils/irda-utils_0.9.18.bb | 2 ++ > >>> .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- > >>> meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- > >>> meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + > >>> meta/recipes-core/dbus/dbus.inc | 2 +- > >>> meta/recipes-core/systemd/systemd_208.bb | 2 +- > >>> meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- > >>> meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + > >>> meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- > >>> meta/recipes-extended/at/at_3.1.14.bb | 3 +-- > >>> meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ > >>> meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ > >>> meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ > >>> meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + > >>> .../oprofile/oprofileui-server_git.bb | 2 +- > >>> 17 files changed, 22 insertions(+), 11 deletions(-) > >> Sorry to come back on this patchset again, but could we not detect > >> installation of an initscript requiring these functions in update-rc.d.bbclass > >> and just add the RDEPENDS automatically? Otherwise it's just another thing > >> people can get wrong when writing a recipe, not to mention all of the existing > >> recipes outside OE-Core that install initscripts. > > Agreed, my current list of components where I've reverted this (to get > > my images building again) is: > > > > dbus, systemd, apmd, rpcbind, nfs-utils, alsa-state, openssh, ppp, > > cronie, irda-utils > > Hi Martin, > > Could you please give me some more information why to revert this? > I really don't want to mess things up. So I really wish I could be aware > of some potential problems before I start working on this. I'm using systemd only, so on my image there aren't sysvinit scripts installed and to make sure that everything stays that way I've blacklisted initscripts recipe in distro configuration. http://lists.openembedded.org/pipermail/openembedded-core/2014-January/088116.html So including it only when update-rc.d is enabled and the sysvinit script isn't masked by systemd service or disabled by openembedded-core/meta/recipes-core/systemd/systemd-compat-units.bb looks like good compromise. Changing it to RRECOMMENDS isn't really correct (because it's either mandatory required or not at all), but it would also make "disabling" it a bit easier. Regards, > > Best Regards, > Chen Qi > > > To be fair, it's possible that some of these need it and I just don't > > use their script in runtime (e.g. irda-utils is included only in image > > on "spitz" MACHINE and I've never used irda here - it's pulled through > > some packagegroup because MACHINE_FEATURES have irda) > > > -- Martin 'JaMa' Jansa jabber: Martin.Jansa@gmail.com [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 205 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions 2014-01-20 10:18 ` Martin Jansa @ 2014-01-23 12:14 ` ChenQi 0 siblings, 0 replies; 9+ messages in thread From: ChenQi @ 2014-01-23 12:14 UTC (permalink / raw) To: Martin Jansa; +Cc: Paul Eggleton, openembedded-core Hi Martin & Paul, I've reworked on this issue, and patches have been sent out. It would be really appreciated if you could have a look at the patches. Best Regards, Chen Qi On 01/20/2014 06:18 PM, Martin Jansa wrote: > On Mon, Jan 20, 2014 at 10:15:41AM +0800, ChenQi wrote: >> On 01/17/2014 11:37 PM, Martin Jansa wrote: >>> On Fri, Jan 17, 2014 at 02:11:39PM +0000, Paul Eggleton wrote: >>>> Hi Qi, >>>> >>>> On Monday 06 January 2014 15:27:35 Chen Qi wrote: >>>>> Now that the initscripts-functions has been packaged separately, >>>>> packages which may use the functions script should have a runtime >>>>> dependency on it. >>>>> >>>>> Signed-off-by: Chen Qi <Qi.Chen@windriver.com> >>>>> --- >>>>> meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- >>>>> meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + >>>>> .../irda-utils/irda-utils_0.9.18.bb | 2 ++ >>>>> .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- >>>>> meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- >>>>> meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + >>>>> meta/recipes-core/dbus/dbus.inc | 2 +- >>>>> meta/recipes-core/systemd/systemd_208.bb | 2 +- >>>>> meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- >>>>> meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + >>>>> meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- >>>>> meta/recipes-extended/at/at_3.1.14.bb | 3 +-- >>>>> meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ >>>>> meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ >>>>> meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ >>>>> meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + >>>>> .../oprofile/oprofileui-server_git.bb | 2 +- >>>>> 17 files changed, 22 insertions(+), 11 deletions(-) >>>> Sorry to come back on this patchset again, but could we not detect >>>> installation of an initscript requiring these functions in update-rc.d.bbclass >>>> and just add the RDEPENDS automatically? Otherwise it's just another thing >>>> people can get wrong when writing a recipe, not to mention all of the existing >>>> recipes outside OE-Core that install initscripts. >>> Agreed, my current list of components where I've reverted this (to get >>> my images building again) is: >>> >>> dbus, systemd, apmd, rpcbind, nfs-utils, alsa-state, openssh, ppp, >>> cronie, irda-utils >> Hi Martin, >> >> Could you please give me some more information why to revert this? >> I really don't want to mess things up. So I really wish I could be aware >> of some potential problems before I start working on this. > I'm using systemd only, so on my image there aren't sysvinit scripts > installed and to make sure that everything stays that way I've > blacklisted initscripts recipe in distro configuration. > > http://lists.openembedded.org/pipermail/openembedded-core/2014-January/088116.html > > So including it only when update-rc.d is enabled and the sysvinit script > isn't masked by systemd service or disabled by > openembedded-core/meta/recipes-core/systemd/systemd-compat-units.bb > looks like good compromise. > > Changing it to RRECOMMENDS isn't really > correct (because it's either mandatory required or not at all), but it > would also make "disabling" it a bit easier. > > Regards, > >> Best Regards, >> Chen Qi >> >>> To be fair, it's possible that some of these need it and I just don't >>> use their script in runtime (e.g. irda-utils is included only in image >>> on "spitz" MACHINE and I've never used irda here - it's pulled through >>> some packagegroup because MACHINE_FEATURES have irda) >>> ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions 2014-01-17 14:11 ` Paul Eggleton 2014-01-17 15:37 ` Martin Jansa @ 2014-01-20 2:12 ` ChenQi 1 sibling, 0 replies; 9+ messages in thread From: ChenQi @ 2014-01-20 2:12 UTC (permalink / raw) To: Paul Eggleton; +Cc: openembedded-core On 01/17/2014 10:11 PM, Paul Eggleton wrote: > Hi Qi, > > On Monday 06 January 2014 15:27:35 Chen Qi wrote: >> Now that the initscripts-functions has been packaged separately, >> packages which may use the functions script should have a runtime >> dependency on it. >> >> Signed-off-by: Chen Qi <Qi.Chen@windriver.com> >> --- >> meta/recipes-bsp/alsa-state/alsa-state.bb | 2 +- >> meta/recipes-bsp/apmd/apmd_3.2.2-14.bb | 1 + >> .../irda-utils/irda-utils_0.9.18.bb | 2 ++ >> .../nfs-utils/nfs-utils_1.2.9.bb | 4 ++-- >> meta/recipes-connectivity/openssh/openssh_6.4p1.bb | 2 +- >> meta/recipes-connectivity/ppp/ppp_2.4.5.bb | 1 + >> meta/recipes-core/dbus/dbus.inc | 2 +- >> meta/recipes-core/systemd/systemd_208.bb | 2 +- >> meta/recipes-core/sysvinit/sysvinit_2.88dsf.bb | 2 +- >> meta/recipes-devtools/distcc/distcc_3.1.bb | 1 + >> meta/recipes-devtools/tcf-agent/tcf-agent_git.bb | 2 +- >> meta/recipes-extended/at/at_3.1.14.bb | 3 +-- >> meta/recipes-extended/cronie/cronie_1.4.11.bb | 2 ++ >> meta/recipes-extended/rpcbind/rpcbind_0.2.1.bb | 2 ++ >> meta/recipes-extended/sysklogd/sysklogd.inc | 2 ++ >> meta/recipes-extended/xinetd/xinetd_2.3.15.bb | 1 + >> .../oprofile/oprofileui-server_git.bb | 2 +- >> 17 files changed, 22 insertions(+), 11 deletions(-) > Sorry to come back on this patchset again, but could we not detect > installation of an initscript requiring these functions in update-rc.d.bbclass > and just add the RDEPENDS automatically? Otherwise it's just another thing > people can get wrong when writing a recipe, not to mention all of the existing > recipes outside OE-Core that install initscripts. > > Cheers, > Paul > Got it. I'll try that out. Best Regards, Chen Qi ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2014-01-23 12:14 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2014-01-06 7:27 [PATCH V3 0/2] Split the functions script into a separate package in initscripts Chen Qi 2014-01-06 7:27 ` [PATCH V3 1/2] initscripts: split the functions script into a separate package Chen Qi 2014-01-06 7:27 ` [PATCH V3 2/2] Add missing RDEPENDS of initscripts-functions Chen Qi 2014-01-17 14:11 ` Paul Eggleton 2014-01-17 15:37 ` Martin Jansa 2014-01-20 2:15 ` ChenQi 2014-01-20 10:18 ` Martin Jansa 2014-01-23 12:14 ` ChenQi 2014-01-20 2:12 ` ChenQi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.