On 10/17/2013 06:33 PM, Martin Jansa wrote:
On Thu, Oct 17, 2013 at 06:03:12PM +0800, Ming Liu wrote:
Signed-off-by: Ming Liu <[email protected]>
---
  meta/recipes-extended/screen/screen_4.0.3.bb | 5 +++++
  1 file changed, 5 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..81e418b 100644
--- a/meta/recipes-extended/screen/screen_4.0.3.bb
+++ b/meta/recipes-extended/screen/screen_4.0.3.bb
@@ -41,3 +41,8 @@ do_install_append () {
                fi
        done
  }
+
+pkg_postinst_${PN} () {
+       touch $D${sysconfdir}/shells
+       grep -q "${bindir}/screen" $D${sysconfdir}/shells || echo ${bindir}/screen 
>> $D${sysconfdir}/shells
                   ^ the same, we don't want to match /usr/bin/screenie
if it's there and "screen" isn't
Yes, you are correct, I am too less comprehensive.


Is there patch to add this postinst to bash and busybox?
For bash, it already has it, as following:
------------------
pkg_postinst_${PN} () {
        touch $D${sysconfdir}/shells
grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash >> $D${sysconfdir}/shells grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh >> $D${sysconfdir}/shells
}
-----------------

But as you pointed out, it needs to be more stringent too.

For busybox, I don't think it need this script for it provides the symbol link: /bin/sh.

//Ming Liu


+}
--
1.8.3.3

_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to