Completed the documentation of the new `-P' commandline option.

Additionally the `have_pidfile' variable was eliminated.

diff --git a/src/collectd.c b/src/collectd.c
index e069bfeb4ad95b394887aaa8a4ae500ef3343544..98b9a1623f77b8840e6dbf36dbd7cfa4c69b2f1e 100644 (file)
--- a/src/collectd.c
+++ b/src/collectd.c
@@ -123,6 +123,8 @@ static void exit_usage (char *name)
                        "  General:\n"
                        "    -C <file>       Configuration file.\n"
                        "                    Default: "CONFIGFILE"\n"
+                       "    -P <file>       PID-file.\n"
+                       "                    Default: "PIDFILE"\n"
 #if COLLECT_DAEMON
                        "    -f              Don't fork to the background.\n"
 #endif
@@ -281,8 +283,7 @@ int main (int argc, char **argv)
        char *configfile = CONFIGFILE;
 #if COLLECT_DAEMON
        struct sigaction sigChldAction;
-       char *pidfile    = PIDFILE;
-       int have_pidfile = 0;
+       char *pidfile    = NULL;
        pid_t pid;
        int daemonize    = 1;
 #endif
@@ -320,8 +321,7 @@ int main (int argc, char **argv)
                                break;
 #if COLLECT_DAEMON
                        case 'P':
-                               pidfile      = optarg;
-                               have_pidfile = 1;
+                               pidfile = optarg;
                                break;
                        case 'f':
                                daemonize = 0;
@@ -373,7 +373,7 @@ int main (int argc, char **argv)
        sigChldAction.sa_handler = SIG_IGN;
        sigaction (SIGCHLD, &sigChldAction, NULL);
 
-       if ((1 != have_pidfile)
+       if ((pidfile == NULL)
                        && ((pidfile = cf_get_option ("PIDFile", PIDFILE)) == NULL))
        {
                fprintf (stderr, "Cannot obtain pidfile. This shoud not happen. Ever.");

diff --git a/src/collectd.conf.pod b/src/collectd.conf.pod
index f3e2585659504f26af3d335bf37e0bacf659726d..7e4b858e84b09ad0db0a2fee0d7b225af3dd0a04 100644 (file)
--- a/src/collectd.conf.pod
+++ b/src/collectd.conf.pod
@@ -49,7 +49,8 @@ will be mostly useless. The names of the plugins are listed in L<collectd(1)>.
 =item B<PIDFile> I<File>
 
 Sets where to write the PID file to. This file is overwritten when it exists
-and deleted when the program ist stopped. Available in B<all modes>.
+and deleted when the program ist stopped. Some init-scripts might override this
+setting using the B<-P> commandline option. Available in B<all modes>.
 
 =item B<DataDir> I<Directory>
 

diff --git a/src/collectd.pod b/src/collectd.pod
index 095463c139c99b5cf86bf9531439d27adf34a1cb..0e1adc50c6f1c3e9a9e772bab1458936cdf135f2 100644 (file)
--- a/src/collectd.pod
+++ b/src/collectd.pod
@@ -113,7 +113,9 @@ directory.
 =item B<-P> I<E<lt>pid-fileE<gt>>
 
 Specify an alternative pid file. This overwrites any settings in the config 
-file.
+file. This is thought for init-scripts that require the PID-file in a certain
+directory to work correctly. For everyday-usage use the B<PIDFile>
+config-option.
 
 =item B<-f>