Granted, the ebuild only dies if PV isn't defined, but it's rather nasty on bash-based tools like eshowkw(1)[1] which source ebuilds without defining PV and other variables that real package managers construct from file names etcetera: jer@astrid cvs/gentoo-x86/dev-util/gprof2dot $ /usr/bin/eshowkw Keywords for dev-util/gprof2dot: | a a a a h i m m p p s s s s x x p x x h i x m a a i x p x x m x p x x s s x x x x | l m m r p a 6 i p p 3 h p p 8 8 p 8 6 p a 8 i m r a 8 p 8 6 6 8 p 8 6 p p 6 8 8 8 | p d d m p 6 8 p c c 9 a a 6 6 c 6 4 p 6 6 p d m 6 6 c 6 4 8 6 c 6 4 a a 4 6 6 6 | h 6 6 a 4 k s 6 0 r r - - - - a 4 - s 6 - 4 - - - - k - - - - r r - - - - | a 4 4 4 c c f a f f - - i - 4 l - l m m m - n o o o c c s s w c | - - b i r r h h n i - i l i a a a m e p p p - 6 o o i y | f f s x e e p p t r l n i n c c c i t e e e s 4 l l n g | b b d e e u u e i i u n u o o o n b n n n o - a a n w | s s b b x x r x n x u x s s s t s b b b l s r r t i | d d s s i u x d s s s a o i i n | d d x x d d d r l s s | i a | s r | i | s ------------+---------------------------------------------------------------------------------- 0_p20100216 | jer@astrid cvs/gentoo-x86/dev-util/gprof2dot $ I don't think any other ebuild checks developer sanity this way, certainly not in global scope, and certainly not with a die() attached. [1] From app-portage/gentoolkit-dev.
Point taken. Fixing bug 337088 fixes this one as a side effect.
