Merge lp:~fboucault/unity-2d/support_install_in_symlink into lp:unity-2d

Proposed by Florian Boucault
Status: Merged
Approved by: Ugo Riboni
Approved revision: 897
Merged at revision: 897
Proposed branch: lp:~fboucault/unity-2d/support_install_in_symlink
Merge into: lp:unity-2d
Diff against target: 77 lines (+13/-11)
3 files modified
config.h.in (+3/-1)
launcher/app/launcherview.cpp (+7/-7)
libunity-2d-private/src/unity2ddeclarativeview.cpp (+3/-3)
To merge this branch: bzr merge lp:~fboucault/unity-2d/support_install_in_symlink
Reviewer Review Type Date Requested Status
Ugo Riboni Pending
Review via email: mp+91706@code.launchpad.net

Description of the change

Support running installed inside a directory that is a symlink.

isRunningInstalled() is made more reliable by resolving all symlinks contained
in the installation path.
Header inclusions were shuffled around to respect the best practice to include
from the more local to the more global headers and to solve a compilation error
as a result.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'config.h.in'
--- config.h.in 2012-01-24 09:54:34 +0000
+++ config.h.in 2012-02-06 19:50:23 +0000
@@ -6,6 +6,7 @@
6#define UNITY_2D_SCHEMA_FILE "@UNITY_2D_SCHEMA_FILE@"6#define UNITY_2D_SCHEMA_FILE "@UNITY_2D_SCHEMA_FILE@"
77
8#include <QCoreApplication>8#include <QCoreApplication>
9#include <QDir>
9#include "qconf.h"10#include "qconf.h"
1011
11static const char* UNITY2D_DCONF_SCHEMA = "com.canonical.Unity2d";12static const char* UNITY2D_DCONF_SCHEMA = "com.canonical.Unity2d";
@@ -54,7 +55,8 @@
5455
5556
56inline bool isRunningInstalled() {57inline bool isRunningInstalled() {
57 return (QCoreApplication::applicationDirPath() == (INSTALL_PREFIX "/" INSTALL_BINDIR));58 static bool installed = (QCoreApplication::applicationDirPath() == QDir((INSTALL_PREFIX "/" INSTALL_BINDIR)).canonicalPath());
59 return installed;
58}60}
5961
60inline QString unity2dDirectory() {62inline QString unity2dDirectory() {
6163
=== modified file 'launcher/app/launcherview.cpp'
--- launcher/app/launcherview.cpp 2011-11-25 11:13:44 +0000
+++ launcher/app/launcherview.cpp 2012-02-06 19:50:23 +0000
@@ -19,6 +19,13 @@
1919
20#include "launcherview.h"20#include "launcherview.h"
2121
22#include <config.h>
23#include <keyboardmodifiersmonitor.h>
24#include <hotkey.h>
25#include <hotkeymonitor.h>
26#include <keymonitor.h>
27#include <debug_p.h>
28
22#include <QApplication>29#include <QApplication>
23#include <QDesktopWidget>30#include <QDesktopWidget>
24#include <QX11Info>31#include <QX11Info>
@@ -37,13 +44,6 @@
37#include <X11/Xlib.h>44#include <X11/Xlib.h>
38#include <X11/Xatom.h>45#include <X11/Xatom.h>
3946
40#include <keyboardmodifiersmonitor.h>
41#include <hotkey.h>
42#include <hotkeymonitor.h>
43#include <keymonitor.h>
44#include <debug_p.h>
45#include <config.h>
46
47static const int KEY_HOLD_THRESHOLD = 250;47static const int KEY_HOLD_THRESHOLD = 250;
4848
49static const char* DASH_DBUS_SERVICE = "com.canonical.Unity2d.Dash";49static const char* DASH_DBUS_SERVICE = "com.canonical.Unity2d.Dash";
5050
=== modified file 'libunity-2d-private/src/unity2ddeclarativeview.cpp'
--- libunity-2d-private/src/unity2ddeclarativeview.cpp 2012-02-03 20:59:54 +0000
+++ libunity-2d-private/src/unity2ddeclarativeview.cpp 2012-02-06 19:50:23 +0000
@@ -16,6 +16,9 @@
1616
17#include "unity2ddeclarativeview.h"17#include "unity2ddeclarativeview.h"
1818
19#include <debug_p.h>
20#include <config.h>
21
19#include <QDebug>22#include <QDebug>
20#include <QGLWidget>23#include <QGLWidget>
21#include <QVariant>24#include <QVariant>
@@ -26,9 +29,6 @@
26#include <X11/Xlib.h>29#include <X11/Xlib.h>
27#include <X11/Xatom.h>30#include <X11/Xatom.h>
2831
29#include <debug_p.h>
30#include <config.h>
31
32// libwnck32// libwnck
33extern "C" {33extern "C" {
34#include <libwnck/libwnck.h>34#include <libwnck/libwnck.h>

Subscribers

People subscribed via source and target branches