ubuntu-touch-coreapps-reviewers team mailing list archive

[Stefano Verzegnassi <stefano92.100@xxxxxxxxx>]

Stefano Verzegnassi has proposed merging lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-cmake-refactor into lp:ubuntu-docviewer-app/reboot with lp:~verzegnassi-stefano/ubuntu-docviewer-app/get-click-deps as a prerequisite.

Commit message:
* Changes to the project structure
* Provide DocViewer plugins as separate .deb packages
* Added exception for '*.user.*' in .bzrignore file

Requested reviews:
  Ubuntu Document Viewer Developers (ubuntu-docviewer-dev)

For more details, see:
https://code.launchpad.net/~verzegnassi-stefano/ubuntu-docviewer-app/reboot-cmake-refactor/+merge/275903

* Changes to the project structure
* Provide DocViewer plugins as separate .deb packages
* Added exception for '*.user.*' in .bzrignore file
-- 
Your team Ubuntu Document Viewer Developers is requested to review the proposed merge of lp:~verzegnassi-stefano/ubuntu-docviewer-app/reboot-cmake-refactor into lp:ubuntu-docviewer-app/reboot.

=== modified file '.bzrignore'
--- .bzrignore	2014-10-28 22:10:16 +0000
+++ .bzrignore	2015-10-27 18:49:01 +0000
@@ -1,11 +1,11 @@
 Makefile
 ubuntu-docviewer-app
 *.user
+*.user.*
 moc_file.cpp
 launcher/build-docviewer-launcher-Desktop-Debug/
 launcher/build-docviewer-launcher-Desktop-Release/
 launcher/src/docviewer-launcher.pro.user
-*.cmake
 CMakeFiles
 *_automoc.cpp
 moc_*

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2015-10-27 18:49:01 +0000
+++ CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -1,34 +1,23 @@
 project(com.ubuntu.docviewer C CXX)
 cmake_minimum_required(VERSION 2.8.9)
 
-find_program(INTLTOOL_MERGE intltool-merge)
-if(NOT INTLTOOL_MERGE)
-  message(FATAL_ERROR "Could not find intltool-merge, please install the intltool package")
-endif(NOT INTLTOOL_MERGE)
-
-find_program(INTLTOOL_EXTRACT intltool-extract)
-if(NOT INTLTOOL_EXTRACT)
-  message(FATAL_ERROR "Could not find intltool-extract, please install the intltool package")
-endif(NOT INTLTOOL_EXTRACT)
-
+set(UBUNTU_MANIFEST_PATH "click/manifest.json.in" CACHE INTERNAL "Relative path to the manifest file")
+set(CMAKE_MODULE_PATH "${CMAKE_CURRENT_SOURCE_DIR}/cmake/modules")
 set(CMAKE_AUTOMOC ON)
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -fno-permissive -pedantic -Wall -Wextra -fPIC")
 
 include(FindPkgConfig)
-# Standard install paths
 include(GNUInstallDirs)
 
-option(INSTALL_TESTS "Install the tests on make install" on)
-option(CLICK_MODE "Installs to a contained location" on)
+option(INSTALL_TESTS    "Install the tests on make install" on)
+option(CLICK_MODE       "Installs to a contained location" on)
 
-set(APP_NAME ubuntu-docviewer-app)
-set(DESKTOP_FILE "${PROJECT_NAME}.desktop")
-set(URLS_FILE "${PROJECT_NAME}.url-dispatcher")
-set(LP_PROJECT ubuntu-docviewer-app)
-set(ICON_FILE docviewer@xxxxxx)
-set(AUTOPILOT_DIR ubuntu_docviewer_app)
-set(EXEC "${APP_NAME}")
-set(UBUNTU_MANIFEST_PATH "manifest.json.in" CACHE INTERNAL "Relative path to the manifest file")
+set(APP_NAME            ubuntu-docviewer-app)
+set(DESKTOP_FILE        "${PROJECT_NAME}.desktop")
+set(URLS_FILE           "${PROJECT_NAME}.url-dispatcher")
+set(LP_PROJECT          ubuntu-docviewer-app)
+set(ICON_FILE           docviewer@xxxxxx)
+set(AUTOPILOT_DIR       ubuntu_docviewer_app)
 
 # Sets BZR_REVNO
 execute_process(
@@ -37,112 +26,26 @@
     OUTPUT_STRIP_TRAILING_WHITESPACE
 )
 
