Bug 212876 – net-misc/rsync should have logrotate script.

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[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 2 SpanKY 2008-03-16 07:22:15 UTC]

included with rsync-3.0.0-r1

http://sources.gentoo.org/net-misc/rsync/files/rsyncd.logrotate?rev=1.1
http://sources.gentoo.org/net-misc/rsync/rsync-3.0.0-r1.ebuild?r1=1.1&r2=1.2

[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 6 SpanKY 2008-03-17 10:55:51 UTC]

added to cvs

http://sources.gentoo.org/net-misc/rsync/files/rsyncd.logrotate?r1=1.1&r2=1.2

[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 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