chiark / gitweb /
lint can no longer properly detect unset Summary/Description
authorHans-Christoph Steiner <hans@eds.org>
Fri, 2 Jun 2017 11:56:57 +0000 (13:56 +0200)
committerHans-Christoph Steiner <hans@eds.org>
Wed, 7 Jun 2017 20:10:25 +0000 (22:10 +0200)
Since the Summary/Description can now be set in the app's source code, or
in fdroiddata/metadata/<packageName>/<locale>/*.txt, this lint check is
no longer valid.  It is important to check whether these texts are empty,
but it'll require some thinking about how and where to best to that.
`fdroid update` will have access to all that data, but perhaps at that
point it is too late.

Also, the current text prioritization puts Summary/Description in the
.txt/.yml file at the highest priority, overriding every other copy,
including in fdroiddata/metadata/<packageName>/<locale> and in the app's
source code.

fdroidserver/lint.py

index 9aebd9981c8b59d3fb7eca9ebf19743c8548aa29..3ffdf5e31b30792b4a202c2c113b490b03d8003e 100644 (file)
@@ -88,8 +88,6 @@ regex_checks = {
          "Unnecessary trailing space"),
     ],
     'Summary': [
-        (re.compile(r'^$'),
-         "Summary yet to be filled"),
         (re.compile(r'.*\b(free software|open source)\b.*', re.IGNORECASE),
          "No need to specify that the app is Free Software"),
         (re.compile(r'.*((your|for).*android|android.*(app|device|client|port|version))', re.IGNORECASE),
@@ -102,8 +100,6 @@ regex_checks = {
          "Unnecessary trailing space"),
     ],
     'Description': [
-        (re.compile(r'^No description available$'),
-         "Description yet to be filled"),
         (re.compile(r'\s*[*#][^ .]'),
          "Invalid bulleted list"),
         (re.compile(r'^\s'),