-if(CLICK_MODE)
-    if(NOT DEFINED BZR_SOURCE)
-        set(BZR_SOURCE "lp:${LP_PROJECT}")
-        message("-- Setting BZR_SOURCE to ${BZR_SOURCE}")
-    endif(NOT DEFINED BZR_SOURCE)
-
-    if(NOT BZR_REVNO)
-        set(BZR_REVNO "latest")
-    endif(NOT BZR_REVNO)
-
-    # Find out the architecture for package building
-    # to determine the plugin's installation path
-    execute_process(
-        COMMAND dpkg-architecture -qDEB_HOST_MULTIARCH
-            OUTPUT_VARIABLE ARCH_TRIPLET
-            OUTPUT_STRIP_TRAILING_WHITESPACE
-    )
-    execute_process(
-      COMMAND dpkg-architecture -qDEB_HOST_ARCH
-      OUTPUT_VARIABLE CLICK_ARCH
-      OUTPUT_STRIP_TRAILING_WHITESPACE
-    )
-    set(CMAKE_INSTALL_PREFIX /)
-    set(QT_IMPORTS_DIR "/lib/${ARCH_TRIPLET}")
-    set(BIN_DIR /lib/${ARCH_TRIPLET}/bin)
-    set(DATA_DIR /)
-    set(ICON ${ICON_FILE})
-    set(DESKTOP_DIR ${DATA_DIR})
-    set(URLS_DIR ${DATA_DIR})
-    configure_file(manifest.json.in ${CMAKE_CURRENT_BINARY_DIR}/manifest.json)
-    install(FILES ${CMAKE_CURRENT_BINARY_DIR}/manifest.json docviewer.apparmor docviewer-content.json ${URLS_FILE}
-            DESTINATION ${CMAKE_INSTALL_PREFIX})
-    # Make the click files visible in Qt Creator
-    file(GLOB CLICK_FILES
-         RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}
-         *.json *.json.in *.apparmor)
-
-    add_custom_target(com_ubuntu_docviewer_CLICKFiles ALL SOURCES ${CLICK_FILES})
-else(CLICK_MODE)
-    # components PATH
-    execute_process(
-        COMMAND qmake -query QT_INSTALL_QML
-            OUTPUT_VARIABLE QT_IMPORTS_DIR
-            OUTPUT_STRIP_TRAILING_WHITESPACE
-    )
-    set(DATA_DIR ${CMAKE_INSTALL_DATADIR}/${APP_NAME})
-    set(ICON "${CMAKE_INSTALL_PREFIX}/${DATA_DIR}/${ICON_FILE}")
-    set(DESKTOP_DIR ${CMAKE_INSTALL_DATADIR}/applications)
-    set(APPLICATION_DIR ${CMAKE_INSTALL_DATADIR}/accounts/applications)
-endif(CLICK_MODE)
-
-file(GLOB_RECURSE I18N_SRC_FILES
-     RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}/po
-     *.qml *.js *.cpp)
-list(APPEND I18N_SRC_FILES ${CMAKE_CURRENT_BINARY_DIR}/po/${DESKTOP_FILE}.in.in.h)
-list(SORT I18N_SRC_FILES)
-message("Found ${I18N_SRC_FILES}")
-
-file(GLOB SRC_FILES
-     RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}
-     *.qml *.js)
-add_custom_target(ubuntu-docviewer-app_QMlFiles ALL SOURCES ${SRC_FILES})
-
-file(GLOB RESOURCE_FILES
-     RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}
-     *.png *.svg)
-install(FILES ${SRC_FILES} ${RESOURCE_FILES} ${ICON_FILE} DESTINATION ${DATA_DIR})
-
-configure_file(${DESKTOP_FILE}.in.in ${DESKTOP_FILE}.in)
-
-add_custom_target(${DESKTOP_FILE} ALL
-  COMMENT "Merging translations into ${DESKTOP_FILE}"
-  COMMAND LC_ALL=C ${INTLTOOL_MERGE} -d -u ${CMAKE_SOURCE_DIR}/po ${DESKTOP_FILE}.in ${DESKTOP_FILE}
-)
-
-install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${DESKTOP_FILE} 
-        DESTINATION ${DESKTOP_DIR})
- 
+# Find out the architecture for package building
+# to determine the plugin's installation path
+execute_process(
+  COMMAND dpkg-architecture -qDEB_HOST_MULTIARCH
+  OUTPUT_VARIABLE ARCH_TRIPLET
+  OUTPUT_STRIP_TRAILING_WHITESPACE
+)
+execute_process(
+  COMMAND dpkg-architecture -qDEB_HOST_ARCH
+  OUTPUT_VARIABLE CLICK_ARCH
+  OUTPUT_STRIP_TRAILING_WHITESPACE
+)
+
+include(Click)
+
 # Tests
 enable_testing()
 
