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.
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.
We should define options such as WITH_NATIVE_GNU_BARCODE (OFF by default) and WITH_JAVASCRIPT, and these tests should end by tde_message_fatal if the conditions are not met.
We should define options such as `WITH_NATIVE_GNU_BARCODE` (OFF by default) and `WITH_JAVASCRIPT`, and these tests should end by `tde_message_fatal` if the conditions are not met.
Since TDE includes neither TQt include does not need to be on the usual paths, test for kjs/interpreter.h fails – we need to set CMAKE_REQUIRED_INCLUDES.
Since kjs/interpreter.h is intended for C++, the check_include_file test fails because the build with C fails – we need to use check_include_file_cxx.
There are additional issues:
1. Since TDE includes neither TQt include does not need to be on the usual paths, test for `kjs/interpreter.h` fails – we need to set `CMAKE_REQUIRED_INCLUDES`.
2. Since `kjs/interpreter.h` is intended for C++, the `check_include_file` test fails because the build with C fails – we need to use `check_include_file_cxx`.
This means to add to CMakeLists.txt:
```
include( CheckIncludeFileCXX )
```
And in ConfigureChecks.cmake change test:
```
if( WITH_JAVASCRIPT )
tde_save( CMAKE_REQUIRED_INCLUDES )
list( APPEND CMAKE_REQUIRED_INCLUDES ${TQT_INCLUDE_DIRS} ${TDE_INCLUDE_DIR} )
check_include_file_cxx( "kjs/interpreter.h" USE_JAVASCRIPT )
tde_restore( CMAKE_REQUIRED_INCLUDES )
endif( )
```
Finished little things:
1. Rebased to current HEAD.
2. Fixed shortcomings in patch for cmake options.
3. Added patch for automake options.
4. Added patch for ascii cast.
Everything seems to be ready for merge.
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
.