Refactored CMakeLists.txt for java module

CMakeLists.txt

@@ -1705,8 +1705,8 @@ endif()
 # documentation
 status("")
 status("  Documentation:")
-status("    Sphinx:"              HAVE_SPHINX                                      THEN "${SPHINX_BUILD} (ver ${SPHINX_VERSION})" ELSE NO)
-status("    PdfLaTeX compiler:"   PDFLATEX_COMPILER                 THEN "${PDFLATEX_COMPILER}" ELSE NO)
+status("    Sphinx:"              HAVE_SPHINX              THEN "${SPHINX_BUILD} (ver ${SPHINX_VERSION})" ELSE NO)
+status("    PdfLaTeX compiler:"   PDFLATEX_COMPILER        THEN "${PDFLATEX_COMPILER}" ELSE NO)
 if (BUILD_DOCS AND HAVE_SPHINX)
     status("    Build Documentation:" PDFLATEX_COMPILER THEN YES ELSE "YES (HTML only)")
 else()

modules/java/CMakeLists.txt

@@ -14,8 +14,25 @@ SET(HDR_PARSER "${CMAKE_CURRENT_SOURCE_DIR}/../python/src2/hdr_parser.py")
 SET(GEN_JAVADOC "${CMAKE_CURRENT_SOURCE_DIR}/gen_javadoc.py")
 SET(RST_PARSER "${CMAKE_CURRENT_SOURCE_DIR}/rst_parser.py")
 
+#SET_SOURCE_FILES_PROPERTIES("${GEN_JAVA}" PROPERTIES GENERATED TRUE)
+#SET_SOURCE_FILES_PROPERTIES("${HDR_PARSER}" PROPERTIES GENERATED TRUE)
+#SET_SOURCE_FILES_PROPERTIES("${GEN_JAVADOC}" PROPERTIES GENERATED TRUE)
+#SET_SOURCE_FILES_PROPERTIES("${RST_PARSER}" PROPERTIES GENERATED TRUE)
+
+#add_custom_command(
+#    OUTPUT "${GEN_JAVA}"
+#    COMMAND ${CMAKE_COMMAND} -E echo "!!!!!!"
+#)
+
 SET(additional_clean_files "")
 
+macro(cleanup)
+    file(GLOB files_to_clean "${CMAKE_CURRENT_BINARY_DIR}/*.java" "${CMAKE_CURRENT_BINARY_DIR}/*.cpp" "${CMAKE_CURRENT_BINARY_DIR}/*.h" "${CMAKE_CURRENT_BINARY_DIR}/*.txt" "${CMAKE_CURRENT_BINARY_DIR}/*.log")
+    if(files_to_clean)
+        file(REMOVE ${files_to_clean})
+    endif()
+endmacro()
+
 macro(capitalize name outputvar)
     string(SUBSTRING "${name}" 0 1 first_letter)
     string(SUBSTRING "${name}" 1 -1 tail_letters)
@@ -23,11 +40,10 @@ macro(capitalize name outputvar)
     SET(${outputvar} "${first_letter}${tail_letters}")
 endmacro()
 
-#cleanup
-execute_process(COMMAND ${CMAKE_COMMAND} -E remove *.java *.cpp *.h *.txt
-                WORKING_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}"
-                OUTPUT_QUIET ERROR_QUIET)
+# cleanup required to correct process situations when some class is removed from exports
+cleanup()
 
+# setup raw java and cpp files generation (without javadoc and at temporary location)
 foreach(module ${OPENCV_JAVA_MODULES})
     IF(EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/config/${module}.filelist")
         FILE(STRINGS "${CMAKE_CURRENT_SOURCE_DIR}/config/${module}.filelist" headers_to_parse)
@@ -43,14 +59,20 @@ foreach(module ${OPENCV_JAVA_MODULES})
         list(SORT module_cppheaders)
     ENDIF()
 
-    # first run =(
+    # first run
     execute_process(COMMAND ${PYTHON_EXECUTABLE} "${GEN_JAVA}" "${HDR_PARSER}" ${module} ${module_cheaders} ${module_cppheaders}
                   WORKING_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}"
                   OUTPUT_QUIET ERROR_QUIET)
 
