help required to compile keedu-4.4.2

Alexander Neundorf neundorf at kde.org
Tue Apr 27 20:54:45 CEST 2010


On Tuesday 27 April 2010, Andreas Pakulat wrote:
> On 27.04.10 10:52:28, David Faure wrote:
> > On Tuesday 27 April 2010, lux-integ wrote:
> > > cd $SOURCES/kdeedu-4.4.2/build/cantor/src/backends/R/rserver
> > > && --slave
> > > --file=$SOURCES/kdeedu-4.4.2/cantor/src/backends/R/rserver/tools/autolo
> > >ads.r
> > > $SOURCES/kdeedu-4.4.2/build/cantor/src/backends/R/rserver/rautoloads.h
> >
> > The name of the binary is missing before --slave.
> >
> > add_custom_target(renvvars COMMAND ${R_EXECUTABLE} --slave --
> > file=${CMAKE_CURRENT_SOURCE_DIR}/tools/envvars.r >
> > ${CMAKE_CURRENT_BINARY_DIR}/renvvars.h )
> >
> > So basically the problem is that R_EXECUTABLE is empty.
> >
> > ./modules/FindR.cmake:11:FIND_PROGRAM(R_EXECUTABLE R)
> > ./modules/FindR.cmake:13:IF(R_EXECUTABLE-NOTFOUND)
> > ./modules/FindR.cmake:15:ENDIF(R_EXECUTABLE-NOTFOUND)
> >
> > No abort on error when not found?
>
> Apart from that, the if-check should rather use if(NOT R_EXECUTABLE) as
> that doesn't rely on "cmake internals".

Actually I don't think thise are "cmake internals", but I think the code in 
the script is just wrong.

Alex


More information about the Kde-buildsystem mailing list