268398 – man ebuild(5) should document econf as ./configure ... || die

Bug 268398 - man ebuild(5) should document econf as ./configure ... || die

Summary: man ebuild(5) should document econf as ./configure ... || die

Status:	RESOLVED FIXED

Alias:	None

Product:	Portage Development
Classification:	Unclassified
Component:	Documentation (show other bugs)
Hardware:	All Linux

Importance:	High minor (vote)
Assignee:	Portage team

URL:
Whiteboard:
Keywords:	InVCS

Depends on:
Blocks:	210077 268228
	Show dependency tree

Reported:	2009-05-03 09:59 UTC by Martin Väth
Modified:	2009-05-03 21:53 UTC (History)
CC List:	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 Martin Väth 2009-05-03 09:59:20 UTC

I suppose the reason why many ebuilds use the redundant
  econf || die
is that the "die"-part is undocumented for econf in the manpage of ebuild(5).
Quite the opposite, it is stated there that econf is a "replacement" for
configure and performs "${ECONF_SOURCE:-.}/configure ..."
I suggest that "|| die" be added there (even if the actual implementation is
slightly different).

Comment 1 Zac Medico gentoo-dev

2009-05-03 20:14:53 UTC

Thanks, this is in svn r13600.

Comment 2 Zac Medico gentoo-dev

2009-05-03 21:53:47 UTC

This is fixed in 2.1.6.13 and 2.2_rc33.