+    # add dependencies to cmake
+    foreach(header ${module_cheaders} ${module_cppheaders})
+        get_filename_component(header_name "${header}" NAME_WE)
+        configure_file("${header}" "${CMAKE_BINARY_DIR}/junk/${header_name}.junk" COPYONLY)
+    endforeach()
+
     FILE(GLOB ${module}_generated_java_sources "${CMAKE_CURRENT_BINARY_DIR}/${module}+*.java")
 
-    # second run =(
+    # second run
     add_custom_command(
         OUTPUT ${${module}_generated_java_sources}
         OUTPUT "${CMAKE_CURRENT_BINARY_DIR}/${module}.cpp"
@@ -58,10 +80,11 @@ foreach(module ${OPENCV_JAVA_MODULES})
         WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
         DEPENDS "${GEN_JAVA}"
         DEPENDS "${HDR_PARSER}"
-        DEPENDS ${module_headers}
         )
 endforeach()
 
+cleanup()
+
 set(target opencv_java)
 include_directories("${CMAKE_CURRENT_BINARY_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}/src/cpp")
 
@@ -70,6 +93,7 @@ SET (generated_java_sources "")
 SET (dependent_libs "")
 SET (dependent_extra_libs "")
 
+# summarize all sources for Java API
 foreach(module ${OPENCV_JAVA_MODULES})
     LIST(APPEND generated_cpp_sources "${CMAKE_CURRENT_BINARY_DIR}/${module}.cpp")
     LIST(APPEND generated_java_sources ${${module}_generated_java_sources})
@@ -77,6 +101,7 @@ foreach(module ${OPENCV_JAVA_MODULES})
     include_directories("${CMAKE_CURRENT_SOURCE_DIR}/../${module}/include")
 endforeach()
 
+# set libopencv_java.so shared library includes and dependencies
 foreach(module ${OPENCV_EXTRA_JAVA_MODULES})
     LIST(APPEND dependent_extra_libs opencv_${module})
     include_directories("${CMAKE_CURRENT_SOURCE_DIR}/../${module}/include")
@@ -84,12 +109,79 @@ endforeach()
 
 FILE(GLOB handwrittren_cpp_sources "${CMAKE_CURRENT_SOURCE_DIR}/src/cpp/*.cpp")
 FILE(GLOB handwrittren_h_sources "${CMAKE_CURRENT_SOURCE_DIR}/src/cpp/*.h*")
+FILE(GLOB handwrittren_java_sources "${CMAKE_CURRENT_SOURCE_DIR}/src/java/*.java")
+
+# created list of documented files targets
+SET(documented_java_files)
+SET(undocumented_java_files)
+foreach(java_file ${handwrittren_java_sources} ${generated_java_sources})
+    get_filename_component(java_file_name "${java_file}" NAME_WE)
+    if (NOT java_file_name MATCHES ".*-jdoc$")
+        list(APPEND documented_java_files "${CMAKE_CURRENT_BINARY_DIR}/${java_file_name}-jdoc.java")
+        list(APPEND undocumented_java_files "${java_file}")
+    endif()
+endforeach()
+
+# generate javadoc files
+add_custom_command(
+    OUTPUT ${documented_java_files}
+    COMMAND ${PYTHON_EXECUTABLE} "${GEN_JAVADOC}" "${CMAKE_CURRENT_SOURCE_DIR}/src/java" "${CMAKE_CURRENT_BINARY_DIR}" 2>"${CMAKE_CURRENT_BINARY_DIR}/get_javadoc_errors.log"
+    WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
+    DEPENDS ${handwrittren_java_sources}
+    DEPENDS ${undocumented_java_files}
+    DEPENDS "${GEN_JAVADOC}"
+    DEPENDS "${RST_PARSER}"
+)
+
+# copy generated java files to the final location
+SET(JAVA_OUTPUT_DIR "${CMAKE_BINARY_DIR}/src/org/opencv")
+
+set(java_files)
+foreach(java_file ${documented_java_files})
+    get_filename_component(java_file_name "${java_file}" NAME)
+    string(REPLACE "-jdoc.java" ".java" java_file_name "${java_file_name}")
+    string(REPLACE "+" "/" java_file_name "${java_file_name}")
 
