Merge lp:~popey/ubuntu-filemanager-app/fix-1288885 into lp:ubuntu-filemanager-app

Status: Rejected
Rejected by: David Planella
Proposed branch: lp:~popey/ubuntu-filemanager-app/fix-1288885
Merge into: lp:ubuntu-filemanager-app
Diff against target: 26 lines (+7/-1)
2 files modified
CMakeLists.txt (+1/-1)
debian/changelog (+6/-0)
To merge this branch: bzr merge lp:~popey/ubuntu-filemanager-app/fix-1288885
Reviewer Review Type Date Requested Status
David Planella Disapprove
Alan Pope 🍺🐧🐱 πŸ¦„ (community) Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Needs Fixing
Review via email: mp+210173@code.launchpad.net

Commit message

To post a comment you must log in.
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Alan Pope 🍺🐧🐱 πŸ¦„ (popey) :
review: Approve
Revision history for this message
David Planella (dpm) wrote :

Thanks for the fix. This has now been addressed differently in trunk: the app now builds the plugin and it is started by an executable instead of using qmlscene.

review: Disapprove

Unmerged revisions

138. By Alan Pope 🍺🐧🐱 πŸ¦„

fix bug 1288885

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2014-02-19 00:00:15 +0000
+++ CMakeLists.txt 2014-03-10 10:15:57 +0000
@@ -23,7 +23,7 @@
23 set(CMAKE_INSTALL_PREFIX /)23 set(CMAKE_INSTALL_PREFIX /)
24 set(CMAKE_INSTALL_BINDIR /)24 set(CMAKE_INSTALL_BINDIR /)
25 set(DATA_DIR /)25 set(DATA_DIR /)
26 set(EXEC "qmlscene ${MAIN_QML}")26 set(EXEC "qmlscene -qt5 ${MAIN_QML}")
27 set(DESKTOP_DIR ${DATA_DIR})27 set(DESKTOP_DIR ${DATA_DIR})
28 set(URLS_DIR ${DATA_DIR})28 set(URLS_DIR ${DATA_DIR})
29else(CLICK_MODE)29else(CLICK_MODE)
3030
=== modified file 'debian/changelog'
--- debian/changelog 2013-04-19 20:30:19 +0000
+++ debian/changelog 2014-03-10 10:15:57 +0000
@@ -1,3 +1,9 @@
1ubuntu-filemanager-app (0.1.1ubuntu1) UNRELEASED; urgency=medium
2
3 * Add -qt5 command line option (LP: #1288885)
4
5 -- Alan Pope <popey@ubuntu.com> Mon, 10 Mar 2014 10:14:49 +0000
6
1ubuntu-filemanager-app (0.1.1) raring; urgency=low7ubuntu-filemanager-app (0.1.1) raring; urgency=low
28
3 * Initial release9 * Initial release

Subscribers

People subscribed via source and target branches