Bug 78768 - mail-filter/dspam-3.4_beta1 wants mail-mta/courier
Bug#: 78768 Product:  Gentoo Linux Version: unspecified Platform: All
OS/Version: All Status: RESOLVED Severity: normal Priority: P2
Resolution: FIXED Assigned To: st_lim@gentoo.org Reported By: langthang@gentoo.org
Component: Ebuilds
URL: 
Summary: mail-filter/dspam-3.4_beta1 wants mail-mta/courier
Keywords:  
Status Whiteboard: 
Opened: 2005-01-19 21:03 0000
Description:   Opened: 2005-01-19 21:03 0000
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

------- Comment #1 From Roy Marples (RETIRED) 2005-01-20 02:39:56 0000 -------
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

------- Comment #2 From Karl Greunke 2005-01-20 11:43:09 0000 -------
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 #

------- Comment #3 From Marius Mauch (RETIRED) 2005-01-20 19:58:12 0000 -------
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.

------- Comment #4 From Lim Swee Tat (RETIRED) 2005-01-21 07:24:36 0000 -------
Trying to commit to CVS.

------- Comment #5 From Lim Swee Tat (RETIRED) 2005-01-21 21:56:52 0000 -------
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

------- Comment #6 From Tuan Van (RETIRED) 2005-01-21 22:03:52 0000 -------
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 )

------- Comment #7 From Lim Swee Tat (RETIRED) 2005-02-16 19:26:21 0000 -------
I think I wanted it so that you either build for mysql, postgres or oracle, or
sqlite.  A minor change should fix it.