Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 97581 - Die properly if functions in DEPEND fail
Summary: Die properly if functions in DEPEND fail
Status: RESOLVED WORKSFORME
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] KDE (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-07-01 00:55 UTC by Dan Armak (RETIRED)
Modified: 2007-03-18 18:45 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Armak (RETIRED) gentoo-dev 2005-07-01 00:55:20 UTC
Quoting from gentoo-dev thread:
> > Instead of 'exit 1', qt_min_version should use die. I use that in
> > deprange and it does work inside $DEPEND.
>
> Well, it's more visible, but it doesn't stop the emerge.  I just put
> DEPEND="$(die)" into an ebuild to test.  Here is what happens.  It
> also gives you an idea of why putting functions into DEPEND is bad:
> they get evaluated A LOT.  (jump down for more message content)
I see now that in my case, emerge aborted on die() due to a side effect:
beta kchmviewer # emerge konqueror -pv

These are the packages that I would merge, in order:

Calculating dependencies   waiting for lock 
on /dev/shm/aux_db_key_temp.portage_lockfile

!!! ERROR: kde-base/konqueror-3.4.1 failed.
!!! Function deprange-list, Line 426, Exitcode 0
!!! deprange(): unsupported parameters 1 2 - BASEVER must be identical
!!! If you need support, post the topmost build error, NOT this status 
message.

 -

!!! Problem in kde-base/konqueror dependencies.
!!! too many values to unpack exceptions
^^^ Portage died due to a really bad DEPEND, but we can't count on that. So we
need to set a variable DEPEND_ERROR=<text> in deprange(), and see it and abort
in eg kde.eclass. Same applies to qt.eclass qt_min_version.
Comment 1 Jakub Moc (RETIRED) gentoo-dev 2007-03-18 18:45:57 UTC
Irrelevant bug.