Merge lp:~phablet-team/camera-app/disk-space-monitor into lp:camera-app
- disk-space-monitor
- Merge into trunk
Status: | Merged |
---|---|
Approved by: | Bill Filler |
Approved revision: | 494 |
Merged at revision: | 483 |
Proposed branch: | lp:~phablet-team/camera-app/disk-space-monitor |
Merge into: | lp:camera-app |
Diff against target: |
229 lines (+50/-125) 5 files modified
CameraApp/qstorageinfo.cpp (+11/-31) CameraApp/qstorageinfo.h (+11/-31) CameraApp/qstorageinfo_p.h (+12/-31) CameraApp/qstorageinfo_unix.cpp (+11/-31) camera-apparmor.json (+5/-1) |
To merge this branch: | bzr merge lp:~phablet-team/camera-app/disk-space-monitor |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot | continuous-integration | Needs Fixing | |
Ubuntu Phablet Team | Pending | ||
Review via email: mp+247528@code.launchpad.net |
Commit message
warn the user of low disk space while recording or taking a picture
Description of the change
Warn the user when low free space remains.
When critically low free space remains:
- stop any ongoing video recording
- prevent the user from taking pictures and shooting videos
PS Jenkins bot (ps-jenkins) wrote : | # |
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:492
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 492. By Ugo Riboni
-
Fix an incorrect stub in the unit tests
- 493. By Ugo Riboni
-
Merge upstream changes
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:493
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
ABORTED: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
Francis Ginther (fginther) wrote : | # |
The mako test in the above ci results was aborted due to a mako device with a bad image. The CI run has been re-triggered and will run on a different device.
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:493
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 494. By Ugo Riboni
-
Add apparmor permissions so we can read the system files we need to query storage information
PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:494
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 495. By Ugo Riboni
-
Change the license headers from QT stuff to plain GPL3, as we are allowed to do that and licensecheck understands it
Preview Diff
1 | === modified file 'CameraApp/qstorageinfo.cpp' |
2 | --- CameraApp/qstorageinfo.cpp 2015-01-22 16:15:08 +0000 |
3 | +++ CameraApp/qstorageinfo.cpp 2015-01-27 14:34:46 +0000 |
4 | @@ -5,37 +5,17 @@ |
5 | ** |
6 | ** This file is part of the QtCore module of the Qt Toolkit. |
7 | ** |
8 | -** $QT_BEGIN_LICENSE:LGPL$ |
9 | -** Commercial License Usage |
10 | -** Licensees holding valid commercial Qt licenses may use this file in |
11 | -** accordance with the commercial license agreement provided with the |
12 | -** Software or, alternatively, in accordance with the terms contained in |
13 | -** a written agreement between you and Digia. For licensing terms and |
14 | -** conditions see http://qt.digia.com/licensing. For further information |
15 | -** use the contact form at http://qt.digia.com/contact-us. |
16 | -** |
17 | -** GNU Lesser General Public License Usage |
18 | -** Alternatively, this file may be used under the terms of the GNU Lesser |
19 | -** General Public License version 2.1 as published by the Free Software |
20 | -** Foundation and appearing in the file LICENSE.LGPL included in the |
21 | -** packaging of this file. Please review the following information to |
22 | -** ensure the GNU Lesser General Public License version 2.1 requirements |
23 | -** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. |
24 | -** |
25 | -** In addition, as a special exception, Digia gives you certain additional |
26 | -** rights. These rights are described in the Digia Qt LGPL Exception |
27 | -** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. |
28 | -** |
29 | -** GNU General Public License Usage |
30 | -** Alternatively, this file may be used under the terms of the GNU |
31 | -** General Public License version 3.0 as published by the Free Software |
32 | -** Foundation and appearing in the file LICENSE.GPL included in the |
33 | -** packaging of this file. Please review the following information to |
34 | -** ensure the GNU General Public License version 3.0 requirements will be |
35 | -** met: http://www.gnu.org/copyleft/gpl.html. |
36 | -** |
37 | -** |
38 | -** $QT_END_LICENSE$ |
39 | +** This program is free software; you can redistribute it and/or modify |
40 | +** it under the terms of the GNU General Public License as published by |
41 | +** the Free Software Foundation; version 3. |
42 | +** |
43 | +** This program is distributed in the hope that it will be useful, |
44 | +** but WITHOUT ANY WARRANTY; without even the implied warranty of |
45 | +** MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
46 | +** GNU General Public License for more details. |
47 | +** |
48 | +** You should have received a copy of the GNU General Public License |
49 | +** along with this program. If not, see <http://www.gnu.org/licenses/>. |
50 | ** |
51 | ****************************************************************************/ |
52 | |
53 | |
54 | === modified file 'CameraApp/qstorageinfo.h' |
55 | --- CameraApp/qstorageinfo.h 2015-01-22 16:15:08 +0000 |
56 | +++ CameraApp/qstorageinfo.h 2015-01-27 14:34:46 +0000 |
57 | @@ -5,37 +5,17 @@ |
58 | ** |
59 | ** This file is part of the QtCore module of the Qt Toolkit. |
60 | ** |
61 | -** $QT_BEGIN_LICENSE:LGPL$ |
62 | -** Commercial License Usage |
63 | -** Licensees holding valid commercial Qt licenses may use this file in |
64 | -** accordance with the commercial license agreement provided with the |
65 | -** Software or, alternatively, in accordance with the terms contained in |
66 | -** a written agreement between you and Digia. For licensing terms and |
67 | -** conditions see http://qt.digia.com/licensing. For further information |
68 | -** use the contact form at http://qt.digia.com/contact-us. |
69 | -** |
70 | -** GNU Lesser General Public License Usage |
71 | -** Alternatively, this file may be used under the terms of the GNU Lesser |
72 | -** General Public License version 2.1 as published by the Free Software |
73 | -** Foundation and appearing in the file LICENSE.LGPL included in the |
74 | -** packaging of this file. Please review the following information to |
75 | -** ensure the GNU Lesser General Public License version 2.1 requirements |
76 | -** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. |
77 | -** |
78 | -** In addition, as a special exception, Digia gives you certain additional |
79 | -** rights. These rights are described in the Digia Qt LGPL Exception |
80 | -** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. |
81 | -** |
82 | -** GNU General Public License Usage |
83 | -** Alternatively, this file may be used under the terms of the GNU |
84 | -** General Public License version 3.0 as published by the Free Software |
85 | -** Foundation and appearing in the file LICENSE.GPL included in the |
86 | -** packaging of this file. Please review the following information to |
87 | -** ensure the GNU General Public License version 3.0 requirements will be |
88 | -** met: http://www.gnu.org/copyleft/gpl.html. |
89 | -** |
90 | -** |
91 | -** $QT_END_LICENSE$ |
92 | +** This program is free software; you can redistribute it and/or modify |
93 | +** it under the terms of the GNU General Public License as published by |
94 | +** the Free Software Foundation; version 3. |
95 | +** |
96 | +** This program is distributed in the hope that it will be useful, |
97 | +** but WITHOUT ANY WARRANTY; without even the implied warranty of |
98 | +** MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
99 | +** GNU General Public License for more details. |
100 | +** |
101 | +** You should have received a copy of the GNU General Public License |
102 | +** along with this program. If not, see <http://www.gnu.org/licenses/>. |
103 | ** |
104 | ****************************************************************************/ |
105 | |
106 | |
107 | === modified file 'CameraApp/qstorageinfo_p.h' |
108 | --- CameraApp/qstorageinfo_p.h 2015-01-22 16:15:08 +0000 |
109 | +++ CameraApp/qstorageinfo_p.h 2015-01-27 14:34:46 +0000 |
110 | @@ -3,39 +3,20 @@ |
111 | ** Copyright (C) 2014 Ivan Komissarov <ABBAPOH@gmail.com> |
112 | ** Contact: http://www.qt-project.org/legal |
113 | ** |
114 | +** |
115 | ** This file is part of the QtCore module of the Qt Toolkit. |
116 | ** |
117 | -** $QT_BEGIN_LICENSE:LGPL$ |
118 | -** Commercial License Usage |
119 | -** Licensees holding valid commercial Qt licenses may use this file in |
120 | -** accordance with the commercial license agreement provided with the |
121 | -** Software or, alternatively, in accordance with the terms contained in |
122 | -** a written agreement between you and Digia. For licensing terms and |
123 | -** conditions see http://qt.digia.com/licensing. For further information |
124 | -** use the contact form at http://qt.digia.com/contact-us. |
125 | -** |
126 | -** GNU Lesser General Public License Usage |
127 | -** Alternatively, this file may be used under the terms of the GNU Lesser |
128 | -** General Public License version 2.1 as published by the Free Software |
129 | -** Foundation and appearing in the file LICENSE.LGPL included in the |
130 | -** packaging of this file. Please review the following information to |
131 | -** ensure the GNU Lesser General Public License version 2.1 requirements |
132 | -** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. |
133 | -** |
134 | -** In addition, as a special exception, Digia gives you certain additional |
135 | -** rights. These rights are described in the Digia Qt LGPL Exception |
136 | -** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. |
137 | -** |
138 | -** GNU General Public License Usage |
139 | -** Alternatively, this file may be used under the terms of the GNU |
140 | -** General Public License version 3.0 as published by the Free Software |
141 | -** Foundation and appearing in the file LICENSE.GPL included in the |
142 | -** packaging of this file. Please review the following information to |
143 | -** ensure the GNU General Public License version 3.0 requirements will be |
144 | -** met: http://www.gnu.org/copyleft/gpl.html. |
145 | -** |
146 | -** |
147 | -** $QT_END_LICENSE$ |
148 | +** This program is free software; you can redistribute it and/or modify |
149 | +** it under the terms of the GNU General Public License as published by |
150 | +** the Free Software Foundation; version 3. |
151 | +** |
152 | +** This program is distributed in the hope that it will be useful, |
153 | +** but WITHOUT ANY WARRANTY; without even the implied warranty of |
154 | +** MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
155 | +** GNU General Public License for more details. |
156 | +** |
157 | +** You should have received a copy of the GNU General Public License |
158 | +** along with this program. If not, see <http://www.gnu.org/licenses/>. |
159 | ** |
160 | ****************************************************************************/ |
161 | |
162 | |
163 | === modified file 'CameraApp/qstorageinfo_unix.cpp' |
164 | --- CameraApp/qstorageinfo_unix.cpp 2015-01-22 16:15:08 +0000 |
165 | +++ CameraApp/qstorageinfo_unix.cpp 2015-01-27 14:34:46 +0000 |
166 | @@ -5,37 +5,17 @@ |
167 | ** |
168 | ** This file is part of the QtCore module of the Qt Toolkit. |
169 | ** |
170 | -** $QT_BEGIN_LICENSE:LGPL$ |
171 | -** Commercial License Usage |
172 | -** Licensees holding valid commercial Qt licenses may use this file in |
173 | -** accordance with the commercial license agreement provided with the |
174 | -** Software or, alternatively, in accordance with the terms contained in |
175 | -** a written agreement between you and Digia. For licensing terms and |
176 | -** conditions see http://qt.digia.com/licensing. For further information |
177 | -** use the contact form at http://qt.digia.com/contact-us. |
178 | -** |
179 | -** GNU Lesser General Public License Usage |
180 | -** Alternatively, this file may be used under the terms of the GNU Lesser |
181 | -** General Public License version 2.1 as published by the Free Software |
182 | -** Foundation and appearing in the file LICENSE.LGPL included in the |
183 | -** packaging of this file. Please review the following information to |
184 | -** ensure the GNU Lesser General Public License version 2.1 requirements |
185 | -** will be met: http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. |
186 | -** |
187 | -** In addition, as a special exception, Digia gives you certain additional |
188 | -** rights. These rights are described in the Digia Qt LGPL Exception |
189 | -** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. |
190 | -** |
191 | -** GNU General Public License Usage |
192 | -** Alternatively, this file may be used under the terms of the GNU |
193 | -** General Public License version 3.0 as published by the Free Software |
194 | -** Foundation and appearing in the file LICENSE.GPL included in the |
195 | -** packaging of this file. Please review the following information to |
196 | -** ensure the GNU General Public License version 3.0 requirements will be |
197 | -** met: http://www.gnu.org/copyleft/gpl.html. |
198 | -** |
199 | -** |
200 | -** $QT_END_LICENSE$ |
201 | +** This program is free software; you can redistribute it and/or modify |
202 | +** it under the terms of the GNU General Public License as published by |
203 | +** the Free Software Foundation; version 3. |
204 | +** |
205 | +** This program is distributed in the hope that it will be useful, |
206 | +** but WITHOUT ANY WARRANTY; without even the implied warranty of |
207 | +** MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
208 | +** GNU General Public License for more details. |
209 | +** |
210 | +** You should have received a copy of the GNU General Public License |
211 | +** along with this program. If not, see <http://www.gnu.org/licenses/>. |
212 | ** |
213 | ****************************************************************************/ |
214 | |
215 | |
216 | === modified file 'camera-apparmor.json' |
217 | --- camera-apparmor.json 2014-12-02 12:59:18 +0000 |
218 | +++ camera-apparmor.json 2015-01-27 14:34:46 +0000 |
219 | @@ -10,5 +10,9 @@ |
220 | "content_exchange_source", |
221 | "location" |
222 | ], |
223 | - "policy_version": 1.2 |
224 | + "policy_version": 1.2, |
225 | + "read_path": [ |
226 | + "@{PROC}/*/mounts", |
227 | + "/dev/disk/by-label/" |
228 | + ] |
229 | } |
FAILED: Continuous integration, rev:492 /code.launchpad .net/~phablet- team/camera- app/disk- space-monitor/ +merge/ 247528/ +edit-commit- message
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https:/
http:// jenkins. qa.ubuntu. com/job/ camera- app-ci/ 364/ jenkins. qa.ubuntu. com/job/ camera- app-vivid- amd64-ci/ 60/console jenkins. qa.ubuntu. com/job/ camera- app-vivid- armhf-ci/ 60/console jenkins. qa.ubuntu. com/job/ camera- app-vivid- i386-ci/ 60/console jenkins. qa.ubuntu. com/job/ generic- click-autopilot -vivid- touch/97/ console jenkins. qa.ubuntu. com/job/ generic- mediumtests- vivid/453/ console jenkins. qa.ubuntu. com/job/ generic- click-autopilot -runner- mako/737/ console jenkins. qa.ubuntu. com/job/ generic- click-builder- vivid-armhf/ 193 s-jenkins. ubuntu- ci:8080/ job/touch- flash-device/ 17424 jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- vivid-amd64/ 545/console
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/camera- app-ci/ 364/rebuild
http://