Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 83477 - app-antivirus/clamav-0.83 fails to ./configure when USEing milter
Summary: app-antivirus/clamav-0.83 fails to ./configure when USEing milter
Status: VERIFIED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Net-Mail Packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-02-27 10:19 UTC by Drake Wyrm
Modified: 2010-03-27 23:21 UTC (History)
1 user (show)

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


Attachments
fix the bug by adding a DEPEND (clamav-0.83.ebuild.right.diff,257 bytes, patch)
2005-02-27 10:21 UTC, Drake Wyrm
Details | Diff
Fix the bug by tinkering with the myconf logic (clamav-0.83.ebuild.wrong.diff,341 bytes, patch)
2005-02-27 10:22 UTC, Drake Wyrm
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Drake Wyrm 2005-02-27 10:19:41 UTC
In order to build clamav-milter, libmilter from sendmail is needed. If that is not available, ./configure for clamav fails with:

	checking for mi_stop in -lmilter... no
	checking for library containing strlcpy... no
	checking for mi_stop in -lmilter... no
	configure: error: Cannot find libmilter

The simplest fix would be to add "milter? (mail-mta/sendmail)" to the DEPEND string. This would follow the fail-early philosophy.

The most user-friendly fix would be to change the --enable-milter logic to ignore "USE milter" if sendmail is not installed.

Personally, I'd prefer the former option. I'd rather see portage fail to do what I tell it to, than have portage quietly fail to do what I tell it to and succeed at something different.
Comment 1 Drake Wyrm 2005-02-27 10:21:14 UTC
Created attachment 52248 [details, diff]
fix the bug by adding a DEPEND

I prefer this fix.
Comment 2 Drake Wyrm 2005-02-27 10:22:19 UTC
Created attachment 52249 [details, diff]
Fix the bug by tinkering with the myconf logic

I prefer the other fix.
Comment 3 Andrej Kacian (RETIRED) gentoo-dev 2005-02-27 12:34:36 UTC
From what I understand, libmilter only gets built when sendmail is compiled with "milter" USE flag enabled. Unfortunately, portage can't handle this kind of dependencies yet. See bug #2272 for more info.

A (bit clumsy) workaround is built_with_use() function from eutils eclass. I think using the "milter? ..." depend (your preferred fix) along with a built_with_use test for milter-enabled sendmail in pkg_setup() would be the best solution. Emerge should abort the emerge process with a suitable message to the user.

I'm going to do this - anyone, yell here if you think it's not a good idea.
Comment 4 Tuan Van (RETIRED) gentoo-dev 2005-02-27 15:07:46 UTC
> A (bit clumsy) workaround is built_with_use() function from eutils eclass
still better than nothing ;)
Comment 5 Andrej Kacian (RETIRED) gentoo-dev 2005-02-27 15:22:53 UTC
Ok, modified 0.83 is in CVS, could you please test it when it reaches your mirror? I do not have any sendmail setup available for this.
Comment 6 Drake Wyrm 2010-03-27 23:12:38 UTC
Long since fixed.
Comment 7 Drake Wyrm 2010-03-27 23:20:49 UTC
Reopening to close properly.