Bugzilla Bug 78768

mail-filter/dspam-3.4_beta1 wants mail-mta/courier but I have postfix and
courier-imap.

# emerge dspam -vp

These are the packages that I would merge, in order:

Calculating dependencies ...done!
[blocks B     ] net-mail/courier-imap (from pkg mail-mta/courier-0.48.1)
[blocks B     ] mail-mta/postfix (from pkg mail-mta/courier-0.48.1)
[ebuild  N    ] mail-mta/courier-0.48.1  +crypt -fax +ipv6 +ldap -mailwrapper
+mysql +nls -norewrite +pam +postgres +spell (-uclibc) 5,737 kB
[ebuild  N    ] app-admin/logrotate-3.7  (-selinux) 0 kB
[ebuild  N    ] mail-filter/dspam-3.4_beta1  -cyrus -debug -exim -large-domain
-maildrop +mysql -neural -oci8 +postgres -procmail +sqlite 0 kB

Created an attachment (id=49027) [details]
Fixes maildrop USE flag to use maildrop OR courier

The build has a mistake - it assumes that if the USE flag maildrop is set then
depend on maildrop, otherwise courier.

Instead, it should read if USE flag maildrop is set then depend on maildrop OR
courier.

This patch is for 3.3.8, but should apply cleanly to 3.3.11 and others

This bug applies to all current DSPAM Ebuilds

grep maildrop\? /usr/portage/mail-filter/dspam/*.ebuild
/usr/portage/mail-filter/dspam/dspam-3.2.4.ebuild:              maildrop? ( >=mail-filter/maildrop-1.5.3 ) || ( >=mail-mta/courier-0.46 )
/usr/portage/mail-filter/dspam/dspam-3.2.6.ebuild:              maildrop? ( >=mail-filter/maildrop-1.5.3 ) || ( >=mail-mta/courier-0.46 )
/usr/portage/mail-filter/dspam/dspam-3.3.11.ebuild:             maildrop? ( >=mail-filter/maildrop-1.5.3 ) || ( >=mail-mta/courier-0.46 )
/usr/portage/mail-filter/dspam/dspam-3.3.8.ebuild:              maildrop? ( >=mail-filter/maildrop-1.5.3 ) || ( >=mail-mta/courier-0.46 )
/usr/portage/mail-filter/dspam/dspam-3.4_beta1.ebuild:          maildrop? ( >=mail-filter/maildrop-1.5.3 ) || ( >=mail-mta/courier-0.46 )
tux config #

Well, this package is completely broken IMO:
- the DEPEND is somewhere between broken and plain invalid, see ebuild(5) what the || operator really does. I don't think you wanted to use it, but it's hard to guess what you wanted to do there in the first place (hint: the patch doesn't fix anything)
- the cyrus and exim use flags should be removed, it's not our business to add trivial flags just for minor changes to config files. And even if they stay, they shouldn't affect DEPEND but RDEPEND. But I strongly recommend to remove them.
- system users should have a low uid in general, usually all users with a uid > 500 or 1000 are considered real users.

Trying to commit to CVS.

I've made a commit to CVS.  Does this solve all the issues?  I'm reopening the
bug so that someone can tell me what to do about this. :)

Regards
Lim Swee Tat

mysql? ( >=dev-db/mysql-3.23 ) || ( >=sys-libs/db-4.0 ) is wrong syntax. I am
not sure what are you trying to do there.
if mysql? then either >=dev-db/mysql-3.23 or >=sys-libs/db-4.0 satify the dep:

mysql? || (
        >=mail-filter/maildrop-1.5.3
        >=mail-mta/courier-0.46
)

or

if mysql is missing, then depends on >=sys-libs/db-4.0?

mysql? ( >=dev-db/mysql-3.23 ) !mysql? ( >=sys-libs/db-4.0 )

I think I wanted it so that you either build for mysql, postgres or oracle, or
sqlite.  A minor change should fix it.