295330 – econf does not need to || die

Bug 295330 - econf does not need to || die

Summary: econf does not need to || die

Status:	RESOLVED FIXED

Alias:	None

Product:	Documentation
Classification:	Unclassified
Component:	[OLD] Developer Handbook (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Gentoo Community Relations Team

URL:	http://www.gentoo.org/proj/en/devrel/...
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2009-12-01 13:30 UTC by Justin Lecher (RETIRED)
Modified:	2010-07-11 21:39 UTC (History)
CC List:	2 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 Justin Lecher (RETIRED) gentoo-dev

2009-12-01 13:30:33 UTC

The ebuild HowTo tells that econf needs a || die. That's wrong.

Code Listing 2.5: Conditionals based on USE-settings

DEPEND="X? ( >=x11-base/xfree-4.3 )
mysql? ( >=dev-db/mysql-3.23.49 )
apache2? ( >=net-www/apache-2 )
!apache2? ( =net-www/apache-1* )"

src_compile() {
  econf \
    $(use_enable X x11) \
    $(use_enable mysql) \
    || die "Error: econf failed!"
  emake || die "Error: emake failed!"
}

Comment 1 Justin Lecher (RETIRED) gentoo-dev

2009-12-30 20:00:07 UTC

Perhaps not only because of this many devs are using this in the tree. This affects at least >3000 ebuilds.

Comment 2 Markos Chandras (RETIRED) gentoo-dev

2010-07-11 21:39:24 UTC

fixed