+add_subdirectory(po)
+add_subdirectory(click)
+add_subdirectory(data)
 add_subdirectory(src)
 add_subdirectory(tests)
-add_subdirectory(po)
-
-# If running in CLICK_MODE, include binary dependencies of docviewer
-if(CLICK_MODE)
-    MESSAGE("Grabbing upstream libs to ${CMAKE_CURRENT_BINARY_DIR}/upstream-libs")
-    execute_process(
-        COMMAND mkdir ${CMAKE_CURRENT_BINARY_DIR}/upstream-libs
-        COMMAND ${CMAKE_CURRENT_SOURCE_DIR}/get-click-deps -d ${CMAKE_CURRENT_SOURCE_DIR}/docviewer-libs.json ${CLICK_ARCH} ${CMAKE_CURRENT_BINARY_DIR}/upstream-libs
-    )
-
-    MESSAGE("Installing upstream libs from ${CMAKE_CURRENT_BINARY_DIR}/upstream-libs/usr/lib/${ARCH_TRIPLET}/ to ${DATA_DIR}lib/${ARCH_TRIPLET}")
-    file(GLOB_RECURSE UPSTREAM_LIBS "${CMAKE_CURRENT_BINARY_DIR}/upstream-libs/usr/lib/${ARCH_TRIPLET}/*")
-    foreach(ITEM ${UPSTREAM_LIBS})
-        IF( IS_DIRECTORY "${ITEM}" )
-            LIST( APPEND DIRS_TO_DEPLOY "${ITEM}" )
-        ELSE()
-            LIST( APPEND FILES_TO_DEPLOY "${ITEM}" )
-        ENDIF()
-    endforeach()
-    MESSAGE("Following files to install:- ${FILES_TO_DEPLOY}")
-    INSTALL( FILES ${FILES_TO_DEPLOY} DESTINATION ${DATA_DIR}lib/${ARCH_TRIPLET} )
-
-
-    MESSAGE("Installing LibreOffice from ${CMAKE_CURRENT_BINARY_DIR}/upstream-libs/opt/libreoffice/lib/libreoffice to ${DATA_DIR}lib/${ARCH_TRIPLET}/libreoffice")
-    INSTALL( DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/upstream-libs/opt/libreoffice/lib/libreoffice/ DESTINATION ${DATA_DIR}lib/${ARCH_TRIPLET}/libreoffice )
-endif(CLICK_MODE)

=== added directory 'click'
=== added file 'click/CMakeLists.txt'
--- click/CMakeLists.txt	1970-01-01 00:00:00 +0000
+++ click/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -0,0 +1,18 @@
+if(CLICK_MODE)
+  if(NOT BZR_REVNO)
+    set(BZR_REVNO "latest")
+  endif(NOT BZR_REVNO)
+
+  configure_file(manifest.json.in ${CMAKE_CURRENT_BINARY_DIR}/manifest.json)
+  install(FILES ${CMAKE_CURRENT_BINARY_DIR}/manifest.json DESTINATION ${CMAKE_INSTALL_PREFIX})
+  install(FILES docviewer.apparmor docviewer-content.json DESTINATION ${CMAKE_INSTALL_PREFIX})
+endif(CLICK_MODE)
+
+
+# Make the click files visible in Qt Creator
+file(GLOB CLICK_FILES
+  RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}
+  *.json *.json.in *.apparmor
+)
+
+add_custom_target(com_ubuntu_docviewer_CLICKFiles ALL SOURCES ${CLICK_FILES})

