Created attachment 335444 [details] Build log 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[1]: 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[1]: *** [test] Error 1 make[1]: 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 > 'tclsh'. Yeah, that would be the obvious solution. I'll see what I can do (from a upstream point of view).
See https://github.com/antirez/redis/pull/877 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.
