Bug 169885
Summary: | logrotate cleanups and indentations | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Mateus César Gröess <mateuscg> | ||||||
Component: | logrotate | Assignee: | Peter Vrabec <pvrabec> | ||||||
Status: | CLOSED RAWHIDE | QA Contact: | |||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | medium | ||||||||
Version: | rawhide | CC: | me, ro | ||||||
Target Milestone: | --- | ||||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2005-10-12 13:00:44 UTC | Type: | --- | ||||||
Regression: | --- | Mount Type: | --- | ||||||
Documentation: | --- | CRM: | |||||||
Verified Versions: | Category: | --- | |||||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||
Cloudforms Team: | --- | Target Upstream Version: | |||||||
Embargoed: | |||||||||
Attachments: |
|
Description
Mateus César Gröess
2005-10-04 18:40:30 UTC
Created attachment 119619 [details]
Cleanups and indentation fixes
I understand the first code(whole context), it drops old logs. Example: rotatecount = 2 foo foo-20051008 foo-20051009 foo-20051010 foo-20051008 should have not existed, because rotatecount==2, but if it exists(somehow) it is removed in case mail address is provided. I guess second code should do something similar but don't not how. I suggest to do indention by "indent -bsd"(I vote for bsd style) Created attachment 119770 [details]
Cleanups only, without the indentation changes.
Ok, I just proposed changes to correct bad indentation in some lines, haven't
used "indent". I attached another patch with cleanups only and a change to
second "if" described above, to make it act like the first. Maybe it's better
the maintainer do indentation changes according with his taste.
well, according to the manpage, a to-be-removed logfile is - mailed out if an address is given - removed then, regardless if the address was given or not in that sense, I guess it is perfectly ok to remove the logfile in both code snippets, regardless if logAdress is set or not. Fixed in logrotate-3.7.2-6 in devel. Indent will by apllied in next release 3.7.3. |