NetCube-Systems-Austria/kumquat-buildroot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

support/scripts/pkg-stats: check all files for warnings

Browse Source 
Instead of only checking .mk and Config.in{,.host}, check
all files in a package directory.
.checkpackageignore isn't considered here, therefore the shown number
includes ignored warnings as well.
Add another css class to signal some warning, compared to a lot (>5),
similar to patches.

Signed-off-by: Daniel Lang <dalang@gmx.at>
Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
This commit is contained in:
Daniel Lang 2023-09-01 21:27:10 +02:00 committed by Thomas Petazzoni
parent 1aa198f728
commit c3c5320ce2
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
support/scripts/pkg-stats
View File

@ -272,8 +272,7 @@ class Package:
self.status['pkg-check'] = ("error", "Missing")
for root, dirs, files in os.walk(pkgdir):
for f in files:
if f.endswith(".mk") or f.endswith(".hash") or f == "Config.in" or f == "Config.in.host":
cmd.append(os.path.join(root, f))
cmd.append(os.path.join(root, f))
o = subprocess.Popen(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE).communicate()[1]
lines = o.splitlines()
for line in lines:
@ -876,7 +875,7 @@ function expandField(fieldId){
.wrong, .lotsofpatches, .invalid_url, .version-needs-update, .cpe-nok, .cve-nok {
background: #ff9a69;
}
.somepatches, .missing_url, .version-unknown, .cpe-unknown, .cve-unknown {
.somepatches, .somewarnings, .missing_url, .version-unknown, .cpe-unknown, .cve-unknown {
background: #ffd870;
}
.cve_ignored, .version-error {
@ -1033,6 +1032,8 @@ def dump_html_pkg(f, pkg):
div_class.append(f'_{pkg_css_class}')
if pkg.warnings == 0:
div_class.append("correct")
elif pkg.warnings < 5:
div_class.append("somewarnings")
else:
div_class.append("wrong")
f.write(f' <div id="{data_field_id}" class="{" ".join(div_class)}">{pkg.warnings}</div>\n')