On Tue, Mar 14, 2017 at 10:45:43AM +0100, Silamael wrote: > @@ -63,6 +64,11 @@ seconds. > The shortest reporting period for state changes > will be three times this value. > The default is 20. > +.It Fl f Ar file > +Read configuration from > +.Ar file > +instead of the default configuration file > +.Pa /etc/sensorsd.conf . > .It Fl d > Do not daemonize. > If this option is specified,
The -f must be documented after -d. > @@ -124,6 +125,12 @@ main(int argc, char *argv[]) > break; > case 'd': > debug = 1; > + break; > + case 'f': > + configfile = optarg; > + if (access(configfile, R_OK) != 0) > + err(1, "Cannot access configuration file %s", > + configfile); > break; > default: > usage(); All other error messages are lower case. I would write "err(1, "access configuration file %s", configfile)" as err(3) privides a reason anyway. apart from that OK bluhm@
