diff options
author | Ricardo Martincoski <ricardo.martincoski@gmail.com> | 2017-04-19 15:06:21 -0300 |
---|---|---|
committer | Thomas Petazzoni <thomas.petazzoni@free-electrons.com> | 2017-04-20 22:29:41 +0200 |
commit | 7b394c4926dcb860a356eeb46b1f1f5d807041f7 (patch) | |
tree | 0b61db0b8f7433dc1c2619ab3f0353201c84bcd0 /support/scripts/checkpackagelib/lib.py | |
parent | 33481124c7913fbdaee327afd97fd8ba138434e6 (diff) | |
download | buildroot-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.py | 54 |
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] |