Created attachment 335444 [details]
I've just tried to build dev-db/redis with FEATURES=test and I get:
>>> Test phase [test]: dev-db/redis-2.6.8
cd src && make test
make: Entering directory `/tmp/portage/dev-db/redis-2.6.8/work/redis-2.6.8/src'
which: no tclsh8.5 in (/usr/lib64/distcc/bin:/usr/lib/portage/bin/ebuild-helpers:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/opt/bin:/usr/x86_64-pc-linux-gnu/gcc-bin/4.7.2:/usr/i686-pc-linux-gnu/gcc-bin/4.5.0:/opt/intel/composerxe-2013_update1.1.117/bin/intel64:/opt/intel/composerxe-2013_update1.1.117/mpirt/bin/intel64:/opt/intel/composerxe-2013_update1.1.117/bin/ia32:/opt/intel/composerxe-2013_update1.1.117/mpirt/bin/ia32)
You need 'tclsh8.5' in order to run the Redis test
make: *** [test] Error 1
make: Leaving directory `/tmp/portage/dev-db/redis-2.6.8/work/redis-2.6.8/src'
make: *** [test] Error 2
I would guess that it fails because I have tcl-8.6 installed.
As a note: it seems that dumb-replacing 'tclsh8.5' with 'tclsh8.6' in all files makes it work well. I think the correct solution would be to use bare 'tclsh'.
Johan, can you take that up with upstream?
(In reply to comment #1)
> As a note: it seems that dumb-replacing 'tclsh8.5' with 'tclsh8.6' in all
> files makes it work well. I think the correct solution would be to use bare
Yeah, that would be the obvious solution. I'll see what I can do (from a upstream point of view).
Hopefully we get it merged upstream; otherwise we'll just have to pull it into files like we do with the shared patches.
An updated (and bumped) ebuild is available in bug 453606.
Fixed in 2.6.9.