Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 330875 - mail-client/balsa-2.4.7.ebuild issues unnecessary ewarn
Summary: mail-client/balsa-2.4.7.ebuild issues unnecessary ewarn
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High minor (vote)
Assignee: Gentoo Linux Gnome Desktop Team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2010-08-02 16:48 UTC by Jack
Modified: 2010-08-30 20:40 UTC (History)
0 users

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 Jack 2010-08-02 16:48:57 UTC 
The ebuild contains
	if use webkit || use gtkhtml ; then
		ewarn "Only one html widget can be enabled at the same time."
		ewarn "Selecting gtkhtml by default."
Shouldn't the "||" be "&&" ?

I have webkit specified and gtkhtml is not in make.conf or package.use, so the warning is irrelevant for me.  In fact, balsa actually does build correctly with webkit.

emerge -pv mail-client/balsa gives me
mail-client/balsa-2.4.7  USE="crypt gtkspell libnotify ssl webkit -doc -gnome -gtkhtml -kerberos -ldap -networkmanager -rubrica -sqlite -xface"

Reproducible: Always

Steps to Reproduce:
1.emerge mail-client/balsa-2.4.7 (with use flag webkit set and gtkhtml not set)
2.
3.

Actual Results:  
emerge issues the warning about only one html widget at a time.

Expected Results:  
that warning should not be issued.
Comment 1 Gilles Dartiguelongue (RETIRED) gentoo-dev 2010-08-02 16:49:56 UTC 
You are correct.
Comment 2 Gilles Dartiguelongue (RETIRED) gentoo-dev 2010-08-30 20:40:29 UTC 
+*balsa-2.4.8 (30 Aug 2010)
+
+  30 Aug 2010; Gilles Dartiguelongue <eva@gentoo.org>
+  -balsa-2.4.1-r1.ebuild, -files/balsa-2.4.1-vcard.patch,
+  balsa-2.4.7.ebuild, +balsa-2.4.8.ebuild:
+  Version bump. Clean up old revisions. Make ewarn display a bit more
+  logical, bug #330875.

Thanks for reporting.