Hello,
I found out that actually the optional macro for ffmpeg indexer was not
checked for at all.
In addition to this, when enabling ffmpeg indexer, a wrong include led to
build failure.

I fixed it with the trivial patch that follows.
I hope this can go in as it brings NFO::Video ontology to be used for such
indexed videos.

Regards

-- 
Alessandro Diaferia
KDE Developer
KDE e.V. member
Index: libstreamanalyzer/plugins/endplugins/ffmpegendanalyzer.cpp
===================================================================
--- libstreamanalyzer/plugins/endplugins/ffmpegendanalyzer.cpp	(revisione 1137074)
+++ libstreamanalyzer/plugins/endplugins/ffmpegendanalyzer.cpp	(copia locale)
@@ -24,7 +24,7 @@
 #include <strigi/analysisresult.h>
 #include <strigi/fieldtypes.h>
 #include <strigi/textutils.h>
-#include <strigi/rdfnamespaces.h>
+#include <rdfnamespaces.h>
 #include <strigi/strigi_thread.h>
 extern "C" {
 #include <libavcodec/avcodec.h>
Index: libstreamanalyzer/CMakeLists.txt
===================================================================
--- libstreamanalyzer/CMakeLists.txt	(revisione 1137074)
+++ libstreamanalyzer/CMakeLists.txt	(copia locale)
@@ -90,6 +90,7 @@ find_package(Threads REQUIRED)
 find_package(LibXml2 REQUIRED)
 find_optional_dep(CLucene ENABLE_CLUCENE CLucene_FOUND "CLucene backend")
 find_optional_dep(Exiv2 ENABLE_EXIV2 EXIV2_FOUND "indexing of EXIF/IPTC metadata")
+find_optional_dep(FFMPEG ENABLE_FFMPEG FFMPEG_FOUND "indexing FFMPEG")
 find_package(XAttr)
 include(ConfigureChecks.cmake)
 configure_file(LibStreamAnalyzer.cmake.in ${CMAKE_CURRENT_BINARY_DIR}/LibStreamAnalyzerConfig.cmake @ONLY )
@@ -103,6 +104,7 @@ configure_file(${libstreamanalyzer_SOURC
 include_directories(${LIBSTREAMS_INCLUDE_DIRS}
 	${libstreamanalyzer_SOURCE_DIR}/include
 	${libstreamanalyzer_BINARY_DIR}/include
+        ${libstreamanalyzer_SOURCE_DIR}/lib
 	${libstreamanalyzer_BINARY_DIR}/lib
 	${ICONV_INCLUDE_DIR}
 	${LIBSTREAMS_LIBRARY_DIRS}
_______________________________________________
Nepomuk mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/nepomuk

Reply via email to