-add_library(${target} SHARED ${handwrittren_cpp_sources} ${generated_cpp_sources})
+    add_custom_command(
+        OUTPUT "${JAVA_OUTPUT_DIR}/${java_file_name}"
+        COMMAND ${CMAKE_COMMAND} -E copy "${java_file}" "${JAVA_OUTPUT_DIR}/${java_file_name}"
+        DEPENDS "${java_file}"
+        DEPENDS "${JAVA_OUTPUT_DIR}"
+        COMMENT "Generating src/org/opencv/${java_file_name}"
+        )
+    list(APPEND java_files "${JAVA_OUTPUT_DIR}/${java_file_name}")
+    if(ANDROID)
+        get_filename_component(install_dir "${java_file_name}" PATH)
+        install(FILES "${JAVA_OUTPUT_DIR}/${java_file_name}" DESTINATION src/org/opencv/${install_dir} COMPONENT main)
+    endif()
+endforeach()
+
+# setup dependencies
+SET(dependency_file "${CMAKE_CURRENT_BINARY_DIR}/dependencies.cpp")
+add_custom_command(
+    OUTPUT "${dependency_file}"
+    COMMAND ${CMAKE_COMMAND} -E touch "${dependency_file}"
+)
+SET_SOURCE_FILES_PROPERTIES(${java_files} PROPERTIES HEADER_FILE_ONLY TRUE)
+SET(opencv_java_dependency_files "${java_files}")
+
+# add opencv_java project
+add_library(${target} SHARED ${handwrittren_h_sources} ${handwrittren_cpp_sources} ${generated_cpp_sources} ${dependency_file})
 target_link_libraries(${target} ${dependent_libs} ${dependent_extra_libs} ${OPENCV_LINKER_LIBS})
 add_dependencies(${target} "${GEN_JAVA}" "${HDR_PARSER}")
 add_dependencies(${target} ${dependent_extra_libs} ${dependent_libs})
 
+# Additional target properties
+set_target_properties(${target} PROPERTIES
+    OUTPUT_NAME "${target}"
+    ARCHIVE_OUTPUT_DIRECTORY ${LIBRARY_OUTPUT_PATH}
+    RUNTIME_OUTPUT_DIRECTORY ${EXECUTABLE_OUTPUT_PATH}
+    INSTALL_NAME_DIR ${OPENCV_LIB_INSTALL_PATH}
+    )
+
+install(TARGETS ${target} LIBRARY DESTINATION ${OPENCV_LIB_INSTALL_PATH} COMPONENT main)
+
+
 if(ANDROID)
     target_link_libraries(${target} jnigraphics)
 
@@ -100,25 +192,55 @@ if(ANDROID)
         POST_BUILD
         COMMAND ${CMAKE_STRIP} "${LIBRARY_OUTPUT_PATH}/lib${target}.so"
         )
+
+    # create Android library project in build folder
+    add_custom_command(
+        TARGET ${target} POST_BUILD
+        COMMAND ${CMAKE_COMMAND} -E make_directory "${CMAKE_BINARY_DIR}/res"
+        COMMAND ${CMAKE_COMMAND} -E make_directory "${CMAKE_BINARY_DIR}/gen"
+        )
+
+    # library project blank
+    file(GLOB_RECURSE android_lib_project_files "${CMAKE_CURRENT_SOURCE_DIR}/android/*")
+    foreach(file ${android_lib_project_files})
+        if(NOT file MATCHES "\\.svn")
+            file(RELATIVE_PATH file_rel "${CMAKE_CURRENT_SOURCE_DIR}/android/" "${file}")
+            add_custom_command(
+                OUTPUT "${CMAKE_BINARY_DIR}/${file_rel}"
+                COMMAND ${CMAKE_COMMAND} -E copy "${file}" "${CMAKE_BINARY_DIR}/${file_rel}"
+                DEPENDS "${file}"
+                COMMENT "Generating ${file_rel}"
+                )
+            SET_SOURCE_FILES_PROPERTIES("${CMAKE_BINARY_DIR}/${file_rel}" PROPERTIES HEADER_FILE_ONLY TRUE)
+            list(APPEND opencv_java_dependency_files "${CMAKE_BINARY_DIR}/${file_rel}")
+        endif()
+    endforeach()
+
+    # library project jni sources
+    foreach(jni_file ${handwrittren_cpp_sources} ${handwrittren_h_sources} ${generated_cpp_sources})
+        get_filename_component(jni_file_name "${jni_file}" NAME)
+        add_custom_command(
+            OUTPUT "${CMAKE_BINARY_DIR}/jni/${jni_file_name}"
+            COMMAND ${CMAKE_COMMAND} -E copy "${jni_file}" "${CMAKE_BINARY_DIR}/jni/${jni_file_name}"
+            DEPENDS "${jni_file}"
+            COMMENT "Generating jni/${jni_file_name}"
+        )
+        SET_SOURCE_FILES_PROPERTIES("${CMAKE_BINARY_DIR}/jni/${jni_file_name}" PROPERTIES HEADER_FILE_ONLY TRUE)
+        list(APPEND opencv_java_dependency_files "${CMAKE_BINARY_DIR}/jni/${jni_file_name}")
+    endforeach()
 endif()
 