=== renamed file 'docviewer-content.json' => 'click/docviewer-content.json'
=== renamed file 'docviewer-libs.json' => 'click/docviewer-libs.json'
=== renamed file 'docviewer.apparmor' => 'click/docviewer.apparmor'
=== renamed file 'manifest.json.in' => 'click/manifest.json.in'
=== added directory 'cmake'
=== added directory 'cmake/modules'
=== added file 'cmake/modules/Click.cmake'
--- cmake/modules/Click.cmake	1970-01-01 00:00:00 +0000
+++ cmake/modules/Click.cmake	2015-10-27 18:49:01 +0000
@@ -0,0 +1,47 @@
+if(CLICK_MODE)
+  if(NOT DEFINED BZR_SOURCE)
+    set(BZR_SOURCE "lp:${LP_PROJECT}")
+    message("-- Setting BZR_SOURCE to ${BZR_SOURCE}")
+  endif(NOT DEFINED BZR_SOURCE)
+
+  set(QT_IMPORTS_DIR "/lib/${ARCH_TRIPLET}")
+  set(CMAKE_INSTALL_PREFIX /)
+  set(DATA_DIR /)
+
+  # Path for ubuntu-docviewer-app executable
+  set(BIN_DIR ${DATA_DIR}lib/${ARCH_TRIPLET}/bin)
+
+  # If running in CLICK_MODE, include binary dependencies of docviewer
+  set(GET_CLICK_DEPS_TOOL ${CMAKE_SOURCE_DIR}/tools/get-click-deps)
+  set(DEPS_MANIFEST ${CMAKE_CURRENT_SOURCE_DIR}/docviewer-libs.json)
+  set(UPSTREAM_LIBS_DIR ${CMAKE_BINARY_DIR}/upstream-libs)
+
+  MESSAGE("Grabbing upstream libs to ${UPSTREAM_LIBS_DIR}")
+  execute_process(
+    COMMAND mkdir ${UPSTREAM_LIBS_DIR}
+    COMMAND ${GET_CLICK_DEPS_TOOL} -d ${DEPS_MANIFEST} ${CLICK_ARCH} ${UPSTREAM_LIBS_DIR}
+  )
+
+  MESSAGE("Installing upstream libs from ${UPSTREAM_LIBS_DIR}/usr/lib/${ARCH_TRIPLET}/ to ${DATA_DIR}lib/${ARCH_TRIPLET}")
+  file(GLOB_RECURSE UPSTREAM_LIBS "${UPSTREAM_LIBS_DIR}/usr/lib/${ARCH_TRIPLET}/*")
+  foreach(ITEM ${UPSTREAM_LIBS})
+    IF( IS_DIRECTORY "${ITEM}" )
+      LIST( APPEND DIRS_TO_DEPLOY "${ITEM}" )
+    ELSE()
+      LIST( APPEND FILES_TO_DEPLOY "${ITEM}" )
+    ENDIF()
+  endforeach()
+  MESSAGE("Following files to install:- ${FILES_TO_DEPLOY}")
+  INSTALL( FILES ${FILES_TO_DEPLOY} DESTINATION ${DATA_DIR}lib/${ARCH_TRIPLET} )
+
+  MESSAGE("Installing LibreOffice from ${UPSTREAM_LIBS_DIR}/opt/libreoffice/lib/libreoffice to ${DATA_DIR}lib/${ARCH_TRIPLET}/libreoffice")
+  INSTALL( DIRECTORY ${UPSTREAM_LIBS_DIR}/opt/libreoffice/lib/libreoffice/ DESTINATION ${DATA_DIR}lib/${ARCH_TRIPLET}/libreoffice )
+else(CLICK_MODE)
+  execute_process(
+    COMMAND qmake -query QT_INSTALL_QML
+    OUTPUT_VARIABLE QT_IMPORTS_DIR
+    OUTPUT_STRIP_TRAILING_WHITESPACE
+  )
+
+  set(DATA_DIR ${CMAKE_INSTALL_DATADIR}/${APP_NAME})
+endif(CLICK_MODE)

