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>
55 lines
1.6 KiB
Python
55 lines
1.6 KiB
Python
# 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]
|