-# Additional target properties
-set_target_properties(${target} PROPERTIES
-    OUTPUT_NAME "${target}"
-    ARCHIVE_OUTPUT_DIRECTORY ${LIBRARY_OUTPUT_PATH}
-    RUNTIME_OUTPUT_DIRECTORY ${EXECUTABLE_OUTPUT_PATH}
-    INSTALL_NAME_DIR ${OPENCV_LIB_INSTALL_PATH}
-    )
+# apply dependencies
+SET_SOURCE_FILES_PROPERTIES("${dependency_file}" PROPERTIES OBJECT_DEPENDS "${opencv_java_dependency_files}")
 
-install(TARGETS ${target} LIBRARY DESTINATION ${OPENCV_LIB_INSTALL_PATH} COMPONENT main)
 
-set(api_target opencv_java_api)
-add_custom_target(${api_target} ALL DEPENDS ${target})
+#set(api_target opencv_java_api)
+#add_custom_target(${api_target} ALL DEPENDS ${target})
 
-IF(ANDROID)
+IF(OFF)
     # create Android library project in build folder
     add_custom_command(
-        TARGET ${api_target}
+        TARGET ${target} POST_BUILD
         COMMAND ${CMAKE_COMMAND} -E make_directory "${CMAKE_BINARY_DIR}/res"
         COMMENT "Creating res directory"
         )
@@ -151,49 +273,49 @@ IF(ANDROID)
 ENDIF()
 
 # generate javadoc comments
