kongaskristjan/fire-hpp

A C++ library that uses clever tricks to create super low-code, yet fully functional CLIs

For business/contract/employment queries, please message on Linkedin. You can also contact me using the method described here.

fire-hpp's Issues

[Feature Request] Support for assignment without space

For example, we use gcc -I/path/to/a/directory without space between -I and /path/to/a/directory

Add a dedicated LICENSE file

This makes packaging fire-hpp easier and makes it also easier to automatically detect and identify the license

licensing

This is such a great repo, really love what you're doing! Even in the early stages this project is useful and ready for certain production environments.

It's difficult to consume though without understanding what the licensing requirements will be.

The description string does not work as documented

When I provide a help description string, it is used as a default instead of a description:

int fired_main(std::string file = fire::arg(0, "<first>"))

When I run this as of commit 2a562ff, I get:

$ ./build/colormapper.exe --help

Usage:
  C:\git\maph\submodules\colormapper\build\colormapper.exe [<0>]

Options:
  [<0> STRING]   [default: <first>]

<first> becomes the default value instead of the description. I tried this with an int positional argument as well std::string and it has the same problem.

Thank you for the header! This is an interesting project and it looks promising to reduce boilerplate.

Downloading the project (gtest) is unsafe unless you store and verify the cryptographic hash

Here: https://github.com/kongaskristjan/fire-hpp/blob/master/tests/CMakeLists.txt#L10

git protocol doesn't guarantee that actual downloaded file corresponds to the requested tag. Malicious git server can easily return a malicious file instead.

Please either ensure that the cryptographic hash (fingerprint) is verified, or use a pre-installed gtest.

What about "--" to separate positional arguments?

Ideally a command line parsing library should support -- to separate positional arguments from options., to support CLI interface like this:

$ touch -- -actual-file-with-dash-prefix
$ ls -l -- -actual-file-with-dash-prefix
-rw-r--r-- 1 bronek users 0 Aug  8 19:03 -actual-file-with-dash-prefix
$ rm -- -actual-file-with-dash-prefix

BTW, this is lovely library, thank you for doing it!