Cargando…
Referencia en una nueva incidencia
Aún no existe contenido.
Eliminar la rama con el nombre 'feat/cmakeConv'
Eliminar una rama es permanente. NO PUEDE deshacerse. ¿Continuar?
Eliminar una rama es permanente. NO PUEDE deshacerse. ¿Continuar?
I've added a man page taken from your Debian building system.
The download location for kbarcode-2.0.0.pdf has been set to "gentoo.ussg.indiana.edu", this could be change for an other location (wiki) later.
Please add the build options and adjust the tests as mentioned in the note below.
#### check for headers
check_include_file( "barcode.h" _ENABLE_NATIVE_GNU_BARCODE )
check_include_file( "kjs/interpreter.h" USE_JAVASCRIPT )
We should define options such as
WITH_NATIVE_GNU_BARCODE
(OFF by default) andWITH_JAVASCRIPT
, and these tests should end bytde_message_fatal
if the conditions are not met.There are additional issues:
kjs/interpreter.h
fails – we need to setCMAKE_REQUIRED_INCLUDES
.kjs/interpreter.h
is intended for C++, thecheck_include_file
test fails because the build with C fails – we need to usecheck_include_file_cxx
.This means to add to CMakeLists.txt:
And in ConfigureChecks.cmake change test:
A little typo in the comment – closing > is one word later.
Sorry, but there are some new shortcomings – see comments.
option( BUILD_DOC "Build documentation" ${BUILD_ALL} )
option( BUILD_TRANSLATIONS "Build translations" ${BUILD_ALL} )
option( WITH_NATIVE_GNU_BARCODE "Enable GNU Barcode" OFF )
option( WITH_JAVASCRIPT "Enable Javascript" OFF )
These are
WITH_…
options – they belong to the section##### optional stuff
. ForWITH_JAVASCRIPT
please use default value${WITH_ALL_OPTIONS}
.list( APPEND CMAKE_REQUIRED_INCLUDES ${TQT_INCLUDE_DIRS} ${TDE_INCLUDE_DIR} )
check_include_file_cxx( "kjs/interpreter.h" HAVE_JAVASCRIPT )
if( HAVE_JAVASCRIPT )
set( USE_JAVASCRIPT 1 )
Here it would be good to add also
set( NO_KJS_EMBED 1 )
.endif( HAVE_JAVASCRIPT )
else()
set( USE_JAVASCRIPT false )
tde_restore( CMAKE_REQUIRED_INCLUDES )
This
tde_restore
is misplaced – it belongs beforeelse( )
#cmakedefine _ENABLE_NATIVE_GNU_BARCODE @_ENABLE_NATIVE_GNU_BARCODE@
/* Define to 1 if you have <kjs/interpreter.h> header file. */
#cmakedefine USE_JAVASCRIPT @USE_JAVASCRIPT@
Here,
#cmakedefine NO_KJS_EMBED
should also be set.Finished little things:
Everything seems to be ready for merge.
Everything seems to be ready for merge.
20a84daca2
.