65102 – SA-3 ebuild sets DATADIR badly and breaks amavisd and others.

Bug 65102 - SA-3 ebuild sets DATADIR badly and breaks amavisd and others.

Summary: SA-3 ebuild sets DATADIR badly and breaks amavisd and others.

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	New packages (show other bugs)
Hardware:	All Linux

Importance:	High major (vote)
Assignee:	Gentoo Perl team

URL:	http://bugs.gentoo.org/show_bug.cgi?i...
Whiteboard:
Keywords:

Duplicates (1):	65182 (view as bug list)
Depends on:
Blocks:

Reported:	2004-09-23 08:33 UTC by klavs klavsen
Modified:	2004-09-27 06:14 UTC (History)
CC List:	5 users (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description klavs klavsen 2004-09-23 08:33:26 UTC

We agreed here: http://bugs.gentoo.org/show_bug.cgi?id=64462

That the real problem is the SA-3 ebuild:
(from SA-3 ebuild)
# - Set DATADIR to a subdirectory so other ebuilds which offer rules can store
#   their stuff below the dir, too.
myconf="SYSCONFDIR=/etc DATADIR=/usr/share/spamassassin/core"

This makes no sense to me - if other ebuilds did that - SA-3 wouldn't look for them there - otherwise we wouldn't have the problem with amavisd-new that we have.
ie. ALL rules needs has to be in the same dir AFAIK - and atleast the SA-core rules, should be where they are suppose to be.



Reproducible: Always
Steps to Reproduce:
1.
2.
3.

Comment 1 Michael Cummings (RETIRED) gentoo-dev

2004-09-24 09:25:21 UTC

-r1 updated. This was a mistake on my part - sorry about that. No longer sets the DATADIR, defaults to /usr/share/spamassassin like the good old days. Let me know if this closes this out for you,

Mike

Comment 2 Michael Cummings (RETIRED) gentoo-dev

2004-09-25 06:14:47 UTC

*** Bug 65182 has been marked as a duplicate of this bug. ***

Comment 3 Michael Cummings (RETIRED) gentoo-dev

2004-09-26 05:11:55 UTC

Hoping for any more comments from anyone before closing this out...

Comment 4 Michael Cramer 2004-09-26 05:59:55 UTC

spamd works now well with this ebuild, no longer need to copy the rules

Comment 5 Michael Cummings (RETIRED) gentoo-dev

2004-09-26 06:35:26 UTC

kl?

Comment 6 klavs klavsen 2004-09-26 06:42:54 UTC

IMHO SA is now okay - and I see no reason why amavisd-new shouldn't work - although I haven't had time to test.

Comment 7 Michael Cummings (RETIRED) gentoo-dev

2004-09-26 06:53:47 UTC

OK. Will close this out - but please reopen this bug if the problem persists (if its a new problem, you know the mantra, open a new bug and all that jazz). Thanks for your patience this week,

Mike

Comment 8 Malte S. Stretz 2004-09-27 06:14:00 UTC

Sorry, been out a week in Cologne.

Setting DATADIR to a different value was actually done by me; the idea behind it:  You MUST NOT install rules into the DATADIR used by SpamAssassin.  The correct way to install user defined rules is to put them into /etc/mail/spamassassin.  But that dir is protected against changes done by portage.  So where should rules ebuilds put them (in a way that they can be unmerged again)?  That's why I moved the DATADIR to .../core.

What I didn't notice was that the DATADIR set in the install process is hardcoded into the apps, not the modules so each app which uses the modules has to know the DATADIR in use :-/  That's the point which doesn't make any sense at all (and I'll fix for 3.1).

Just to clean the issue up a bit :)