|Summary:||dev-db/redis[test]: You need 'tclsh8.5' in order to run the Redis test|
|Product:||Gentoo Linux||Reporter:||Michał Górny <mgorny>|
|Component:||New packages||Assignee:||Johan Bergström <bugs>|
|Severity:||normal||CC:||djc, lu_zero, robbat2|
|Package list:||Runtime testing required:||---|
|Bug Depends on:||453606|
Description Michał Górny 2013-01-13 11:22:04 UTC
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: 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.
Comment 1 Michał Górny 2013-01-13 11:35:07 UTC
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'.
Comment 2 Dirkjan Ochtman 2013-01-13 11:37:38 UTC
Johan, can you take that up with upstream?
Comment 3 Johan Bergström 2013-01-13 19:50:30 UTC
(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).
Comment 4 Johan Bergström 2013-01-13 20:21:51 UTC
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.
Comment 5 Johan Bergström 2013-01-23 03:34:39 UTC
An updated (and bumped) ebuild is available in bug 453606.
Comment 6 Dirkjan Ochtman 2013-01-24 12:11:43 UTC
Fixed in 2.6.9.