Command line description for --interface

[thomasw-mitutoyo-ctl]

The help says that there's an optional parameter

[--interface]

Later it says

--interface , -i      Serial interface type. Allowed values are: i2c, spi,
                        bitbang (default: i2c)

So, if that parameter needs values, it should read

[--interface VALUE]

where VALUE can be replaced by i2c, spi or bitbang.

[thomasw-mitutoyo-ctl]

The same applies for --display, --rotate, --block-orientation, --mode, --framebuffer, --backlight-active and --transform

[thijstriemstra]

Probably related to the metavar='', e.g. https://github.com/rm-hull/luma.core/blob/master/luma/core/cmdline.py#L246 Not sure why metavar is used there.

[thijstriemstra]

Yea, removing metavar fixes it:

  --interface {i2c,spi,bitbang}, -i {i2c,spi,bitbang}
                        Serial interface type. Allowed values are: i2c, spi,
                        bitbang (default: i2c)

but introduces a lot of clutter. Specifying an uppercase metavar seems to produce desired result:

  --interface INTERFACE, -i INTERFACE
                        Serial interface type. Allowed values are: i2c, spi,
                        bitbang (default: i2c)

I'll create a PR.

[thijstriemstra]

The help says that there's an optional parameter

it's optional because you can use the --config parameter and specify all params there.