From owner-freebsd-bugs Wed Aug 1 3:40: 9 2001 Delivered-To: freebsd-bugs@hub.freebsd.org Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 046CF37B401 for ; Wed, 1 Aug 2001 03:40:03 -0700 (PDT) (envelope-from gnats@FreeBSD.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.11.4/8.11.4) id f71Ae2x24949; Wed, 1 Aug 2001 03:40:02 -0700 (PDT) (envelope-from gnats) Date: Wed, 1 Aug 2001 03:40:02 -0700 (PDT) Message-Id: <200108011040.f71Ae2x24949@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org Cc: From: dirk.meyer@dinoex.sub.org (Dirk Meyer) Subject: Re: bin/29363: [PATCH] newsyslog can support time as extension Reply-To: dirk.meyer@dinoex.sub.org (Dirk Meyer) Sender: owner-freebsd-bugs@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.org The following reply was made to PR bin/29363; it has been noted by GNATS. From: dirk.meyer@dinoex.sub.org (Dirk Meyer) To: freebsd-gnats-submit@FreeBSD.org Cc: Subject: Re: bin/29363: [PATCH] newsyslog can support time as extension Date: Wed, 01 Aug 2001 12:33:27 +0200 The stable patch had an stale line (conflicts). here the correct version. Index: newsyslog.8 =================================================================== RCS file: /home/ncvs/src/usr.sbin/newsyslog/newsyslog.8,v retrieving revision 1.32 diff -u -r1.32 newsyslog.8 --- newsyslog.8 2001/07/30 15:17:17 1.32 +++ newsyslog.8 2001/08/01 10:28:56 @@ -28,6 +28,7 @@ .Op Fl Fnrv .Op Fl f Ar config_file .Op Fl a Ar directory +.Op Fl T Ar timeformat .Op Ar .Sh DESCRIPTION .Nm Newsyslog @@ -354,6 +355,14 @@ to trim the logs, even if the trim conditions have not been met. This option is useful for diagnosing system problems by providing you with fresh logs that contain only the problems. +.It Fl T +Time based extension +.Nm +to rename all logs that get trimmed into with an extension based on the +argument given by this option. This option can be any strfime(3) string. + This function is intended for permantely storing of logfiles. If the +patten results to an already existing file, operations breaks without +cleanup, leaving stale logs in place, that might be ovewritten. .El .Pp If additional command line arguments are given, Index: newsyslog.c =================================================================== RCS file: /home/ncvs/src/usr.sbin/newsyslog/newsyslog.c,v retrieving revision 1.37 diff -u -r1.37 newsyslog.c --- newsyslog.c 2001/07/31 16:25:55 1.37 +++ newsyslog.c 2001/08/01 10:28:59 @@ -90,7 +90,9 @@ int needroot = 1; /* Root privs are necessary */ int noaction = 0; /* Don't do anything, just show it */ int force = 0; /* Force the trim no matter what */ -char *archdirname; /* Directory path to old logfiles archive */ +int timename = 0; /* Use time of rotation as suffix */ +const char *archdirname; /* Directory path to old logfiles archive */ +const char *timearg; /* Pattern to expand into extension */ const char *conf = _PATH_CONF; /* Configuration file to use */ time_t timenow; @@ -227,7 +229,7 @@ if ((p = strchr(hostname, '.'))) { *p = '\0'; } - while ((c = getopt(argc, argv, "nrvFf:a:t:")) != -1) + while ((c = getopt(argc, argv, "nrvFf:a:t:T:")) != -1) switch (c) { case 'n': noaction++; @@ -248,6 +250,10 @@ case 'F': force++; break; + case 'T': + timename++; + timearg = optarg; + break; default: usage(); } @@ -516,6 +522,7 @@ int notified, need_notification, fd, _numdays; struct stat st; pid_t pid; + time_t now; #ifdef _IBMR2 /* @@ -633,6 +640,16 @@ if (noaction) printf("mv %s to %s\n", log, file1); else { + /* change file1 to hold the new name */ + if (timename) { + now = time( NULL ); + (void) strftime(file2, sizeof(file2), timearg, localtime( &now ) ); + if (archtodir) + (void) snprintf(file1, sizeof(file1), "%s/%s.%s", dirpart, namepart, file2); + else + (void) snprintf(file1, sizeof(file1), "%s.%s", log, file2); + } + if (archtodir) movefile(log, file1, perm, owner_uid, group_gid); @@ -683,26 +700,17 @@ "log %s not compressed because daemon not notified", log); else if (noaction) - printf("Compress %s.0\n", log); + printf("Compress %s\n", file1); else { if (notified) { if (verbose) printf("small pause to allow daemon to close log\n"); sleep(10); - } - if (archtodir) { - (void) snprintf(file1, sizeof(file1), "%s/%s", - dirpart, namepart); - if (flags & CE_COMPACT) - compress_log(file1); - else if (flags & CE_BZCOMPACT) - bzcompress_log(file1); - } else { - if (flags & CE_COMPACT) - compress_log(log); - else if (flags & CE_BZCOMPACT) - bzcompress_log(log); } + if (flags & CE_COMPACT) + compress_log(file1); + else if (flags & CE_BZCOMPACT) + bzcompress_log(file1); } } } @@ -729,7 +737,7 @@ pid_t pid; char tmp[MAXPATHLEN]; - (void) snprintf(tmp, sizeof(tmp), "%s.0", log); + (void) snprintf(tmp, sizeof(tmp), "%s", log); pid = fork(); if (pid < 0) err(1, "gzip fork"); To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-bugs" in the body of the message