On 09/22/2014 08:56 AM, Michał Górny wrote: > Fixes: https://bugs.gentoo.org/show_bug.cgi?id=523182 > --- > bin/phase-functions.sh | 4 ++++ > bin/phase-helpers.sh | 4 ++++ > 2 files changed, 8 insertions(+)
Your changes appear to correctly fix the specific issue reported in bug 523182. However, I have a question about some of the other code in commit 772ed29fd9e7cf722aed943adbe33a27f250e1ff: case "$eapi" in 0|1) # EAPIs not supporting 'default' for x in pkg_nofetch src_unpack src_test ; do declare -F $x >/dev/null || \ eval "$x() { __eapi0_$x; }" done It appears as though pkg_nofetch is only defined for EAPIs 0 and 1. Doesn't it also need to be defined for all of the other EAPIs? I can see that src_unpack and src_test are defined later for the other EAPIs, but pkg_nofetch is conspicuously absent. -- Thanks, Zac