Merge lp:~noskcaj/ubuntu/wily/dell-dup/dh-python into lp:ubuntu/wily/dell-dup

Proposed by Jackson Doak
Status: Work in progress
Proposed branch: lp:~noskcaj/ubuntu/wily/dell-dup/dh-python
Merge into: lp:ubuntu/wily/dell-dup
Diff against target: 130 lines (+33/-35)
8 files modified
debian/changelog (+8/-0)
debian/compat (+1/-1)
debian/control (+13/-5)
debian/copyright (+1/-1)
debian/pycompat (+0/-1)
debian/pyversions (+0/-1)
debian/rules (+9/-26)
debian/source/format (+1/-0)
To merge this branch: bzr merge lp:~noskcaj/ubuntu/wily/dell-dup/dh-python
Reviewer Review Type Date Requested Status
Jared Dominguez (community) Abstain
Mario Limonciello Abstain
Luke Faraone (community) Needs Information
Ubuntu branches Pending
Review via email: mp+264080@code.launchpad.net

Description of the change

Switch to using dh and dh-python. This package might be better RMed though

To post a comment you must log in.
Revision history for this message
Luke Faraone (lfaraone) wrote :

We should probably check with ~superm1 as to whether this package is still useful, agreed.

review: Needs Information
Revision history for this message
Piotr Ożarowski (piotr) wrote :

is missing --buildsystem=pybuild in debian/rules intentional?

Revision history for this message
Jackson Doak (noskcaj) wrote :

Is pybuild needed?

On Wednesday, August 12, 2015, Piotr Ożarowski <email address hidden> wrote:

> is missing --buildsystem=pybuild in debian/rules intentional?
>
> --
>
> https://code.launchpad.net/~noskcaj/ubuntu/wily/dell-dup/dh-python/+merge/264080
> You are the owner of lp:~noskcaj/ubuntu/wily/dell-dup/dh-python.
>

Revision history for this message
Mario Limonciello (superm1) wrote :

jared is probably better to review if this is still relevant. Adding him.

review: Abstain
Revision history for this message
Jared Dominguez (jared-dominguez) wrote :

I'll need to check on whether this is needed at all anymore.

Revision history for this message
Jared Dominguez (jared-dominguez) wrote :

Changes in this package are fine with me, but the internal feedback I got is that this package is no longer in use by Dell for building DUPs.

review: Abstain
Revision history for this message
Michael Terry (mterry) wrote :

Sounds like this can be removed from the archive. I've filed bug 1491140 to do that (assigned to you, Jared to confirm it should go ahead -- if you say yes, we can subscribe ~ubuntu-archive).

In the meantime, I'll mark this branch as WIP to get it off the sponsors list.

Thanks, Jackson! Even though we may not use it, good patch. :)

Unmerged revisions

8. By Jackson Doak

Depend on firmware-tools, firmware-addon-dell

7. By Jackson Doak

Add missing build-depends

6. By Jackson Doak

* Switch to dh and dh-python
* Bump standards-version

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2009-08-27 11:56:18 +0000
3+++ debian/changelog 2015-07-07 22:38:42 +0000
4@@ -1,3 +1,11 @@
5+dell-dup (1.1.3-0ubuntu3) wily; urgency=medium
6+
7+ * Switch to dh and dh-python
8+ * Bump standards-version
9+ * Depend on firmware-tools, firmware-addon-dell
10+
11+ -- Jackson Doak <noskcaj@ubuntu.com> Wed, 08 Jul 2015 08:14:21 +1000
12+
13 dell-dup (1.1.3-0ubuntu2) karmic; urgency=low
14
15 * Adjust debian/rules to install all Python modules into the right place,
16
17=== modified file 'debian/compat'
18--- debian/compat 2008-02-12 14:42:54 +0000
19+++ debian/compat 2015-07-07 22:38:42 +0000
20@@ -1,1 +1,1 @@
21-5
22+9
23
24=== modified file 'debian/control'
25--- debian/control 2009-08-27 11:56:18 +0000
26+++ debian/control 2015-07-07 22:38:42 +0000
27@@ -3,15 +3,23 @@
28 Priority: optional
29 Maintainer: Ubuntu Developers <ubuntu-devel-discuss@lists.ubuntu.com>
30 XSBC-Original-Maintainer: Michael Brown <Michael_E_Brown@dell.com>
31-Build-Depends: debhelper (>= 5.0.38), cdbs (>= 0.4.49)
32-Build-Depends-Indep: python-support (>= 0.5.3)
33-Standards-Version: 3.8.3
34+Build-Depends:
35+ debhelper (>= 9),
36+ firmware-addon-dell,
37+ firmware-tools,
38+ python:any
39+Standards-Version: 3.9.6
40 Homepage: http://linux.dell.com/firmware-tools
41
42 Package: dell-dup
43 Architecture: all
44-Depends: ${python:Depends}, ${misc:Depends}
45-Provides: ${python:Provides}
46+Depends:
47+ firmware-addon-dell,
48+ firmware-tools,
49+ ${misc:Depends},
50+ ${python:Depends}
51+Provides:
52+ ${python:Provides}
53 Description: A firmware-tools plugin for Dell DUP images
54 Firmware archives can be distributed in a variety of formats.
55 This package provides a plugin for firmware tools to extract
56
57=== modified file 'debian/copyright'
58--- debian/copyright 2009-08-27 11:56:18 +0000
59+++ debian/copyright 2015-07-07 22:38:42 +0000
60@@ -63,7 +63,7 @@
61
62 4. to perform the Original Work publicly; and
63
64- 5. to display the Original Work publicly.
65+ 5. to display the Original Work publicly.
66
67 2. Grant of Patent License. Licensor grants You a worldwide,
68 royalty-free, non-exclusive, sublicensable license, under patent
69
70=== removed file 'debian/pycompat'
71--- debian/pycompat 2008-02-12 14:42:54 +0000
72+++ debian/pycompat 1970-01-01 00:00:00 +0000
73@@ -1,1 +0,0 @@
74-2
75
76=== removed file 'debian/pyversions'
77--- debian/pyversions 2008-02-12 14:42:54 +0000
78+++ debian/pyversions 1970-01-01 00:00:00 +0000
79@@ -1,1 +0,0 @@
80-2.2-
81
82=== modified file 'debian/rules'
83--- debian/rules 2009-08-27 11:56:18 +0000
84+++ debian/rules 2015-07-07 22:38:42 +0000
85@@ -1,28 +1,11 @@
86 #!/usr/bin/make -f
87-# -*- makefile -*-
88-# vim:noet:ts=8:sw=8:filetype=make
89-
90-DEB_PYTHON_SYSTEM=pysupport
91-include /usr/share/cdbs/1/rules/debhelper.mk
92-include /usr/share/cdbs/1/class/autotools.mk
93-include /usr/share/cdbs/1/rules/utils.mk
94-
95-DEB_INSTALL_CHANGELOGS_ALL := ChangeLog
96-
97-TARFILE+=$(DEB_SOURCE_PACKAGE)_$(DEB_UPSTREAM_VERSION).orig.tar.gz
98-URL+=http://linux.dell.com/git/dell-dup.git/
99-
100-get-orig-source:: git-export build-source
101-git-export::
102- git clone $(URL) $(DEB_SOURCE_PACKAGE)-$(DEB_UPSTREAM_VERSION)
103-build-source::
104- cd $(DEB_SOURCE_PACKAGE)-$(DEB_UPSTREAM_VERSION) && sh autogen.sh
105- cd $(DEB_SOURCE_PACKAGE)-$(DEB_UPSTREAM_VERSION) && rm -rf autom4te.cache
106- tar czf $(CURDIR)/../$(TARFILE) $(DEB_SOURCE_PACKAGE)-$(DEB_UPSTREAM_VERSION) --exclude=.git
107- rm -rf $(DEB_SOURCE_PACKAGE)-$(DEB_UPSTREAM_VERSION)
108-
109-DEB_INSTALL_DOCS_ALL :=
110-
111-install/dell-dup::
112+
113+%:
114+ dh $@ --with python2
115+
116+override_dh_installchangelogs:
117+ dh_installchangelogs ChangeLog
118+
119+override_dh_install:
120 find ./ -name *.pyc | xargs rm
121- dh_pysupport -pdell-dup
122+ dh_install
123
124=== added directory 'debian/source'
125=== added file 'debian/source/format'
126--- debian/source/format 1970-01-01 00:00:00 +0000
127+++ debian/source/format 2015-07-07 22:38:42 +0000
128@@ -0,0 +1,1 @@
129+3.0 (quilt)
130\ No newline at end of file

Subscribers

People subscribed via source and target branches