=== added directory 'data'
=== added file 'data/CMakeLists.txt'
--- data/CMakeLists.txt	1970-01-01 00:00:00 +0000
+++ data/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -0,0 +1,29 @@
+if(CLICK_MODE)
+  set(ICON ${ICON_FILE})
+  set(DESKTOP_DIR ${DATA_DIR})
+
+  install(FILES ${ICON_FILE} DESTINATION ${DATA_DIR})
+else(CLICK_MODE)
+  set(ICON "${CMAKE_INSTALL_PREFIX}/${DATA_DIR}/${ICON_FILE}")
+  set(DESKTOP_DIR ${CMAKE_INSTALL_DATADIR}/applications)
+
+  install(DIRECTORY icons/hicolor DESTINATION ${CMAKE_INSTALL_DATADIR}/icons
+    FILES_MATCHING PATTERN *.png
+  )
+endif(CLICK_MODE)
+
+set(EXEC "${APP_NAME}")
+
+configure_file(${DESKTOP_FILE}.in.in ${DESKTOP_FILE}.in @ONLY)
+
+add_custom_target(${DESKTOP_FILE} ALL
+  COMMENT "Merging translations into ${DESKTOP_FILE}"
+  COMMAND LC_ALL=C ${INTLTOOL_MERGE} -d -u ${CMAKE_SOURCE_DIR}/po
+    ${CMAKE_CURRENT_BINARY_DIR}/${DESKTOP_FILE}.in ${DESKTOP_FILE}
+)
+
+install(FILES ${CMAKE_CURRENT_BINARY_DIR}/${DESKTOP_FILE}
+  DESTINATION ${DESKTOP_DIR}
+)
+
+install(FILES ${URLS_FILE} DESTINATION ${CMAKE_INSTALL_PREFIX})

=== renamed file 'com.ubuntu.docviewer.desktop.in.in' => 'data/com.ubuntu.docviewer.desktop.in.in'
=== renamed file 'com.ubuntu.docviewer.url-dispatcher' => 'data/com.ubuntu.docviewer.url-dispatcher'
=== renamed file 'docviewer@xxxxxx' => 'data/docviewer@xxxxxx'
=== modified file 'debian/control'
--- debian/control	2015-09-19 14:54:11 +0000
+++ debian/control	2015-10-27 18:49:01 +0000
@@ -21,20 +21,52 @@
 Standards-Version: 3.9.6
 Section: misc
 Homepage: https://launchpad.net/ubuntu-docviewer-app
-Vcs-Bzr: https://code.launchpad.net/~ubuntu-docviewer-dev/ubuntu-docviewer-app/trunk
+Vcs-Bzr: https://code.launchpad.net/~ubuntu-docviewer-dev/ubuntu-docviewer-app/reboot
 
 Package: ubuntu-docviewer-app
 Architecture: any
-Depends: qtdeclarative5-qtquick2-plugin,
+Depends: ${misc:Depends},
+         qtdeclarative5-documentviewer-common (= ${source:Version}),
+         qtdeclarative5-documentviewer-libreoffice (= ${source:Version}),
+         qtdeclarative5-documentviewer-pdf (= ${source:Version}),  
+         suru-icon-theme (>= 14.04+15.10.20150707-0ubuntu1),
+         qtdeclarative5-qtquick2-plugin,
          qtdeclarative5-ubuntu-ui-toolkit-plugin,
          qtdeclarative5-ubuntu-content1,
          qtdeclarative5-quicklayouts-plugin,
-         qtdeclarative5-window-plugin,
-         suru-icon-theme (>= 14.04+15.10.20150707-0ubuntu1),
-         ${misc:Depends}
+         qtdeclarative5-window-plugin      
 Description: Document Viewer application
  Core Document Viewer application
 
