support/check-package: don't check filenames of hashes
Currently, we check that the filenames in hash lists do not contain a slash '/' character, because all we are checking so far are the downloaded archives, and we explicitly need the filename to not contain a directory component at all. However, we're soon to also check the hashes of the license files in packages sources, and those license files may be at any arbitrary directory-depth in the packages source tree. [Peter: Remove reference to files with same basename] Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr> Cc: Luca Ceresoli <luca@lucaceresoli.net> Cc: Peter Korsgaard <peter@korsgaard.com> Cc: Rahul Bedarkar <rahulbedarkar89@gmail.com> Cc: Ricardo Martincoski <ricardo.martincoski@gmail.com> Acked-by: Ricardo Martincoski <ricardo.martincoski@gmail.com> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
This commit is contained in:
parent
e7548edb5f
commit
efb61ae07b
@ -17,22 +17,6 @@ def _empty_line_or_comment(text):
|
|||||||
return text.strip() == "" or text.startswith("#")
|
return text.strip() == "" or text.startswith("#")
|
||||||
|
|
||||||
|
|
||||||
class HashFilename(_CheckFunction):
|
|
||||||
def check_line(self, lineno, text):
|
|
||||||
if _empty_line_or_comment(text):
|
|
||||||
return
|
|
||||||
|
|
||||||
fields = text.split()
|
|
||||||
if len(fields) < 3:
|
|
||||||
return
|
|
||||||
|
|
||||||
if '/' in fields[2]:
|
|
||||||
return ["{}:{}: use filename without directory component"
|
|
||||||
" ({}#adding-packages-hash)"
|
|
||||||
.format(self.filename, lineno, self.url_to_manual),
|
|
||||||
text]
|
|
||||||
|
|
||||||
|
|
||||||
class HashNumberOfFields(_CheckFunction):
|
class HashNumberOfFields(_CheckFunction):
|
||||||
def check_line(self, lineno, text):
|
def check_line(self, lineno, text):
|
||||||
if _empty_line_or_comment(text):
|
if _empty_line_or_comment(text):
|
||||||
|
Loading…
Reference in New Issue
Block a user