Merge lp:~kamstrup/bamf-qt/fixes-for-jhbuild into lp:bamf-qt

Proposed by Mikkel Kamstrup Erlandsen
Status: Merged
Approved by: Gerry Boland
Approved revision: 392
Merged at revision: 392
Proposed branch: lp:~kamstrup/bamf-qt/fixes-for-jhbuild
Merge into: lp:bamf-qt
Diff against target: 25 lines (+10/-3)
1 file modified
CMakeLists.txt (+10/-3)
To merge this branch: bzr merge lp:~kamstrup/bamf-qt/fixes-for-jhbuild
Reviewer Review Type Date Requested Status
Gerry Boland (community) Approve
Review via email: mp+85614@code.launchpad.net

Description of the change

Fix installation of libQtBamfQml.so into qt4/imports under jhbuild, by allowing one to override the directory by setting -DQT_IMPORTS_DIR=/my/dir on the cmake command line

I am working on adding the entire unity-2d stack to the lp:unity-jhbuild setup, and this fix is a requirement to get that working

To post a comment you must log in.
391. By Mikkel Kamstrup Erlandsen

Fix installation of libQtBamfQml.so into qt4/imports under jhbuild, by allowing one to override the directory by setting -DQT_IMPORTS_DIR on the cmake commend line

392. By Mikkel Kamstrup Erlandsen

Update comments on QT_IMPORTS_DIR and QT_INSTALL_IMPORTS usage

Revision history for this message
Gerry Boland (gerboland) wrote :

Works well, will accept.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2011-11-29 06:21:49 +0000
+++ CMakeLists.txt 2011-12-14 12:34:04 +0000
@@ -100,11 +100,18 @@
100add_custom_target(check)100add_custom_target(check)
101101
102# Install102# Install
103# We don’t refer to QT_IMPORTS_DIR, because it requires at least one QML plugin103# You can override the directory in which we install QML imports
104# installed. Query qmake instead. (see bug #894805)104# by setting QT_IMPORTS_DIR to a path on the cmake command line
105_qt4_query_qmake(QT_INSTALL_IMPORTS qt_imports_dir)105if(QT_IMPORTS_DIR)
106 set(qt_imports_dir ${QT_IMPORTS_DIR})
107else()
108 # Query qmake directly because of
109 # https://bugs.launchpad.net/ubuntu/+source/cmake/+bug/894805
110 _qt4_query_qmake(QT_INSTALL_IMPORTS qt_imports_dir)
111endif()
106set(IMPORT_INSTALL_DIR ${qt_imports_dir}/bamf)112set(IMPORT_INSTALL_DIR ${qt_imports_dir}/bamf)
107set(INCLUDE_INSTALL_DIR include/QtBamf)113set(INCLUDE_INSTALL_DIR include/QtBamf)
114message(STATUS "Installing imports in ${IMPORT_INSTALL_DIR}")
108115
109## QtBamf116## QtBamf
110install(TARGETS QtBamf117install(TARGETS QtBamf

Subscribers

People subscribed via source and target branches