<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif;"><div><br></div><span id="OLK_SRC_BODY_SECTION"><div style="font-family:Calibri; font-size:11pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt"><span style="font-weight:bold">From: </span> "Burton, Ross" <<a href="mailto:ross.burton@intel.com">ross.burton@intel.com</a>><br><span style="font-weight:bold">Date: </span> Wednesday 14 September 2016 at 19:38<br><span style="font-weight:bold">To: </span> Markus Lehtonen <<a href="mailto:markus.lehtonen@linux.intel.com">markus.lehtonen@linux.intel.com</a>><br><span style="font-weight:bold">Cc: </span> OE-core <<a href="mailto:openembedded-core@lists.openembedded.org">openembedded-core@lists.openembedded.org</a>><br><span style="font-weight:bold">Subject: </span> Re: [OE-core] [PATCH] scripts: introduce buildstats-diff<br></div><div><br></div><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On 14 September 2016 at 12:24, Markus Lehtonen <span dir="ltr"><<a href="mailto:markus.lehtonen@linux.intel.com" target="_blank">markus.lehtonen@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":1ic" class="a3s aXjCH m157287239512fb54">+ if not tasks1:<br>
+ pkg_op = '++'<br>
+ elif not tasks2:<br>
+ pkg_op = '--'</div></blockquote></div><br>BTW, I've been reading buildstats-diff output all day and find the lack of whitespace between the operator and the package name makes the output harder to read, so I've patched it locally to be just '+ ' or '- ' for clarity.</div></div></span><div><br></div><div>Sounds fine to me. I don't object your modification.</div><div> - Markus</div><span id="OLK_SRC_BODY_SECTION"><div dir="ltr"><div class="gmail_extra"><br></div></div></span></body></html>