Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 212876

Summary: net-misc/rsync should have logrotate script.
Product: Gentoo Linux Reporter: Anton Bolshakov <anton.bugs>
Component: New packagesAssignee: Gentoo's Team for Core System packages <base-system>
Status: RESOLVED FIXED    
Severity: enhancement CC: gentoo.org
Priority: Lowest    
Version: 2007.0   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: rsync logrotate script
dateext removed to be similar with other default log files
logrotate USE flag added

Description Anton Bolshakov 2008-03-09 23:34:56 UTC
Similar with redhat, debian and others I suggest to add logrotate use flag and script for rsync daemon.
Comment 1 Anton Bolshakov 2008-03-09 23:38:55 UTC
Created attachment 145706 [details]
rsync logrotate script

It could be good idea to rename the logfile to rsyncd.log as well.
Comment 3 Anton Bolshakov 2008-03-17 09:33:39 UTC
Created attachment 146385 [details]
dateext removed to be similar with other default log files
Comment 4 Anton Bolshakov 2008-03-17 09:34:42 UTC
Created attachment 146387 [details]
logrotate USE flag added
Comment 5 Anton Bolshakov 2008-03-17 09:37:18 UTC
Sorry for trouble. Could you review and update new files?
Comment 7 Anton Bolshakov 2008-03-17 14:02:26 UTC
Thanks. How about the proposed "logrotate" flag?
I noticed that some packages (vsftpd, for example) have it. Not a big deal, just out of curiosity.
Comment 8 SpanKY gentoo-dev 2008-03-17 14:11:10 UTC
the trend was to get rid of it rather than expand it
Comment 9 Terra 2009-11-30 20:30:18 UTC
The following line in rsyncd.logrotate will break Augeas parsing.

Broken: size=+1024k

I checked the logrotate man page, and did not see a '+' prefix being valid for 'size'.  The only one I saw was for 'tabooext'.

Fix:  size=1024k