<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Sep 4, 2013 at 7:24 AM, Jelle van der Waa <span dir="ltr"><<a href="mailto:jelle@vdwaa.nl" target="_blank">jelle@vdwaa.nl</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">---<br>
 Namcap/rules/missingvars.py | 7 +++++++<br>
 namcap-tags                 | 1 +<br>
 2 files changed, 8 insertions(+)<br>
<br>
diff --git a/Namcap/rules/missingvars.py b/Namcap/rules/missingvars.py<br>
index 90146f3..8211c1b 100644<br>
--- a/Namcap/rules/missingvars.py<br>
+++ b/Namcap/rules/missingvars.py<br>
@@ -76,4 +76,11 @@ class TagsRule(PkgbuildRule):<br>
                if maintainertag != 1:<br>
                        self.warnings.append(("missing-maintainer", ()))<br>
<br>
+class DescSetRule(PkgbuildRule):<br>
+       name = "descset"<br></blockquote><div>Why not just "Description"/"description" instead of "DescSet"/"descset"? <br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

+       description = "Verifies that the description is set in a PKGBUILD"<br>
+       def analyze(self, pkginfo, tar):<br>
+               if "desc" not in pkginfo or len(pkginfo["desc"]) == 0:<br>
+                       self.errors.append(("missing-description", ()))<br>
+<br>
 # vim: set ts=4 sw=4 noet:<br>
diff --git a/namcap-tags b/namcap-tags<br>
index 818c7a5..bb5f664 100644<br>
--- a/namcap-tags<br>
+++ b/namcap-tags<br>
@@ -47,6 +47,7 @@ link-level-dependence %s in %s :: Link-level dependence (%s) in file %s<br>
 lots-of-docs %f :: Package was %.0f%% docs by size; maybe you should split out a docs package<br>
 mime-cache-not-updated :: Mime-file found. Add "update-mime-database usr/share/mime" to the install file<br>
 missing-backup-file %s :: File in backup array (%s) not found in package<br>
+missing-description :: Missing description in PKGBUILD.<br></blockquote><div>None of the rest of these end in a full stop; probably don't want to break that convention here.<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

 missing-contributor :: Missing Contributor tag<br>
 missing-custom-license-dir usr/share/licenses/%s :: Missing custom license directory (usr/share/licenses/%s)<br>
 missing-custom-license-file usr/share/licenses/%s/* :: Missing custom license file in package (usr/share/licenses/%s/*)<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.8.4<br>
<br>
</font></span></blockquote></div><br></div></div>