<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Oct 3, 2015 at 11:39 AM, Christopher Larson <span dir="ltr">&lt;<a href="mailto:clarson@kergoth.com" target="_blank">clarson@kergoth.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><div class="gmail_quote">On Fri, Oct 2, 2015 at 9:36 PM, Robert Yang <span dir="ltr">&lt;<a href="mailto:liezhi.yang@windriver.com" target="_blank">liezhi.yang@windriver.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="overflow:hidden">+QAPATHTEST[conffiles] = &quot;package_qa_check_conffiles&quot;<br>
+def package_qa_check_conffiles(path, name, d, elf, messages):<br>
+    &quot;&quot;&quot; Check files in ${sysconfdir} are marked by CONFFILES &quot;&quot;&quot;<br>
+<br>
+    if os.path.islink(path) or not os.path.exists(path):<br>
+        return True<br>
+<br>
+    sysconfdir = d.getVar(&#39;sysconfdir&#39;, True)<br>
+    dest = d.getVar(&#39;PKGDEST&#39;, True)<br>
+    sysconfdir_path = os.path.join(os.path.join(dest, name), \<br>
+                        sysconfdir.lstrip(&#39;/&#39;))<br>
+<br>
+    if not path.startswith(sysconfdir_path):<br>
+        return True</div></blockquote></div><br></span>This is missing a trailing slash, so e.g. checking /etcfoo would match, not just /etc. Whenever you use startswith() on a path, remember to append &#39;/&#39; or os.sep to the string being checked</blockquote></div><br>To clarify, os.sep needs adding to sysconfdir_path, not path :) I realized my wording was ambiguous.<br>-- <br><div class="gmail_signature">Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics</div>
</div></div>