summaryrefslogtreecommitdiffstats
path: root/support/scripts/checkpackagelib/lib.py
diff options
context:
space:
mode:
authorRicardo Martincoski <ricardo.martincoski@gmail.com>2017-04-19 15:06:21 -0300
committerThomas Petazzoni <thomas.petazzoni@free-electrons.com>2017-04-20 22:29:41 +0200
commit7b394c4926dcb860a356eeb46b1f1f5d807041f7 (patch)
tree0b61db0b8f7433dc1c2619ab3f0353201c84bcd0 /support/scripts/checkpackagelib/lib.py
parent33481124c7913fbdaee327afd97fd8ba138434e6 (diff)
downloadbuildroot-7b394c4926dcb860a356eeb46b1f1f5d807041f7.tar.gz
buildroot-7b394c4926dcb860a356eeb46b1f1f5d807041f7.zip
check-package: move parts to subdirectory
Currently the check-package script uses many files in the same directory. This commit keeps the main script in support/scripts/ and moves the rest into a subdirectory. The modules were previously prefixed to make it easy to identify which script they belong to. This is no longer needed when using a subdirectory, so the prefix is removed. Note: if this commit is checked out and the script is run, and later on a previous version is checked out, the file support/scripts/checkpackagelib/__init__.pyc needs to be manually removed to prevent Python interpreter to look for checkpackagelib package when only the checkpackagelib module is available. Reported-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com> Signed-off-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
Diffstat (limited to 'support/scripts/checkpackagelib/lib.py')
-rw-r--r--support/scripts/checkpackagelib/lib.py54
1 files changed, 54 insertions, 0 deletions
diff --git a/support/scripts/checkpackagelib/lib.py b/support/scripts/checkpackagelib/lib.py
new file mode 100644
index 0000000000..3077f518b3
--- /dev/null
+++ b/support/scripts/checkpackagelib/lib.py
@@ -0,0 +1,54 @@
+# See support/scripts/checkpackagelib/readme.txt before editing this file.
+
+from base import _CheckFunction
+
+
+class ConsecutiveEmptyLines(_CheckFunction):
+ def before(self):
+ self.lastline = "non empty"
+
+ def check_line(self, lineno, text):
+ if text.strip() == "" == self.lastline.strip():
+ return ["{}:{}: consecutive empty lines"
+ .format(self.filename, lineno)]
+ self.lastline = text
+
+
+class EmptyLastLine(_CheckFunction):
+ def before(self):
+ self.lastlineno = 0
+ self.lastline = "non empty"
+
+ def check_line(self, lineno, text):
+ self.lastlineno = lineno
+ self.lastline = text
+
+ def after(self):
+ if self.lastline.strip() == "":
+ return ["{}:{}: empty line at end of file"
+ .format(self.filename, self.lastlineno)]
+
+
+class NewlineAtEof(_CheckFunction):
+ def before(self):
+ self.lastlineno = 0
+ self.lastline = "\n"
+
+ def check_line(self, lineno, text):
+ self.lastlineno = lineno
+ self.lastline = text
+
+ def after(self):
+ if self.lastline == self.lastline.rstrip("\r\n"):
+ return ["{}:{}: missing newline at end of file"
+ .format(self.filename, self.lastlineno),
+ self.lastline]
+
+
+class TrailingSpace(_CheckFunction):
+ def check_line(self, lineno, text):
+ line = text.rstrip("\r\n")
+ if line != line.rstrip():
+ return ["{}:{}: line contains trailing whitespace"
+ .format(self.filename, lineno),
+ text]
OpenPOWER on IntegriCloud