Merge lp:~jpakkane/thumbnailer/install-fix into lp:thumbnailer

Proposed by Jussi Pakkanen
Status: Merged
Approved by: Jussi Pakkanen
Approved revision: 57
Merged at revision: 57
Proposed branch: lp:~jpakkane/thumbnailer/install-fix
Merge into: lp:thumbnailer
Diff against target: 20 lines (+2/-2)
2 files modified
CMakeLists.txt (+1/-1)
debian/libthumbnailer0.install (+1/-1)
To merge this branch: bzr merge lp:~jpakkane/thumbnailer/install-fix
Reviewer Review Type Date Requested Status
Jamie Strandboge Approve
PS Jenkins bot (community) continuous-integration Approve
Unity Team Pending
Review via email: mp+190150@code.launchpad.net

Commit message

Fix install directory of vs-thumb.

Description of the change

Fix install directory of vs-thumb. Apparmor changes have already been done elsewhere so this should just work.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Jamie Strandboge (jdstrand) wrote :

ACK, vs-thumb on amd64 is now installed to:
/usr/lib/x86_64-linux-gnu/thumbnailer/vs-thumb

which matches the apparmor rule:
/usr/lib/@{multiarch}/thumbnailer/vs-thumb ixr,

and is policy compliant.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2013-10-02 09:43:16 +0000
3+++ CMakeLists.txt 2013-10-09 14:38:25 +0000
4@@ -17,7 +17,7 @@
5
6 include(GNUInstallDirs)
7 set(LIBDIR ${CMAKE_INSTALL_LIBDIR})
8-set(SHARE_PRIV_DIR share/thumbnailer)
9+set(SHARE_PRIV_DIR ${CMAKE_INSTALL_LIBDIR}/thumbnailer)
10 set(SHARE_PRIV_ABS ${CMAKE_INSTALL_PREFIX}/${SHARE_PRIV_DIR})
11
12 include(FindPkgConfig)
13
14=== modified file 'debian/libthumbnailer0.install'
15--- debian/libthumbnailer0.install 2013-10-02 13:39:45 +0000
16+++ debian/libthumbnailer0.install 2013-10-09 14:38:25 +0000
17@@ -1,2 +1,2 @@
18 usr/lib/*/libthumb*.so.*
19-usr/share/thumbnailer/vs-thumb
20+usr/lib/*/thumbnailer/vs-thumb

Subscribers

People subscribed via source and target branches

to all changes: