<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 31 March 2015 at 23:03, Burton, Ross <span dir="ltr">&lt;<a href="mailto:ross.burton@intel.com" target="_blank">ross.burton@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"><div dir="ltr"><div class="gmail_extra"><span class=""><br><div class="gmail_quote">On 31 March 2015 at 21:11, Beth Flanagan <span dir="ltr">&lt;<a href="mailto:elizabeth.flanagan@intel.com" target="_blank">elizabeth.flanagan@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"><div style="overflow:hidden">+        if pnstripped.find(&quot;-native&quot;) != -1:<br></div></blockquote></div><br></span>Shouldn&#39;t this be .endswith() or .startswith() to prevent false-positives?</div></div></blockquote><div><br></div><div>Also this all clearly says &quot;put me in a function&quot;.  But what you need is *almost* the BPN variable but with some prefix stripping that the BPN-logic doesn&#39;t do (BPN only has suffix stripped, oddly).</div><div><br></div><div>I&#39;d argue that the BPN logic should also strip nativesdk- prefixes, so by fixing that the distrotools class could just get PN and BPN from the data store and be done with it.</div><div><br></div><div>Ross</div></div></div></div>