+Package: qtdeclarative5-documentviewer-common
+Architecture: any
+Depends: ${misc:Depends},
+         ${shlibs:Depends},
+         ${misc:Pre-Depends},
+Description: Document Viewer QML plugin
+ This package contains a plugin with some helper functions for the Ubuntu
+ Document Viewer.
+
+Package: qtdeclarative5-documentviewer-libreoffice
+Architecture: any
+Depends: ${misc:Depends},
+         ${shlibs:Depends},
+         ${misc:Pre-Depends},
+         libreoffice-vanilla (>= 5.0.0) | libreoffice (>= 5.0.0)
+Description: LibreOffice QML plugin
+ This package contains a plugin that enables QML apps to read
+ and display any LibreOffice document.
+
+Package: qtdeclarative5-documentviewer-pdf
+Architecture: any
+Depends: ${misc:Depends},
+         ${shlibs:Depends},
+         ${misc:Pre-Depends},
+         libpoppler-qt5-1
+Description: Poppler PDF QML plugin
+ This package contains a plugin that enables QML apps to read
+ and display any PDF document, via the Poppler Qt5 bindings.
+
 Package: ubuntu-docviewer-app-autopilot
 Architecture: all
 Depends: libautopilot-qt,

=== added file 'debian/qtdeclarative5-documentviewer-common.install'
--- debian/qtdeclarative5-documentviewer-common.install	1970-01-01 00:00:00 +0000
+++ debian/qtdeclarative5-documentviewer-common.install	2015-10-27 18:49:01 +0000
@@ -0,0 +1,2 @@
+usr/lib/*/qt5/qml/DocumentViewer/qmldir
+usr/lib/*/qt5/qml/DocumentViewer/libfileqmlplugin.so

=== added file 'debian/qtdeclarative5-documentviewer-libreoffice.install'
--- debian/qtdeclarative5-documentviewer-libreoffice.install	1970-01-01 00:00:00 +0000
+++ debian/qtdeclarative5-documentviewer-libreoffice.install	2015-10-27 18:49:01 +0000
@@ -0,0 +1,1 @@
+usr/lib/*/qt5/qml/DocumentViewer/LibreOffice

=== added file 'debian/qtdeclarative5-documentviewer-pdf.install'
--- debian/qtdeclarative5-documentviewer-pdf.install	1970-01-01 00:00:00 +0000
+++ debian/qtdeclarative5-documentviewer-pdf.install	2015-10-27 18:49:01 +0000
@@ -0,0 +1,1 @@
+usr/lib/*/qt5/qml/DocumentViewer/PDF

=== modified file 'debian/ubuntu-docviewer-app.install'
--- debian/ubuntu-docviewer-app.install	2015-01-29 19:23:09 +0000
+++ debian/ubuntu-docviewer-app.install	2015-10-27 18:49:01 +0000
@@ -1,5 +1,4 @@
 usr/bin/ubuntu-docviewer-app
