framework: change "check_for_man" return value upon failure

fedora-selinux / setroubleshoot

Abandoned, use https://gitlab.com/setroubleshoot

9 stars 12 forks source link

framework: change "check_for_man" return value upon failure #46

Closed vmojzis closed 7 years ago

vmojzis commented 7 years ago

check_for_man is designed to be used by plugins, which might "save" the return value in "args" using "report" (viz. checkall_boolean). In that case None is changed to string "None", causing hard to find bugs when recognising non-existent man page.

vmojzis commented 7 years ago

https://bugzilla.redhat.com/show_bug.cgi?id=1142731

bachradsusi commented 7 years ago

Merged https://github.com/fedora-selinux/setroubleshoot/commit/c357e0735736d90e2bbdd4a5351367fbe51057eb Thanks!