Merge lp:~michihenning/thumbnailer/fix-qt-pkg-config into lp:thumbnailer

Proposed by Michi Henning
Status: Merged
Approved by: James Henstridge
Approved revision: 146
Merged at revision: 146
Proposed branch: lp:~michihenning/thumbnailer/fix-qt-pkg-config
Merge into: lp:thumbnailer
Diff against target: 200 lines (+53/-51)
7 files modified
CMakeLists.txt (+8/-11)
debian/changelog (+10/-0)
include/internal/version.h.in (+3/-3)
include/unity/thumbnailer/qt/CMakeLists.txt (+2/-1)
src/libthumbnailer-qt/libthumbnailer-qt.pc.in (+4/-8)
src/service/credentialscache.cpp (+13/-14)
src/service/credentialscache.h (+13/-14)
To merge this branch: bzr merge lp:~michihenning/thumbnailer/fix-qt-pkg-config
Reviewer Review Type Date Requested Status
James Henstridge Approve
Review via email: mp+299913@code.launchpad.net

Commit message

* Changed copyright from GPL to LGPL for two files used in storage framework.
* Fixed pkgconfig file for qt library.

Description of the change

* Changed copyright from GPL to LGPL for two files used in storage framework.
* Fixed pkgconfig file for qt library.

To post a comment you must log in.
Revision history for this message
James Henstridge (jamesh) wrote :

Looks good.

review: Approve
147. By Michi Henning

merged:
  James Henstridge 2016-07-18 Fix typo introduced when cleaning up CMake files.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2016-03-17 01:46:29 +0000
