Sign me up Login

Details about package python-opower

Name:	python-opower
Uploader:	Sourav Bhattacharya <souravbhattacharya@tutamail.com> (Debian QA page)
Description:	python3-opower - Python library to get usage/cost from utilities

Package uploads

Upload #2

Information

Version:	0.8.3-1
Uploaded:	2024-10-13 06:25
Source package:	python-opower_0.8.3-1.dsc
Distribution:	unstable
Section:	python
Priority:	optional
Homepage:	https://github.com/tronikos/opower
Vcs-Browser:	https://salsa.debian.org/sxctic-54l/python-opower
Vcs-Git:	https://salsa.debian.org/sxctic-54l/python-opower.git
Closes bugs:	#1084967

Changelog

 python-opower (0.8.3-1) unstable; urgency=medium
 .
   * Initial release. (Closes: #1084967)

QA information

–
Package uses debhelper-compat

Debhelper compatibility level 13

–

Package is the latest upstream version

Local:	0.8.3
Upstream:	0.8.3
Url:	https://github.com/tronikos/opower/archive/refs/tags/v0.8.3.tar.gz

–
Package is not native

Format: 3.0 (quilt)
–
The uploader is in the package's "Uploaders" field
–
Package has lintian informational tags
python-opower source
- X debian-watch-does-not-check-openpgp-signature
  - [debian/watch]
- X prefer-uscan-symlink
  - filenamemangle s/.+\/v?(\d\S+)\.tar\.gz/opower-$1\.tar\.gz/ [debian/watch:3]
- X very-long-line-length-in-source-file
  - 10157 > 512 [tests/opower/utilities/smud/smudOkta.html:61]
  - 1638 > 512 [tests/opower/utilities/smud/myAccountResponse.html:338]
- O source-is-missing
  - [tests/opower/utilities/smud/myAccountResponse.html] (override comment: Files myAccountResponse.html and smudOkta.html serve no purpose other than test data)
  - [tests/opower/utilities/smud/smudOkta.html] (override comment: Files myAccountResponse.html and smudOkta.html serve no purpose other than test data)
python3-opower
- I package-contains-documentation-outside-usr-share-doc
  - [usr/lib/python3/dist-packages/opower-0.8.3.dist-info/top_level.txt]
–
Package closes ITP bug
- python-opower:
  - #1084967 (Wishlist, ITP): ITP: python-opower -- Python library to get usage/cost from utilities
–
Package is not in Debian
–
d/copyright is in DEP5 format

Upstream Contact: tronikos

Licenses: Apache-2.0

Comments

The lintian override needs to be specific to the html files. The current one is overly broad and risks hiding problems elsewhere detected by future lintian versions or upstream releases.

For an example, see https://salsa.debian.org/jcfp/puremagic/-/blob/ac6ecff7e2e17cdbd00d2538ce984023b150a994/debian/source/lintian-overrides

As you can see, the asterisks in there can be used for globbing. The tricky part is balancing that in a way prevents trivial upstream changes (e.g. the cause of the lintian hit moving to a different line) from invalidating the override, whilst also avoiding making the override so broad that it ends up hiding problems elsewhere (things like keeping filenames in an override is often a good idea).

Needs work jcfp at Oct. 19, 2024, 4:24 p.m.

Upload #1

Information

Version:	0.8.3-1
Uploaded:	2024-10-12 13:10
Source package:	python-opower_0.8.3-1.dsc
Distribution:	unstable
Section:	python
Priority:	optional
Homepage:	https://github.com/tronikos/opower
Vcs-Browser:	https://salsa.debian.org/sxctic-54l/python-opower
Vcs-Git:	https://salsa.debian.org/sxctic-54l/python-opower.git
Closes bugs:	#1084967

Changelog

 python-opower (0.8.3-1) unstable; urgency=medium
 .
   * Initial release. (Closes: #1084967)

QA information

–
Package uses debhelper-compat

Debhelper compatibility level 13

–

Package is the latest upstream version

Local:	0.8.3
Upstream:	0.8.3
Url:	https://github.com/tronikos/opower/archive/refs/tags/v0.8.3.tar.gz

–
Package is not native

Format: 3.0 (quilt)
–
The uploader is in the package's "Uploaders" field
–
Package has lintian informational tags
python-opower source
- X debian-watch-does-not-check-openpgp-signature
  - [debian/watch]
- X prefer-uscan-symlink
  - filenamemangle s/.+\/v?(\d\S+)\.tar\.gz/opower-$1\.tar\.gz/ [debian/watch:3]
- X very-long-line-length-in-source-file
  - 10157 > 512 [debian/missing-sources/smudOkta.html:61]
  - 10157 > 512 [tests/opower/utilities/smud/smudOkta.html:61]
  - 1638 > 512 [debian/missing-sources/myAccountResponse.html:338]
  - 1638 > 512 [tests/opower/utilities/smud/myAccountResponse.html:338]
python3-opower
- I package-contains-documentation-outside-usr-share-doc
  - [usr/lib/python3/dist-packages/opower-0.8.3.dist-info/top_level.txt]
–
Package closes ITP bug
- python-opower:
  - #1084967 (Wishlist, ITP): ITP: python-opower -- Python library to get usage/cost from utilities
–
Package is not in Debian
–
d/copyright is in DEP5 format

Upstream Contact: tronikos

Licenses: Apache-2.0

Comments

I see you're a quick learner! really nice work, I just have one suggestion and found one real issue (apart from the team-related stuff you are already aware of):

* control: in the long description, the standard way for lists seems to use asterisks (see the "codelite" package for an example).

* d/missing-sources lists two HTML files that are identical to the files found under tests/opower/utilities/smud/. Storing an exact copy of files with (supposedly) missing sources obviously isn't the idea behind this directory; rather, it's meant for the actual missing source code. That said, in this particular case the files aren't part of the program's code but only used as test data. In that context, there's nothing missing. Please remove the d/missing-sources directory; if you added this because of a lintian hit, override it with a comment indicating the two files serve no other purpose than test data.

Needs work jcfp at Oct. 12, 2024, 2:51 p.m.

Upstream Contact:	tronikos
Licenses:	Apache-2.0

Upstream Contact:	tronikos
Licenses:	Apache-2.0