-FILE(GLOB java_project_files "${CMAKE_CURRENT_SOURCE_DIR}/src/java/*.java")
-
-SET(documented_java_files)
-foreach(java_file ${java_project_files} ${generated_java_sources})
-    get_filename_component(java_file_name "${java_file}" NAME_WE)
-    if (NOT java_file_name MATCHES ".*-jdoc$")
-        list(APPEND documented_java_files "${CMAKE_CURRENT_BINARY_DIR}/${java_file_name}-jdoc.java")
-    endif()
-endforeach()
-
-add_custom_command(
-    TARGET ${api_target} PRE_BUILD
-    COMMAND ${PYTHON_EXECUTABLE} "${GEN_JAVADOC}" "${CMAKE_CURRENT_SOURCE_DIR}/src/java" "${CMAKE_CURRENT_BINARY_DIR}" 2>"${CMAKE_CURRENT_BINARY_DIR}/get_javadoc_errors.log"
-    WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
-    DEPENDS ${java_project_files}
-    DEPENDS ${generated_java_sources}
-    DEPENDS "${GEN_JAVA}"
-    DEPENDS "${HDR_PARSER}"
-    DEPENDS "${GEN_JAVADOC}"
-    DEPENDS "${RST_PARSER}"
-)
-
-
-SET(JAVA_OUTPUT_DIR "${CMAKE_BINARY_DIR}/src/org/opencv")
-
-foreach(java_file ${documented_java_files})
-    get_filename_component(java_file_name "${java_file}" NAME)
-    string(REPLACE "-jdoc.java" ".java" java_file_name "${java_file_name}")
-    string(REPLACE "+" "/" java_file_name "${java_file_name}")
-
-    add_custom_command(
-        TARGET ${api_target}
-        COMMAND ${CMAKE_COMMAND} -E copy "${java_file}" "${JAVA_OUTPUT_DIR}/${java_file_name}"
-        DEPENDS "${java_file}"
-        DEPENDS "${JAVA_OUTPUT_DIR}"
-        COMMENT "Generating src/org/opencv/${java_file_name}"
-        )
-    LIST(APPEND additional_clean_files "${JAVA_OUTPUT_DIR}/${java_file_name}")
-    if(ANDROID)
-        get_filename_component(install_dir "${java_file_name}" PATH)
-        install(FILES "${JAVA_OUTPUT_DIR}/${java_file_name}" DESTINATION src/org/opencv/${install_dir} COMPONENT main)
-    endif()
-endforeach()
+#FILE(GLOB java_project_files "${CMAKE_CURRENT_SOURCE_DIR}/src/java/*.java")
+
+#SET(documented_java_files)
+#foreach(java_file ${java_project_files} ${generated_java_sources})
+#    get_filename_component(java_file_name "${java_file}" NAME_WE)
+#    if (NOT java_file_name MATCHES ".*-jdoc$")
+#        list(APPEND documented_java_files "${CMAKE_CURRENT_BINARY_DIR}/${java_file_name}-jdoc.java")
+#    endif()
+#endforeach()
+
+#add_custom_command(
+#    TARGET ${api_target} PRE_BUILD
+#    COMMAND ${PYTHON_EXECUTABLE} "${GEN_JAVADOC}" "${CMAKE_CURRENT_SOURCE_DIR}/src/java" "${CMAKE_CURRENT_BINARY_DIR}" 2>"${CMAKE_CURRENT_BINARY_DIR}/get_javadoc_errors.log"
+#    WORKING_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
+#    DEPENDS ${java_project_files}
+#    DEPENDS ${generated_java_sources}
+#    DEPENDS "${GEN_JAVA}"
+#    DEPENDS "${HDR_PARSER}"
+#    DEPENDS "${GEN_JAVADOC}"
+#    DEPENDS "${RST_PARSER}"
+#)
+
+
+#SET(JAVA_OUTPUT_DIR "${CMAKE_BINARY_DIR}/src/org/opencv")
+
+#foreach(java_file ${documented_java_files})
+#    get_filename_component(java_file_name "${java_file}" NAME)
+#    string(REPLACE "-jdoc.java" ".java" java_file_name "${java_file_name}")
+#    string(REPLACE "+" "/" java_file_name "${java_file_name}")
+#
+#    add_custom_command(
+#        TARGET ${api_target}
+#        COMMAND ${CMAKE_COMMAND} -E copy "${java_file}" "${JAVA_OUTPUT_DIR}/${java_file_name}"
+#        DEPENDS "${java_file}"
+#        DEPENDS "${JAVA_OUTPUT_DIR}"
+#        COMMENT "Generating src/org/opencv/${java_file_name}"
+#        )
+#    LIST(APPEND additional_clean_files "${JAVA_OUTPUT_DIR}/${java_file_name}")
+#    if(ANDROID)
+#        get_filename_component(install_dir "${java_file_name}" PATH)
+#        install(FILES "${JAVA_OUTPUT_DIR}/${java_file_name}" DESTINATION src/org/opencv/${install_dir} COMPONENT main)
+#    endif()
+#endforeach()
 
 #android test project
 if(ANDROID AND BUILD_TESTS AND CAN_BUILD_ANDROID_PROJECTS)
@@ -208,7 +330,7 @@ if(ANDROID AND BUILD_TESTS AND CAN_BUILD_ANDROID_PROJECTS)
     SET(test_target ${test_name}_android)
 
     add_custom_target(${test_target} ALL)
-    add_dependencies(${test_target} opencv_java opencv_java_api)
+    add_dependencies(${test_target} opencv_java)
 
     file(RELATIVE_PATH OPENCV_REFERENCE_PATH "${test_dir}" "${CMAKE_BINARY_DIR}")
 

modules/java/gen_java.py

 import sys, re, os.path
+
 from string import Template
 
 try: