<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jun 7, 2013 at 11:10 PM, ChenQi <span dir="ltr">&lt;<a href="mailto:Qi.Chen@windriver.com" target="_blank">Qi.Chen@windriver.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div text="#000000" bgcolor="#FFFFFF"><div class="im">
    <div>On 06/07/2013 08:32 PM, Otavio Salvador
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr"><br>
        <div class="gmail_extra"><br>
          <br>
          <div class="gmail_quote">On Fri, Jun 7, 2013 at 3:13 AM, <span dir="ltr">&lt;<a href="mailto:Qi.Chen@windriver.com" target="_blank">Qi.Chen@windriver.com</a>&gt;</span>
            wrote:<br>
            <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From:
              Chen Qi &lt;<a href="mailto:Qi.Chen@windriver.com" target="_blank">Qi.Chen@windriver.com</a>&gt;<br>
              <br>
              Remove the pkg_postinst_${PN} from this recipe, as it&#39;s
              redundant.<br>
              It basically wants to do the same thing as the
              update-alternatives<br>
              does. But it doesn&#39;t do it well.<br>
              <br>
              Signed-off-by: Chen Qi &lt;<a href="mailto:Qi.Chen@windriver.com" target="_blank">Qi.Chen@windriver.com</a>&gt;<br>
            </blockquote>
            <div><br>
            </div>
            <div>Instead of removing it, please convert it here
              for update-alternatives so it keeps working on this commit
              and allow bisect in future if need. </div>
          </div>
          <div><br>
          </div>
        </div>
      </div>
    </blockquote></div>
    Hi Otavio,<br>
    <br>
    Thanks for your review and comments. <br>
    Sorry but I don&#39;t understand it :(<br>
    What do you mean by &quot;convert it for update-alternatives&quot;?<br>
    Could you please give me a little more explanation?<br></div></blockquote></div><div class="gmail_extra"><br></div>Yes, add the update-alternatives support here as well so we can remove the postinst and add the  new code in same patch; it allows for bisect :)<br clear="all">
<div><br></div>-- <br>Otavio Salvador                             O.S. Systems<br><a href="http://www.ossystems.com.br" target="_blank">http://www.ossystems.com.br</a>        <a href="http://projetos.ossystems.com.br" target="_blank">http://projetos.ossystems.com.br</a><br>
Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750<br>
</div></div>