<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jun 8, 2016 at 12:06 AM, Paul Eggleton <span dir="ltr">&lt;<a href="mailto:paul.eggleton@linux.intel.com" target="_blank">paul.eggleton@linux.intel.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Sasha,<br>
<br>
Functionality wise this looks good.<br>
<span class=""><br>
On Mon, 06 Jun 2016 00:44:29 Alexander D. Kanevskiy wrote:<br>
&gt; New option can be used for displaying full information about binary<br>
&gt; package(s), including name, full version, recipe name, recipe full version<br>
&gt; and package size.<br>
&gt;<br>
&gt; This information can be useful to produce detailed image manifest<br>
&gt; for further analysis.<br>
<br>
</span>Possibly worth noting that since this looks up current information rather than<br>
what&#39;s in the image, this is only going to be accurate for that purpose if no<br>
packages have been rebuilt (strictly speaking no do_packagedata tasks have<br>
executed) since the manifest was generated.<br></blockquote><div><br></div><div>Assumption that it would be used in extra buildhistory class, so pkgdata still should be valid.</div><div>But good point that it can be executed manually. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div><div class="h5">&gt; +    for pkg in packages:<br>
&gt; +        items.append((&quot;%s %s%s%s %s %s%s%s %s&quot; % (<br>
&gt; +            pkg,<br>
&gt; +            mappings[pkg][&#39;PKGE&#39;] + &quot;:&quot; if mappings[pkg][&#39;PKGE&#39;] else &quot;&quot;,<br>
&gt; +            mappings[pkg][&#39;PKGV&#39;],<br>
&gt; +            &quot;-&quot; + mappings[pkg][&#39;PKGR&#39;] if mappings[pkg][&#39;PKGR&#39;] else &quot;&quot;,<br>
&gt; +            mappings[pkg][&#39;PN&#39;],<br>
&gt; +            mappings[pkg][&#39;PE&#39;] + &quot;:&quot; if mappings[pkg][&#39;PE&#39;] else &quot;&quot;,<br>
&gt; +            mappings[pkg][&#39;PV&#39;],<br>
&gt; +            &quot;-&quot; + mappings[pkg][&#39;PR&#39;] if mappings[pkg][&#39;PR&#39;] else &quot;&quot;,<br>
&gt; +            mappings[pkg][&#39;PKGSIZE&#39;]<br>
&gt; +        )).strip()<br>
&gt; +        )<br>
<br>
</div></div>This seems like a bit of an ugly construction. If it were me writing it I<br>
would probably break it down using variables rather than trying to do it all<br>
on one line. That said it&#39;s not bad enough to block it, if nobody else feels<br>
strongly about it then you&#39;re in the clear.<br></blockquote><div><br></div><div>ok. I&#39;ll send new version of patch a bit later today.</div><div> </div></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">br, Alexander Kanevskiy</div>
</div></div>