Merge lp:~phablet-team/camera-app/disk-space-monitor into lp:camera-app

Proposed by Florian Boucault
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
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

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
492. By Ugo Riboni

Fix an incorrect stub in the unit tests

493. By Ugo Riboni

Merge upstream changes

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
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.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
494. By Ugo Riboni

Add apparmor permissions so we can read the system files we need to query storage information

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
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

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
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 }

Subscribers

People subscribed via source and target branches