<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">On 10/17/2013 06:33 PM, Martin Jansa
      wrote:<br>
    </div>
    <blockquote cite="mid:20131017103355.GB3930@jama" type="cite">
      <pre wrap="">On Thu, Oct 17, 2013 at 06:03:12PM +0800, Ming Liu wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">Signed-off-by: Ming Liu <a class="moz-txt-link-rfc2396E" href="mailto:ming.liu@windriver.com">&lt;ming.liu@windriver.com&gt;</a>
---
 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 &gt;&gt; $D${sysconfdir}/shells
</pre>
      </blockquote>
      <pre wrap="">                  ^ the same, we don't want to match /usr/bin/screenie
if it's there and "screen" isn't</pre>
    </blockquote>
    Yes, you are correct, I am
    <meta http-equiv="content-type" content="text/html;
      charset=ISO-8859-1">
    too less comprehensive.<br>
    <br>
    <blockquote cite="mid:20131017103355.GB3930@jama" type="cite">
      <pre wrap="">

Is there patch to add this postinst to bash and busybox?</pre>
    </blockquote>
    For bash, it already has it, as following:<br>
    ------------------<br>
    pkg_postinst_${PN} () {<br>
    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; touch $D${sysconfdir}/shells<br>
    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash
    &gt;&gt; $D${sysconfdir}/shells<br>
    &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh
    &gt;&gt; $D${sysconfdir}/shells<br>
    }<br>
    -----------------<br>
    <br>
    But as you pointed out, it needs to be more stringent too.<br>
    <br>
    For busybox, I don't think it need this script for it provides the
    symbol link: /bin/sh.<br>
    <br>
    //Ming Liu<br>
    <br>
    <blockquote cite="mid:20131017103355.GB3930@jama" type="cite">
      <pre wrap="">

</pre>
      <blockquote type="cite">
        <pre wrap="">+}
-- 
1.8.3.3

_______________________________________________
Openembedded-core mailing list
<a class="moz-txt-link-abbreviated" href="mailto:Openembedded-core@lists.openembedded.org">Openembedded-core@lists.openembedded.org</a>
<a class="moz-txt-link-freetext" href="http://lists.openembedded.org/mailman/listinfo/openembedded-core">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a>
</pre>
      </blockquote>
      <pre wrap="">
</pre>
    </blockquote>
    <br>
  </body>
</html>