Merge lp:~popey/ubuntu-filemanager-app/set-bzr-revno into lp:ubuntu-filemanager-app

Status: Merged
Approved by: Carlos Jose Mazieri
Approved revision: 481
Merged at revision: 483
Proposed branch: lp:~popey/ubuntu-filemanager-app/set-bzr-revno
Merge into: lp:ubuntu-filemanager-app
Diff against target: 17 lines (+7/-0)
1 file modified
CMakeLists.txt (+7/-0)
To merge this branch: bzr merge lp:~popey/ubuntu-filemanager-app/set-bzr-revno
Reviewer Review Type Date Requested Status
Carlos Jose Mazieri Approve
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Review via email: mp+274957@code.launchpad.net

Commit message

Correctly set the bzr revno during build. Otherwise in some circumstances it builds with "latest" in the filename, where we standardised on the bzr revision number in the filename. This fixes that.

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
Carlos Jose Mazieri (carlos-mazieri) wrote :

Nice, Thanks.

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 2015-10-13 16:49:05 +0000
+++ CMakeLists.txt 2015-10-19 22:15:50 +0000
@@ -25,6 +25,13 @@
25add_definitions(-D_LARGEFILE_SOURCE)25add_definitions(-D_LARGEFILE_SOURCE)
26add_definitions(-D_FILE_OFFSET_BITS=64)26add_definitions(-D_FILE_OFFSET_BITS=64)
2727
28# Sets BZR_REVNO
29execute_process(
30 COMMAND bzr revno ${CMAKE_SOURCE_DIR}
31 OUTPUT_VARIABLE BZR_REVNO
32 OUTPUT_STRIP_TRAILING_WHITESPACE
33)
34
28if(CLICK_MODE)35if(CLICK_MODE)
29 if(NOT DEFINED BZR_SOURCE)36 if(NOT DEFINED BZR_SOURCE)
30 set(BZR_SOURCE "lp:${LP_PROJECT}")37 set(BZR_SOURCE "lp:${LP_PROJECT}")

Subscribers

People subscribed via source and target branches