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

Bug 258182

Summary: mail-mta/exim invalid use of logrotate USE flag
Product: Gentoo Linux Reporter: Thilo Bangert (RETIRED) (RETIRED) <bangert>
Component: New packagesAssignee: Net-Mail Packages <net-mail+disabled>
Status: RESOLVED FIXED    
Severity: QA    
Priority: High    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 198901    

Description Thilo Bangert (RETIRED) (RETIRED) gentoo-dev 2009-02-08 18:31:13 UTC
Having a use flag for installing one plaintext file doesn't make sense.

- ebuilds don't handle this consistently (some just use this flag for
doins/newins a script to /etc/logrotate.d, some add app-admin/logrotate
dependency, some install the scripts unconditionally)

- users can use INSTALL_MASK if diskspace is an issue for them (extremely
unlikely), plus the installed scripts have no effect until you install
logrotate anyway, so if you don't want this functionality, just don't install
it, if you do, emerge logrotate and you don't have to care about anything else
and no need to re-emerge anything).

(BTW, the usage of this flag in net-proxy/squid really is special, don't quite
understand why not just use the rotate functionality squid itself has.)

So, please nuke the use flag and install the stuff unconditionally.

see #198901 and gentoo-dev for more info.
Comment 1 Colin Morey (RETIRED) gentoo-dev 2009-02-09 16:45:43 UTC
it is a pre-existing USE flag, and as such, it makes sense to me to re-use what is available. Exim has a pre-existing facility for cycling logs, (like squid). However some users have requested logrotate integration.

I would like to follow the principle of least surprise here, a new emerge shouldn't start to roll logs twice if the user already has logrotate installed for some other reason.

So, rather than do it unconditionally, I'd rather follow Squid's current practice of adding a file to cron.d instead. 
Comment 2 Torsten Veller (RETIRED) gentoo-dev 2009-06-24 13:33:17 UTC
Reassign to net-mail herd as peitolm retires
Comment 3 Fabian Groffen gentoo-dev 2009-07-24 09:18:54 UTC
flag nuked