[Nut-upsdev] [nut-commits] svn commit r1376 - in trunk: . server

[Arnaud Quette]

2008/4/2, Arjen de Korte <nut+devel at de-korte.org>:
> >> By sorting this list alphabetically, you're removing the possibility to
>  >>  group commands logically. The 'calibrate.*' and 'test.*' commands are
>  >>  similar, so it is nice to list them as one group. By sorting commands
>  >>  alphabetically, the 'load.*' and 'shutdown.*' commands will be put in
>  >>  between them. This will look different for quite a few drivers.
>  > right. so the solution might be between, like using the order given in
>  > cmdvartab.
>
>
> That would be an option, although it requires quite a bit of coding effort
>  to make this work. They will be added to the list in the server as they
>  are, but the server doesn't have a means to report the position of the
>  command in cmdvartab (so it can't give a preference value here). And also
>  note that the order of the commands in cmdvartab is currently still a mess
>  too, so this won't really help either.

exactly

>  > that allows both sorting and grouping smartly.
>  > note that this is just cosmetic, and not urging for 2.2.2...
>
>
> Sorting might still be a good idea, since it will be easier to lookup in
>  the list if a command is used (you'd skip the remainder of the list if the
>  command you're looking for is already passed in the list).

that was the point. for the moment, this solution improves the situation.
for the long run, we'll have some more time to study NUT usability,
and there are other more important subjects (such as the doc and
configuration).

btw, would you be interested in working on the configuration?

Arnaud
-- 
Linux / Unix Expert R&D - MGE Office Protection Systems - http://www.mgeops.com
Network UPS Tools (NUT) Project Leader - http://www.networkupstools.org/
Debian Developer - http://people.debian.org/~aquette/
Free Software Developer - http://arnaud.quette.free.fr/