Basically same reasons as in Bug 158114 Comment #0 apply here... 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 alogrotate 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. Thanks app-antivirus/clamav-0.91 app-antivirus/clamav-0.91.1 app-antivirus/clamav-0.91.2 app-antivirus/clamav-0.92_rc2 app-backup/bacula-2.0.3 app-backup/bacula-2.2.4 app-backup/bacula-2.2.5 app-backup/bacula-2.2.5-r1 mail-filter/dspam-3.8.0-r6 mail-filter/spamassassin-fuzzyocr-3.5.1 mail-filter/spamassassin-fuzzyocr-3.5.1-r1 net-ftp/vsftpd-2.0.5-r3 net-mail/mailgraph-1.13 net-mail/mailgraph-1.14 net-proxy/squid-2.6.16 net-proxy/squid-3.0_rc1 sys-apps/qingy-0.9.6-r1 sys-cluster/vzctl-3.0.14 sys-cluster/vzctl-3.0.16 sys-cluster/vzctl-3.0.18 sys-cluster/vzctl-3.0.18-r1 sys-power/acpid-1.0.4-r5 sys-power/hibernate-script-1.95-r2 sys-power/hibernate-script-1.95-r3 sys-power/hibernate-script-1.96 sys-power/hibernate-script-1.96-r1 sys-power/hibernate-script-1.97-r1 www-servers/ocsigen-0.99.2 www-servers/ocsigen-0.99.3
Let me reiterate my position on squid's logrotate flag. This flag selects the mode in which log rotation is performed: 1) native log rotation support (a cron script that calls squid -k rotate) 2) through logrotate program I don't see how this could be done in any other way than having a USE flag for it. mail-filter/dspam-3.8.0-r6 has been removed (-r7 no longer has logrotate USE flag).
(In reply to comment #1) Thanks for clarifying. This bug is about nuking the flag in ebuilds where it does nothing else but installing a logrotate script (and adding a optional dependency). The squid usage looks perfectly valid.
hibernate-script done.
ocsigen done
clamav and mailgraph done. The flag usage in spamassassin-fuzzyocr is a bit less trivial, as it also decides on whether to enable logging, so I will leave it to maintainers (tomk + chutzpah) to modify this package.
Reassigning since Jakub has left Gentoo.
reworking this into a TRACKER bug. individual bugs are being filed. removing everybody from CC - please re-add yourself if you want to. thanks
All the bugs are here closed, bug the issue is not solved: local use flags (searching: logrotate) ************************************************************ [- ] logrotate (app-backup/bacula): Install support files for logrotate [- ] logrotate (mail-filter/spamassassin-fuzzyocr): Install support files for app-admin/logrotate [- ] logrotate (net-misc/iaxmodem): Install support files for app-admin/logrotate [- ] logrotate (net-proxy/squid): Use app-admin/logrotate for rotating logs [- ] logrotate (sys-process/acct): Install a logrotate script to rotate /var/account/*
(In reply to comment #8) > [- ] logrotate (sys-process/acct): > Install a logrotate script to rotate /var/account/* > 10 Jun 2010; Jeroen Roovers <jer@gentoo.org> acct-6.5.4-r1.ebuild, metadata.xml: Remove IUSE=logrotate (bug #198901).
(In reply to comment #8) > [- ] logrotate (net-misc/iaxmodem): > Install support files for app-admin/logrotate 07 May 2010; Stefan Briesenick <sbriesen@gentoo.org> iaxmodem-1.2.0.ebuild: installing logrotate stuff unconditionally, solving bug #258178. > [- ] logrotate (net-proxy/squid): > Use app-admin/logrotate for rotating logs Genuine use, see other comments on this bug report.
Is it still a proper way for ebuilds? To remove the flag from all places.
