pixelb / crudini

A utility for manipulating ini files
GNU General Public License v2.0
443 stars 63 forks source link

--ini-options=nospace not recognized in crudini 0.9.3 #92

Closed DanielFlaum closed 1 year ago

DanielFlaum commented 1 year ago

With crudini 0.9.3 on Fedora Linux 37:

fish ⯈ crudini --ini-options=nospace --set example.ini '' key value
option --ini-options not recognized
A utility for manipulating ini files

Usage: crudini --set [OPTION]...   config_file section   [param] [value]
  or:  crudini --get [OPTION]...   config_file [section] [param]
  or:  crudini --del [OPTION]...   config_file section   [param] [list value]
  or:  crudini --merge [OPTION]... config_file [section]

Options:

  --existing[=WHAT]  For --set, --del and --merge, fail if item is missing,
                       where WHAT is 'file', 'section', or 'param', or if
                       not specified; all specified items.
  --format=FMT       For --get, select the output FMT.
                       Formats are sh,ini,lines
  --inplace          Lock and write files in place.
                       This is not atomic but has less restrictions
                       than the default replacement method.
  --list             For --set and --del, update a list (set) of values
  --list-sep=STR     Delimit list values with "STR" instead of " ,"
  --output=FILE      Write output to FILE instead. '-' means stdout
  --verbose          Indicate on stderr if changes were made
  --help             Write this help to stdout
  --version          Write version to stdout

And sure enough, even though the usage information at least appears correct at https://github.com/pixelb/crudini/blob/master/crudini.py#L541, if I download the source code from https://github.com/pixelb/crudini/releases/tag/0.9.3 and look inside, it doesn't seem to be there.

Perhaps the build/release process wasn't carried out quite right?

pixelb commented 1 year ago

Right, --ini-options=nospace was not released yet, only being in commit 0.9.3-16-g65ace17 I'll do a 0.9.4 release over the next couple of days