clearlinux / cve-check-tool

Original Automated CVE Checking Tool
GNU General Public License v2.0
205 stars 78 forks source link

--type option issue when showing available types #10

Closed estebancrw closed 9 years ago

estebancrw commented 9 years ago

Man page says:

[...] Available types can be determine by using this option without any parameters.[...]

When only the following command is typed: cve-check-tool -t Output: Invalid options: Missing argument for -t

For it to show the available types, a "parameter" and a "path" need to be sent as following: cve-check-tool -t something / Output:

Unknown type: "something".
Currently supported types: eopkg, rpm, pkgbuild[a line break is missing here]

Is this expected? If so, maybe the man page could be updated to something like "Available types can be determine by using this option with a dummy parameter and a path."

ikeydoherty commented 9 years ago

Added 'list' - per the commit message the GOption* stuff is all a bit crazy and broken, so I'll make a note to swap out the option handling in the near future.

estebancrw commented 9 years ago

Using the following commands now outputs the list of available packages:

$ cve-check-tool -t list
Currently supported types: eopkg, rpm, pkgbuild
$ cve-check-tool -t something
Unknown type: "something"
Currently supported types: eopkg, rpm, pkgbuild

Closing this issue since it has been solved.

ikeydoherty commented 9 years ago

Awesome, thanks :D