+++ CMakeLists.txt 2016-07-19 01:10:03 +0000
@@ -4,11 +4,11 @@
4 set(LOCAL_INSTALL "ON")4 set(LOCAL_INSTALL "ON")
5endif()5endif()
66
7project(thumbnailer C CXX)7cmake_minimum_required(VERSION 3.0.2)
8project(thumbnailer VERSION "2.4.0" LANGUAGES C CXX)
89
9set(CMAKE_INCLUDE_CURRENT_DIR ON)10set(CMAKE_INCLUDE_CURRENT_DIR ON)
1011
11cmake_minimum_required(VERSION 2.8.11)
1212
13# Suppress complaints by cmake about COMPILE_DEFINITIONS_<Config> caused by qt5 cmake macros.13# Suppress complaints by cmake about COMPILE_DEFINITIONS_<Config> caused by qt5 cmake macros.
14cmake_policy(SET CMP0043 OLD)14cmake_policy(SET CMP0043 OLD)
@@ -47,18 +47,15 @@
4747
48add_definitions(-DQT_NO_KEYWORDS)48add_definitions(-DQT_NO_KEYWORDS)
4949
50include(GNUInstallDirs)
51
50# libthumbnailer-qt version52# libthumbnailer-qt version
51set(LIBTHUMBNAILER_QT thumbnailer-qt)53set(LIBTHUMBNAILER_QT thumbnailer-qt)
52set(LIBTHUMBNAILER_QT_SO_VERSION_MAJOR "1")54set(LIBTHUMBNAILER_QT_SO_VERSION_MAJOR "1")
53set(LIBTHUMBNAILER_QT_SO_VERSION_MINOR "0")55set(LIBTHUMBNAILER_QT_SO_VERSION_MINOR "0")
54set(LIBTHUMBNAILER_QT_SO_VERSION_MICRO "0")56set(LIBTHUMBNAILER_QT_SO_VERSION_MICRO "0")
55set(LIBTHUMBNAILER_QT_SO_VERSION ${LIBTHUMBNAILER_QT_SO_VERSION_MAJOR}.${LIBTHUMBNAILER_QT_SO_VERSION_MINOR})57set(LIBTHUMBNAILER_QT_SO_VERSION ${LIBTHUMBNAILER_QT_SO_VERSION_MAJOR}.${LIBTHUMBNAILER_QT_SO_VERSION_MINOR})
56set(LIBTHUMBNAILER_QT_HDR_INSTALL_DIR include/${LIBTHUMBNAILER_QT}-${LIBTHUMBNAILER_QT_SO_VERSION}/unity/thumbnailer/qt)58set(LIBTHUMBNAILER_QT_HDR_INSTALL_DIR ${CMAKE_INSTALL_FULL_INCLUDEDIR}/${LIBTHUMBNAILER_QT}-${LIBTHUMBNAILER_QT_SO_VERSION})
57
58# thumbnailer-service version
59set(THUMBNAILER_VERSION_MAJOR "2")
60set(THUMBNAILER_VERSION_MINOR "4")
61set(THUMBNAILER_VERSION_MICRO "0")
6259
63# Encoding version of cache files60# Encoding version of cache files
64set(THUMBNAILER_CACHE_VERSION "2")61set(THUMBNAILER_CACHE_VERSION "2")
@@ -106,16 +103,16 @@
106103
107include(cmake/UseGSettings.cmake)104include(cmake/UseGSettings.cmake)
108105
109include(GNUInstallDirs)
110set(LIBDIR ${CMAKE_INSTALL_LIBDIR})
111set(SHARE_PRIV_DIR ${CMAKE_INSTALL_LIBDIR}/thumbnailer)106set(SHARE_PRIV_DIR ${CMAKE_INSTALL_LIBDIR}/thumbnailer)
112set(SHARE_PRIV_ABS ${CMAKE_INSTALL_PREFIX}/${SHARE_PRIV_DIR})107set(SHARE_PRIV_ABS ${CMAKE_INSTALL_FULL_LIBDIR}/thumbnailer)
113108
114find_package(Boost COMPONENTS filesystem iostreams regex system REQUIRED)109find_package(Boost COMPONENTS filesystem iostreams regex system REQUIRED)
115find_package(Threads REQUIRED)110find_package(Threads REQUIRED)
116find_package(Qt5Core REQUIRED)111find_package(Qt5Core REQUIRED)
117find_package(Qt5DBus REQUIRED)112find_package(Qt5DBus REQUIRED)
118find_package(Qt5Gui REQUIRED)113find_package(Qt5Gui REQUIRED)
114find_package(Qt5Qml REQUIRED)
115find_package(Qt5Quick REQUIRED)
119116
120include(FindPkgConfig)117include(FindPkgConfig)
121pkg_check_modules(GST_DEPS REQUIRED gstreamer-1.0 gstreamer-plugins-base-1.0 gstreamer-tag-1.0)118pkg_check_modules(GST_DEPS REQUIRED gstreamer-1.0 gstreamer-plugins-base-1.0 gstreamer-tag-1.0)
122119
=== modified file 'debian/changelog'
--- debian/changelog 2016-05-30 11:07:33 +0000
+++ debian/changelog 2016-07-19 01:10:03 +0000
@@ -1,3 +1,13 @@
1thumbnailer (2.4+16.10.20160530.3-0ubuntu3) UNRELEASED; urgency=medium
2
3 [ Michi Henning ]
4 * Changed copyright from GPL to LGPL for two files used in storage framework.
5
6 [ James Henstridge ]
7 * Fixed pkgconfig file for qt library.
8
9 -- Michi Henning <michi.henning@canonical.com> Wed, 13 Jul 2016 16:40:46 +1000
10
1thumbnailer (2.4+16.10.20160530.3-0ubuntu1) yakkety; urgency=medium11thumbnailer (2.4+16.10.20160530.3-0ubuntu1) yakkety; urgency=medium
212
3 [ Michi Henning ]13 [ Michi Henning ]
414
=== modified file 'include/internal/version.h.in'
--- include/internal/version.h.in 2016-02-15 06:51:18 +0000
+++ include/internal/version.h.in 2016-07-19 01:10:03 +0000
@@ -30,9 +30,9 @@
30{30{
3131
32// Software release version.32// Software release version.
33#define THUMBNAILER_VERSION_MAJOR @THUMBNAILER_VERSION_MAJOR@33#define THUMBNAILER_VERSION_MAJOR @PROJECT_VERSION_MAJOR@
34#define THUMBNAILER_VERSION_MINOR @THUMBNAILER_VERSION_MINOR@34#define THUMBNAILER_VERSION_MINOR @PROJECT_VERSION_MINOR@
35#define THUMBNAILER_VERSION_MICRO @THUMBNAILER_VERSION_MICRO@35#define THUMBNAILER_VERSION_MICRO @PROJECT_VERSION_PATCH@
36#define THUMBNAILER_VERSION_FILENAME "thumbnailer-version"36#define THUMBNAILER_VERSION_FILENAME "thumbnailer-version"
3737
38// Encoding version of caches.38// Encoding version of caches.
3939
=== modified file 'include/unity/thumbnailer/qt/CMakeLists.txt'
--- include/unity/thumbnailer/qt/CMakeLists.txt 2015-08-11 09:39:00 +0000
+++ include/unity/thumbnailer/qt/CMakeLists.txt 2016-07-19 01:10:03 +0000
@@ -1,1 +1,2 @@
1install(FILES thumbnailer-qt.h DESTINATION ${LIBTHUMBNAILER_QT_HDR_INSTALL_DIR})1set(prefix "unity/thumbnailer/qt")
2install(FILES thumbnailer-qt.h DESTINATION "${LIBTHUMBNAILER_QT_HDR_INSTALL_DIR}/${prefix}")
23
=== modified file 'src/libthumbnailer-qt/libthumbnailer-qt.pc.in'
--- src/libthumbnailer-qt/libthumbnailer-qt.pc.in 2015-08-04 14:13:14 +0000
+++ src/libthumbnailer-qt/libthumbnailer-qt.pc.in 2016-07-19 01:10:03 +0000
@@ -16,13 +16,9 @@
16# Authored by: Xavi Garcia Mena <xavi.garcia.mena@canonical.com>16# Authored by: Xavi Garcia Mena <xavi.garcia.mena@canonical.com>
17#17#
1818
19prefix=@CMAKE_INSTALL_PREFIX@
20includedir=${prefix}/@LIBTHUMBNAILER_QT_HDR_INSTALL_DIR@
21libdir=${prefix}/@LIBDIR@
22
23Name: libthumbnailer-qt19Name: libthumbnailer-qt
24Description: Thumbnailer Qt library20Description: Thumbnailer Qt library
25Requires: thumbnailer-service21Requires: Qt5Gui
26Version: @LIBTHUMBNAILER_QT_SO_VERSION_MAJOR@.@LIBTHUMBNAILER_QT_SO_VERSION_MINOR@.@LIBTHUMBNAILER_QT_SO_VERSION_MICRO@22Version: @PROJECT_VERSION@
27Libs: -L${libdir} -l@LIBTHUMBNAILER_QT@23Libs: -L@CMAKE_INSTALL_FULL_LIBDIR@ -l@LIBTHUMBNAILER_QT@
28Cflags: -I${includedir}24Cflags: -I@LIBTHUMBNAILER_QT_HDR_INSTALL_DIR@
2925
=== modified file 'src/service/credentialscache.cpp'
--- src/service/credentialscache.cpp 2015-09-29 14:05:53 +0000
+++ src/service/credentialscache.cpp 2016-07-19 01:10:03 +0000
@@ -1,20 +1,19 @@
1/*1/*
2 * Copyright (C) 2015 Canonical, Ltd.2 * Copyright (C) 2015 Canonical Ltd
3 *3 *
4 * This library is free software; you can redistribute it and/or modify it under4 * This program is free software: you can redistribute it and/or modify
5 * the terms of version 3 of the GNU General Public License as published5 * it under the terms of the GNU Lesser General Public License version 3 as
6 * by the Free Software Foundation.6 * published by the Free Software Foundation.
7 *7 *
8 * This library is distributed in the hope that it will be useful, but WITHOUT8 * This program is distributed in the hope that it will be useful,
9 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS9 * but WITHOUT ANY WARRANTY; without even the implied warranty of
10 * FOR A PARTICULAR PURPOSE. See the GNU General Public License for more10 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 * details.11 * GNU Lesser General Public License for more details.
12 *12 *
13 * You should have received a copy of the GNU General Public License13 * You should have received a copy of the GNU Lesser General Public License
14 * along with this program. If not, see <http://www.gnu.org/licenses/>.14 * along with this program. If not, see <http://www.gnu.org/licenses/>.
15 *15 *
16 * Authors:16 * Authors: James Henstridge <james.henstridge@canonical.com>
17 * James Henstridge <james.henstridge@canonical.com>
18 */17 */
1918
20#include "credentialscache.h"19#include "credentialscache.h"
2120
=== modified file 'src/service/credentialscache.h'
--- src/service/credentialscache.h 2015-09-29 14:05:53 +0000
+++ src/service/credentialscache.h 2016-07-19 01:10:03 +0000
@@ -1,20 +1,19 @@
1/*1/*
2 * Copyright (C) 2015 Canonical, Ltd.2 * Copyright (C) 2015 Canonical Ltd
3 *3 *
4 * This library is free software; you can redistribute it and/or modify it under4 * This program is free software: you can redistribute it and/or modify
5 * the terms of version 3 of the GNU General Public License as published5 * it under the terms of the GNU Lesser General Public License version 3 as
6 * by the Free Software Foundation.6 * published by the Free Software Foundation.
7 *7 *
8 * This library is distributed in the hope that it will be useful, but WITHOUT8 * This program is distributed in the hope that it will be useful,
9 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS9 * but WITHOUT ANY WARRANTY; without even the implied warranty of
10 * FOR A PARTICULAR PURPOSE. See the GNU General Public License for more10 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
11 * details.11 * GNU Lesser General Public License for more details.
12 *12 *
13 * You should have received a copy of the GNU General Public License13 * You should have received a copy of the GNU Lesser General Public License
14 * along with this program. If not, see <http://www.gnu.org/licenses/>.14 * along with this program. If not, see <http://www.gnu.org/licenses/>.
15 *15 *
16 * Authors:16 * Authors: James Henstridge <james.henstridge@canonical.com>
17 * James Henstridge <james.henstridge@canonical.com>
18 */17 */
1918
20#pragma once19#pragma once

Subscribers

People subscribed via source and target branches

to all changes: