[LLVMdev] make TEST=ipodbgopt in sqlite3: problem with TCLSH

[Török Edwin]

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