Bug 83477

Summary:	app-antivirus/clamav-0.83 fails to ./configure when USEing milter
Product:	Gentoo Linux	Reporter:	Drake Wyrm <lilwyrm>
Component:	Current packages	Assignee:	Net-Mail Packages <net-mail+disabled>
Status:	VERIFIED FIXED
Severity:	normal	CC:	antivirus
Priority:	High
Version:	2004.3
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	fix the bug by adding a DEPEND Fix the bug by tinkering with the myconf logic

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.