Merge lp:~matzipan/pantheon-mail/migrations-fix into lp:~elementary-apps/pantheon-mail/trunk

Proposed by Zisu Andrei
Status: Merged
Approved by: Danielle Foré
Approved revision: 1979
Merged at revision: 1982
Proposed branch: lp:~matzipan/pantheon-mail/migrations-fix
Merge into: lp:~elementary-apps/pantheon-mail/trunk
Diff against target: 22 lines (+2/-1)
2 files modified
CMakeLists.txt (+0/-1)
data/CMakeLists.txt (+2/-0)
To merge this branch: bzr merge lp:~matzipan/pantheon-mail/migrations-fix
Reviewer Review Type Date Requested Status
Danielle Foré Approve
Review via email: mp+286913@code.launchpad.net

Commit message

Moved sqlite migration files where local build expects them to be

To post a comment you must log in.
Revision history for this message
Danielle Foré (danrabbit) wrote :

Hey thanks for you branch! Please use "bzr mv" to relocate files and folders instead. This makes the diff smaller and easier to verify

review: Needs Fixing
Revision history for this message
Zisu Andrei (matzipan) wrote :

I noticed that. I didn't know bazaar has that functionality. I'm too used to git :(

1979. By Zisu Andrei

Moved sqlite migration files where local build expects them to be

Revision history for this message
Zisu Andrei (matzipan) wrote :

Fixed.

Revision history for this message
Danielle Foré (danrabbit) wrote :

Works for me, thanks!

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 2016-02-17 01:21:06 +0000
3+++ CMakeLists.txt 2016-02-23 17:03:47 +0000
4@@ -98,6 +98,5 @@
5
6 add_subdirectory(data)
7 add_subdirectory(po)
8-add_subdirectory(sql)
9 add_subdirectory(src)
10
11
12=== modified file 'data/CMakeLists.txt'
13--- data/CMakeLists.txt 2016-02-17 18:59:49 +0000
14+++ data/CMakeLists.txt 2016-02-23 17:03:47 +0000
15@@ -37,3 +37,5 @@
16
17 add_subdirectory(theming)
18 add_subdirectory(ui)
19+add_subdirectory(sql)
20+
21
22=== renamed directory 'sql' => 'data/sql'

Subscribers

People subscribed via source and target branches