Changeset 4987

Timestamp:

10/25/07 11:50:17 (6 years ago)

Author:

khali

Message:

"sensord" no longer needs a default configuration file name. If no
file name is provided on the command line, use the libsensors default.

Location:

lm-sensors/branches/lm-sensors-3.0.0

Files:

• CHANGES (modified) (1 diff)
• prog/sensord/args.c (modified) (3 diffs)
• prog/sensord/lib.c (modified) (3 diffs)
• prog/sensord/sensord.8 (modified) (1 diff)

lm-sensors/branches/lm-sensors-3.0.0/CHANGES

@@ -5 +5 @@
   libsensors: Add a default configuration file
   sensord: Reload config on SIGHUP rather than automatically
+           No default configuration file name
   sensors: Fix file handle leak on library initialization error
            Handle the case where voltage measurements aren't available

lm-sensors/branches/lm-sensors-3.0.0/prog/sensord/args.c

@@ -31 +31 @@
 #include "version.h"
 
-#define DEFAULT_CONFIG_FILE     ETCDIR "/sensors.conf"
-
 #define MAX_CHIP_NAMES 32
 
 int isDaemon = 0;
-const char *sensorsCfgFile = DEFAULT_CONFIG_FILE;
+const char *sensorsCfgFile = NULL;
 const char *pidFile = "/var/run/sensord.pid";
 const char *rrdFile = NULL;
@@ -105 +103 @@
   "  -T, --rrd-no-average      -- switch RRD in non-average mode\n"
   "  -r, --rrd-file <file>     -- RRD file (default <none>)\n"
-  "  -c, --config-file <file>  -- configuration file (default " DEFAULT_CONFIG_FILE ")\n"
+  "  -c, --config-file <file>  -- configuration file\n"
   "  -p, --pid-file <file>     -- PID file (default /var/run/sensord.pid)\n"
   "  -f, --syslog-facility <f> -- syslog facility to use (default local4)\n"
@@ -130 +128 @@
   "  -s, --set                 -- execute set statements (root only)\n"
   "  -r, --rrd-file <file>     -- only update RRD file\n"
-  "  -c, --config-file <file>  -- configuration file (default " DEFAULT_CONFIG_FILE ")\n"
+  "  -c, --config-file <file>  -- configuration file\n"
   "  -d, --debug               -- display some debug information\n"
   "  -v, --version             -- display version and exit\n"

lm-sensors/branches/lm-sensors-3.0.0/prog/sensord/lib.c

@@ -39 +39 @@
   int ret = 0;
 
-  if (!strcmp (cfgPath, "-")) {
+  if (cfgPath && !strcmp (cfgPath, "-")) {
     if (!reload) {
       if ((ret = sensors_init (stdin))) {
@@ -47 +47 @@
       }
     }
-  } else if (stat (cfgPath, &stats) < 0) {
+  } else if (cfgPath && stat (cfgPath, &stats) < 0) {
     sensorLog (LOG_ERR, "Error stating sensors configuration file: %s", cfgPath);
     ret = 10;
@@ -55 +55 @@
       sensors_cleanup ();
     }
-    if (!(cfg = fopen (cfgPath, "r"))) {
+    if (cfgPath && !(cfg = fopen (cfgPath, "r"))) {
       sensorLog (LOG_ERR, "Error opening sensors configuration file: %s", cfgPath);
       ret = 11;
     } else if ((ret = sensors_init (cfg))) {
       sensorLog (LOG_ERR, "Error loading sensors configuration file %s: %s",
-                 cfgPath, sensors_strerror (ret));
+                 cfgPath ? cfgPath : "(default)", sensors_strerror (ret));
       ret = 11;
     }

lm-sensors/branches/lm-sensors-3.0.0/prog/sensord/sensord.8

@@ -80 +80 @@
 Specify a
 .BR libsensors (3)
-configuration file. If no file is specified, the default configuration file
-is used.
+configuration file. If no file is specified, the libsensors default
+configuration file is used.
 
 .IP "-p, --pid-file file"