-usr/lib/*/qt5
 usr/share/applications
 usr/share/locale
 usr/share/ubuntu-docviewer-app

=== modified file 'po/CMakeLists.txt'
--- po/CMakeLists.txt	2015-04-29 16:09:12 +0000
+++ po/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -17,12 +17,22 @@
 set(POT_FILE ${DOMAIN}.pot)
 file(GLOB PO_FILES *.po)
 
+file(GLOB_RECURSE I18N_SRC_FILES
+  RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}
+  ${CMAKE_SOURCE_DIR}/*.qml
+  ${CMAKE_SOURCE_DIR}/*.js
+  ${CMAKE_SOURCE_DIR}/*.cpp)
+list(APPEND I18N_SRC_FILES ${CMAKE_CURRENT_BINARY_DIR}/${DESKTOP_FILE}.in.in.h)
+list(SORT I18N_SRC_FILES)
+
+message("Found ${I18N_SRC_FILES}")
+
 # Creates the .pot file containing the translations template
 add_custom_target(${POT_FILE} ALL
   COMMENT "Generating translation template"
   # Extract the translatable messages from the desktop file 
   COMMAND ${INTLTOOL_EXTRACT} --update --type=gettext/ini
-          --srcdir=${CMAKE_SOURCE_DIR} ${DESKTOP_FILE}.in.in
+          --srcdir=${CMAKE_SOURCE_DIR}/data ${DESKTOP_FILE}.in.in
   # Update the translation file
   COMMAND ${GETTEXT_XGETTEXT_EXECUTABLE} -o ${POT_FILE}
           --from-code=UTF-8
@@ -47,7 +57,6 @@
   RENAME ${DOMAIN}.mo)
 endforeach(PO_FILE)
 
-
 # make the translation files visible on qtcreator
 file(GLOB TRANSLATION_FILES
      RELATIVE ${CMAKE_CURRENT_SOURCE_DIR}

=== modified file 'src/CMakeLists.txt'
--- src/CMakeLists.txt	2015-01-29 16:35:28 +0000
+++ src/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -1,6 +1,2 @@
-find_package(Qt5Core)
-find_package(Qt5Qml)
-find_package(Qt5Quick)
-
 add_subdirectory(app)
 add_subdirectory(plugin)

=== modified file 'src/app/CMakeLists.txt'
--- src/app/CMakeLists.txt	2015-10-20 11:58:48 +0000
+++ src/app/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -1,6 +1,11 @@
 file(GLOB_RECURSE QML_SRCS *.qml *.js)
 file(GLOB_RECURSE IMAGE_FILES *.qml *.js)
 
+find_package(Qt5Core)
+find_package(Qt5Gui)
+find_package(Qt5Qml)
+find_package(Qt5Quick)
+
 add_definitions(
   -DGETTEXT_PACKAGE=\"${PROJECT_NAME}\"
   -DGETTEXT_LOCALEDIR=\"${CMAKE_INSTALL_LOCALEDIR}\"
@@ -23,7 +28,7 @@
 
 add_executable(ubuntu-docviewer-app ${docviewer_SRCS})
 
-qt5_use_modules(ubuntu-docviewer-app Widgets Gui Qml Quick DBus Concurrent)
+qt5_use_modules(ubuntu-docviewer-app Gui Qml Quick)
 
 if(NOT "${CMAKE_CURRENT_SOURCE_DIR}" STREQUAL "${CMAKE_CURRENT_BINARY_DIR}")
 add_custom_target(docviewer-qmlfiles ALL

=== modified file 'src/plugin/file-qml-plugin/CMakeLists.txt'
--- src/plugin/file-qml-plugin/CMakeLists.txt	2015-09-19 15:40:22 +0000
+++ src/plugin/file-qml-plugin/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -1,6 +1,10 @@
 set(PLUGIN_DIR DocumentViewer)
 include_directories(${CMAKE_CURRENT_SOURCE_DIR})
 
+find_package(Qt5Core)
+find_package(Qt5Qml)
+find_package(Qt5Quick)
+
 #add the sources to compile
 set(fileqmlplugin_SRCS
     backend.cpp

=== modified file 'src/plugin/libreofficetoolkit-qml-plugin/CMakeLists.txt'
--- src/plugin/libreofficetoolkit-qml-plugin/CMakeLists.txt	2015-10-18 20:58:32 +0000
+++ src/plugin/libreofficetoolkit-qml-plugin/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -1,5 +1,10 @@
 set(PLUGIN_DIR DocumentViewer/LibreOffice)
 
+find_package(Qt5Core)
+find_package(Qt5Gui)
+find_package(Qt5Qml)
+find_package(Qt5Quick)
+
 file(GLOB_RECURSE QML_SRCS
     qml/*.qml
     qml/*.js

=== modified file 'src/plugin/poppler-qml-plugin/CMakeLists.txt'
--- src/plugin/poppler-qml-plugin/CMakeLists.txt	2015-07-14 15:43:11 +0000
+++ src/plugin/poppler-qml-plugin/CMakeLists.txt	2015-10-27 18:49:01 +0000
@@ -1,4 +1,12 @@
 set(PLUGIN_DIR DocumentViewer/PDF)
+
+find_package(Qt5Core)
+find_package(Qt5Gui)
+find_package(Qt5Qml)
+find_package(Qt5Quick)
+find_package(Qt5Xml)
+find_package(Qt5Concurrent)
+
 include_directories(
 	${CMAKE_CURRENT_SOURCE_DIR}
 	${CMAKE_CURRENT_BINARY_DIR}

=== added directory 'tools'
=== renamed file 'get-click-deps' => 'tools/get-click-deps'
=== renamed file 'run-pep8' => 'tools/run-pep8'
--- run-pep8	2015-04-27 16:29:17 +0000
+++ tools/run-pep8	2015-10-27 18:49:01 +0000
@@ -2,7 +2,7 @@
 set -e
 
 echo "= pep8 ="
-for i in `find . -iname '*.py'` ; do 
+for i in `find .. -iname '*.py'` ; do 
     echo "Checking $i"
     pep8 $i
 done