llvm dev - May 2009 - [LLVMdev] make TEST=ipodbgopt in sqlite3: problem with TCLSH

On Linux, when I run "make TEST=ipodbgopt" in
MultiSource/Applications/sqlite3 I get:

make[4]: Entering directory
`/home/foad/llvm/poo/build/test-suite/MultiSource/Applications/sqlite3'
Compiling shell.c to Output/shell.bc
/home/foad/svn/llvm-project/test-suite/trunk/MultiSource/Applications/sqlite3/speedtest.tcl
make[4]: execvp:
/home/foad/svn/llvm-project/test-suite/trunk/MultiSource/Applications/sqlite3/speedtest.tcl:
Permission denied
make[4]: ***
[/home/foad/svn/llvm-project/test-suite/trunk/MultiSource/Applications/sqlite3/test15.sql]
Error 127
make[4]: Leaving directory
`/home/foad/llvm/poo/build/test-suite/MultiSource/Applications/sqlite3'
make[3]: *** [sqlite3/.maketest] Error 2
make[3]: Leaving directory
`/home/foad/llvm/poo/build/test-suite/MultiSource/Applications'
make[2]: *** [Applications/.maketest] Error 2
make[2]: Leaving directory
`/home/foad/llvm/poo/build/test-suite/MultiSource'
make[1]: *** [MultiSource/.maketest] Error 2
make[1]: Leaving directory `/home/foad/llvm/poo/build/test-suite'
make: *** [test] Error 2

... because the makefile uses $(TCLSH), which hasn't been set to
anything. How is this supposed to work? Does the attached patch look
reasonable? It seems to work for me.

Alternatively, smalltest.tcl and speedtest.tcl could be given the
svn:executable property, so that if $(TCLSH) is empty they just get
executed directly.

Thanks,
Jay.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch.tclsh
Type: application/octet-stream
Size: 491 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20090529/3127d1c7/attachment.obj>

On 2009-05-29 15:55, Jay Foad wrote:
> On Linux, when I run "make TEST=ipodbgopt" in
> MultiSource/Applications/sqlite3 I get:
>
> make[4]: Entering directory
> `/home/foad/llvm/poo/build/test-suite/MultiSource/Applications/sqlite3'
> Compiling shell.c to Output/shell.bc
>
/home/foad/svn/llvm-project/test-suite/trunk/MultiSource/Applications/sqlite3/speedtest.tcl
> make[4]: execvp:
>
/home/foad/svn/llvm-project/test-suite/trunk/MultiSource/Applications/sqlite3/speedtest.tcl:
> Permission denied
> make[4]: ***
[/home/foad/svn/llvm-project/test-suite/trunk/MultiSource/Applications/sqlite3/test15.sql]
> Error 127
> make[4]: Leaving directory
> `/home/foad/llvm/poo/build/test-suite/MultiSource/Applications/sqlite3'
> make[3]: *** [sqlite3/.maketest] Error 2
> make[3]: Leaving directory
> `/home/foad/llvm/poo/build/test-suite/MultiSource/Applications'
> make[2]: *** [Applications/.maketest] Error 2
> make[2]: Leaving directory
`/home/foad/llvm/poo/build/test-suite/MultiSource'
> make[1]: *** [MultiSource/.maketest] Error 2
> make[1]: Leaving directory `/home/foad/llvm/poo/build/test-suite'
> make: *** [test] Error 2
>
> ... because the makefile uses $(TCLSH), which hasn't been set to
> anything. How is this supposed to work? Does the attached patch look
> reasonable? It seems to work for me.
>
> Alternatively, smalltest.tcl and speedtest.tcl could be given the
> svn:executable property, so that if $(TCLSH) is empty they just get
> executed directly.
>   
Doesn't the toplevel configure set TCLSH in Makefile.config?

Best regards,
--Edwin

2009/5/29 Török Edwin <edwintorok at gmail.com>:
> Doesn't the toplevel configure set TCLSH in Makefile.config?
Hmm, yes. Something must have gone wrong with my configure.

Thanks,
Jay.