You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by rj...@apache.org on 2015/03/13 13:15:40 UTC
svn commit: r1666415 - /httpd/httpd/trunk/support/rotatelogs.c
Author: rjung
Date: Fri Mar 13 12:15:40 2015
New Revision: 1666415
URL: http://svn.apache.org/r1666415
Log:
Small changes to rotatelogs:
- add "-n num" to help text
- clarify in help text, that "program is invoked"
is the "-p" case
- prevent crash with "-v" (verbose) if no
"-p prog" is used
- add one space for formatting
Modified:
httpd/httpd/trunk/support/rotatelogs.c
Modified: httpd/httpd/trunk/support/rotatelogs.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/support/rotatelogs.c?rev=1666415&r1=1666414&r2=1666415&view=diff
==============================================================================
--- httpd/httpd/trunk/support/rotatelogs.c (original)
+++ httpd/httpd/trunk/support/rotatelogs.c Fri Mar 13 12:15:40 2015
@@ -149,8 +149,9 @@ static void usage(const char *argv0, con
#if APR_FILES_AS_SOCKETS
" -c Create log even if it is empty.\n"
#endif
+ " -n num Rotate file by adding suffixes '.0', '.1', ..., '.(num-1)'.\n"
"\n"
- "The program is invoked as \"[prog] <curfile> [<prevfile>]\"\n"
+ "The program for '-p' is invoked as \"[prog] <curfile> [<prevfile>]\"\n"
"where <curfile> is the filename of the newly opened logfile, and\n"
"<prevfile>, if given, is the filename of the previously used logfile.\n"
"\n");
@@ -213,7 +214,7 @@ static void dumpConfig (rotate_config_t
fprintf(stderr, "Rotation create empty logs: %12s\n", config->create_empty ? "yes" : "no");
#endif
fprintf(stderr, "Rotation file name: %21s\n", config->szLogRoot);
- fprintf(stderr, "Post-rotation prog: %21s\n", config->postrotate_prog);
+ fprintf(stderr, "Post-rotation prog: %21s\n", config->postrotate_prog ? config->postrotate_prog : "not used");
}
/*
@@ -280,7 +281,7 @@ static void post_rotate(apr_pool_t *pool
if (config->linkfile) {
apr_file_remove(config->linkfile, newlog->pool);
if (config->verbose) {
- fprintf(stderr,"Linking %s to %s\n", newlog->name, config->linkfile);
+ fprintf(stderr, "Linking %s to %s\n", newlog->name, config->linkfile);
}
rv = apr_file_link(newlog->name, config->linkfile);
if (rv != APR_SUCCESS) {