[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#675388: RFS: python-cups/1.9.61-0.1 [RC] [NMU]



(I don't intend to sponsor this NMU.)

* Rob Adams <radams@artlogic.com>, 2012-05-31, 11:53:
dget -x http://mentors.debian.net/debian/pool/main/p/python-cups/python-cups_1.9.61-0.1.dsc

[...]

Changes since the last upload:

* Non-maintainer upload.
* New upstream release. (Resolves: #656640)

Did you mean s/Resolved/Closes/?

* debian/control:
 - bump standard-version to 3.9.3

Were any packaging changes needed to do this? Anyway, this not appropriate for any NMU.

* debian/compat:
 - bump to 9

Were any packaging changes needed to do this? Again, not appropriate for an NMU.

* debian/rules:
 - switch from python-support to dh_python2

We don't do things like this in an NMU.

What is this:

| binary-*:
| 	dh_python2

(in debian/rules) supposed to do?

You added "X-Python-Version: >= 2.6", presumably because the new upstream version doesn't support with 2.5 anymore. Do you know why?

You forgot to bump B-D on python-all-dev to >= 2.6.6-3~ (for dh_python2 support).

* debian/source.lintian-override:
 - package-needs-versioned-debhelper-build-depends 9.

No, lintian is correct. Fix the bug instead.


Other changes you made that are not documented in the changelog:
- added a trailing comma Uploaders (?!);
- changed package description;
- changed debian/copyright;
- removed a patch;
- added DEB_BUILD_HARDENING=1 and DEB_BUILD_HARDENING_STACKPROTECTOR=1 in debian/rules (?!); - prepended -fstack-protector to CFLAGS (shouldn't you use dpkg-buildflags to acquire CFLAGS instead?). Why "[RC]" in the bug title? As far as I can see the only bug this package fixes has severity normal.

--
Jakub Wilk



Reply to: