On 10/22/2013 06:32 AM, Khem Raj wrote: > > > On Sunday, October 20, 2013, Ming Liu > wrote: > > 1 Add base-files to RDEPENDS. > > 2 Add pkg_postinst and pkg_postrm to add and remove a entry in > > /etc/shells. > > > > will it affect upgradability of the package which is providing > /etc/shells ? Yes, you are right, it will. So we need put /etc/shells to CONFFILES in base-files package. //Ming Liu > > > Signed-off-by: Ming Liu > > > --- > > meta/recipes-extended/screen/screen_4.0.3.bb > | 10 ++++++++++ > > 1 file changed, 10 insertions(+) > > > > diff --git a/meta/recipes-extended/screen/screen_4.0.3.bb > > b/meta/recipes-extended/screen/screen_4.0.3.bb > > index 1a7eb20..be42fb9 100644 > > --- a/meta/recipes-extended/screen/screen_4.0.3.bb > > > +++ b/meta/recipes-extended/screen/screen_4.0.3.bb > > > @@ -12,6 +12,8 @@ LIC_FILES_CHKSUM = > "file://COPYING;md5=0774d66808b0f602e94448108f59448b \ > > SECTION = "console/utils" > > DEPENDS = "ncurses \ > > ${@base_contains('DISTRO_FEATURES', 'pam', 'libpam', '', d)}" > > +RDEPENDS_${PN} = "base-files" > > + > > PR = "r3" > > > > SRC_URI = "${GNU_MIRROR}/screen/screen-${PV}.tar.gz;name=tarball \ > > @@ -41,3 +43,11 @@ do_install_append () { > > fi > > done > > } > > + > > +pkg_postinst_${PN} () { > > + grep -q "^${bindir}/screen$" $D${sysconfdir}/shells || echo > ${bindir}/screen >> $D${sysconfdir}/shells > > +} > > + > > +pkg_postrm_${PN} () { > > + printf "$(grep -v "^${bindir}/screen$" > $D${sysconfdir}/shells)\n" > $D${sysconfdir}/shells > > +} > > -- > > 1.8.4.1 > > > > _______________________________________________ > > Openembedded-core mailing list > > Openembedded-core@lists.openembedded.org > > > http://lists.openembedded.org/mailman/listinfo/openembedded-core > >