templates
/
KiBot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[KiBot Check] Added the comments 

- When a tool is missing.
- Currently Pandoc is the only with extra comments
Related to #228
This commit is contained in:
Salvador E. Tropea 2022-07-18 08:07:11 -03:00
parent 0726a3c2d5
commit 8f2f59eeb0
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/kibot-check
View File

@ -930,7 +930,7 @@ def python_module(severity, name, deb_package, roles):
print(RESET) print(RESET)
def binary_tool(severity, name, url, url_down, deb_package, deb, extra_deb, roles, downloader): def binary_tool(severity, name, url, url_down, deb_package, deb, extra_deb, roles, downloader, comments):
if not severity: if not severity:
return return
print(sev2color(severity)+'* {} not installed or too old'.format(name)) print(sev2color(severity)+'* {} not installed or too old'.format(name))
@ -944,6 +944,8 @@ def binary_tool(severity, name, url, url_down, deb_package, deb, extra_deb, role
print(' Visit: '+url) print(' Visit: '+url)
if url_down: if url_down:
print(' Download it from: '+url_down) print(' Download it from: '+url_down)
for c in comments:
print(' '+c)
if isinstance(downloader, dict): if isinstance(downloader, dict):
print(' This tool might be automatically downloaded by KiBot.') print(' This tool might be automatically downloaded by KiBot.')
show_roles(roles) show_roles(roles)
@ -1144,7 +1146,7 @@ for name, d in dependencies.items():
else: else:
# TODO: check pip_ok and if this is a Python tool suggest install pip # TODO: check pip_ok and if this is a Python tool suggest install pip
binary_tool(d['sev'], d['name'], d['url'], d['url_down'], d['deb_package'], d['in_debian'], d['extra_deb'], binary_tool(d['sev'], d['name'], d['url'], d['url_down'], d['deb_package'], d['in_debian'], d['extra_deb'],
d['roles'], d['downloader']) d['roles'], d['downloader'], d['comments'])
labels = ('ok', 'optional for an output', 'optional for general use', 'mandatory for an output', 'mandatory for general use') labels = ('ok', 'optional for an output', 'optional for general use', 'mandatory for an output', 'mandatory for general use')
text = ', '.join([sev2color(c)+l+RESET for c, l in enumerate(labels)]) text = ', '.join([sev2color(c)+l+RESET for c, l in enumerate(labels)])

6
src/kibot-check.in
View File

@ -229,7 +229,7 @@ def python_module(severity, name, deb_package, roles):
print(RESET) print(RESET)
def binary_tool(severity, name, url, url_down, deb_package, deb, extra_deb, roles, downloader): def binary_tool(severity, name, url, url_down, deb_package, deb, extra_deb, roles, downloader, comments):
if not severity: if not severity:
return return
print(sev2color(severity)+'* {} not installed or too old'.format(name)) print(sev2color(severity)+'* {} not installed or too old'.format(name))
@ -243,6 +243,8 @@ def binary_tool(severity, name, url, url_down, deb_package, deb, extra_deb, role
print(' Visit: '+url) print(' Visit: '+url)
if url_down: if url_down:
print(' Download it from: '+url_down) print(' Download it from: '+url_down)
for c in comments:
print(' '+c)
if isinstance(downloader, dict): if isinstance(downloader, dict):
print(' This tool might be automatically downloaded by KiBot.') print(' This tool might be automatically downloaded by KiBot.')
show_roles(roles) show_roles(roles)
@ -443,7 +445,7 @@ for name, d in dependencies.items():
else: else:
# TODO: check pip_ok and if this is a Python tool suggest install pip # TODO: check pip_ok and if this is a Python tool suggest install pip
binary_tool(d['sev'], d['name'], d['url'], d['url_down'], d['deb_package'], d['in_debian'], d['extra_deb'], binary_tool(d['sev'], d['name'], d['url'], d['url_down'], d['deb_package'], d['in_debian'], d['extra_deb'],
d['roles'], d['downloader']) d['roles'], d['downloader'], d['comments'])
labels = ('ok', 'optional for an output', 'optional for general use', 'mandatory for an output', 'mandatory for general use') labels = ('ok', 'optional for an output', 'optional for general use', 'mandatory for an output', 'mandatory for general use')
text = ', '.join([sev2color(c)+l+RESET for c, l in enumerate(labels)]) text = ', '.join([sev2color(c)+l+RESET for c, l in enumerate(labels)])