Merge lp:~chocanto/file-qml-plugin/debian into lp:file-qml-plugin

Proposed by Anthony Granger
Status: Merged
Merged at revision: 6
Proposed branch: lp:~chocanto/file-qml-plugin/debian
Merge into: lp:file-qml-plugin
Diff against target: 111 lines (+80/-0)
6 files modified
debian/changelog (+5/-0)
debian/compat (+1/-0)
debian/control (+17/-0)
debian/copyright (+45/-0)
debian/rules (+11/-0)
debian/source/format (+1/-0)
To merge this branch: bzr merge lp:~chocanto/file-qml-plugin/debian
Reviewer Review Type Date Requested Status
Nekhelesh Ramananthan (community) Needs Information
Review via email: mp+187020@code.launchpad.net

Commit message

Debian packaging

Description of the change

Initial merge of debian packaging

To post a comment you must log in.
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

I start this review by stating that I am no debian expert. I am merely comparing this debian folder with the clock app and my own judgement. Please fix the following in a separate commit so that it is easy to see if they have been fixed.

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

25 +Maintainer: Ubuntu App Cats <email address hidden>

I am wondering why the Ubuntu App Cats is the maintainer of this library. Shouldn't it be you? I guess you need to decide if you are going to merge it to the core apps PPA in which case the above is correct, otherwise you need to change this.

review: Needs Information
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

48 +Files: *
49 +Copyright: 2013 Canonical Ltd.
50 +License: GPL-3

Here you can remove the 2013 Canonical Ltd. and replace it with the contributors's names in the format,

Name <email>

The rest of the copyright seems okay.

review: Needs Fixing
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

45 +Upstream-Name: qtdeclarative5-poppler-qml-plugin

It would be preferable if this upstream package name is the same as the soruce package name itself.

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

34 +Architecture: any

Please cross check this with others. Since the qml plugin is c++, we need to be careful about this.

review: Needs Information
lp:~chocanto/file-qml-plugin/debian updated
8. By Anthony Granger

Change maintainer of the package and some informations in the copyright file

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== added directory 'debian'
=== added file 'debian/changelog'
--- debian/changelog 1970-01-01 00:00:00 +0000
+++ debian/changelog 2013-09-24 18:33:53 +0000
@@ -0,0 +1,5 @@
1file-qml-plugin (0.1-1) saucy; urgency=low
2
3 * Initial release
4
5 -- Granger Anthony <grangeranthony@gmail.com> Mon, 23 Sep 2013 14:51:00 +0200
06
=== added file 'debian/compat'
--- debian/compat 1970-01-01 00:00:00 +0000
+++ debian/compat 2013-09-24 18:33:53 +0000
@@ -0,0 +1,1 @@
18
02
=== added file 'debian/control'
--- debian/control 1970-01-01 00:00:00 +0000
+++ debian/control 2013-09-24 18:33:53 +0000
@@ -0,0 +1,17 @@
1Source: file-qml-plugin
2Section: libs
3Priority: extra
4Maintainer: Anthony Granger <grangeranthony@gmail.com>
5Build-Depends: debhelper (>= 9),
6 qt5-default,
7 qtbase5-dev,
8 qtdeclarative5-dev
9Standards-Version: 3.9.4
10Homepage: https://launchpad.net/file-qml-plugin
11Vcs-Bzr: https://code.launchpad.net/~chocanto/file-qml-plugin/trunk
12
13Architecture: any
14Package: file-qml-plugin
15Depends: ${shlibs:Depends}, ${misc:Depends}
16Description: File access plugin for QML
17 This package contains a simple QML plugin to read a file.
018
=== added file 'debian/copyright'
--- debian/copyright 1970-01-01 00:00:00 +0000
+++ debian/copyright 2013-09-24 18:33:53 +0000
@@ -0,0 +1,45 @@
1Format: http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
2Upstream-Name: file-qml-plugin
3Source:
4
5Files: *
6Copyright: Anthony Granger <grangeranthony@gmail.com>
7License: GPL-3
8
9Files: debian/*
10Copyright: Anthony Granger <grangeranthony@gmail.com>
11License: LGPL-3
12
13License: GPL-3
14 This package is free software; you can redistribute it and/or
15 modify it under the terms of the GNU General Public
16 License as published by the Free Software Foundation; either
17 version 3 of the License.
18 .
19 This package is distributed in the hope that it will be useful,
20 but WITHOUT ANY WARRANTY; without even the implied warranty of
21 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
22 General Public License for more details.
23 .
24 You should have received a copy of the GNU General Public License
25 along with this program. If not, see <http://www.gnu.org/licenses/>.
26 .
27 On Debian systems, the complete text of the GNU General
28 Public License can be found in "/usr/share/common-licenses/GPL-3".
29
30License: LGPL-3
31 This package is free software; you can redistribute it and/or
32 modify it under the terms of the GNU Lesser General Public
33 License as published by the Free Software Foundation; either
34 version 3 of the License.
35 .
36 This package is distributed in the hope that it will be useful,
37 but WITHOUT ANY WARRANTY; without even the implied warranty of
38 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
39 Lesser General Public License for more details.
40 .
41 You should have received a copy of the GNU General Public License
42 along with this program. If not, see <http://www.gnu.org/licenses/>.
43 .
44 On Debian systems, the complete text of the GNU Lesser General
45 Public License can be found in "/usr/share/common-licenses/LGPL-3".
046
=== added file 'debian/rules'
--- debian/rules 1970-01-01 00:00:00 +0000
+++ debian/rules 2013-09-24 18:33:53 +0000
@@ -0,0 +1,11 @@
1#!/usr/bin/make -f
2# -*- makefile -*-
3
4# Uncomment this to turn on verbose mode.
5#export DH_VERBOSE=1
6
7%:
8 dh $@
9
10override_dh_install:
11 dh_install --fail-missing
012
=== added directory 'debian/source'
=== added file 'debian/source/format'
--- debian/source/format 1970-01-01 00:00:00 +0000
+++ debian/source/format 2013-09-24 18:33:53 +0000
@@ -0,0 +1,1 @@
11.0

Subscribers

People subscribed via source and target branches

to all changes: