Details about package aiolimiter
Package uploads
Upload #2
Information
Changelog
aiolimiter (1.1.0-1) unstable; urgency=low
.
* Initial release, autogenerated by py2dsp/3.20230219; Closes: #1080255
Comments
-
Yuri,
Preamble...
Thank you for taking the time to prepare this package and your contribution to the Debian project.
The review below is for assistance. This review is offered to help package submitters to Debian mentors inorder to improve their packages prior to possible sponsorship into Debian. There is no obligation on behalf of the submitter to make any alterations based upon information provided in the review.
Review...
1. Build:
* pbuilder [1]: Good
* sbuild [2]: Good
2. Lintian [3]: Good
3. Licenses [4]: Good
4. Watch file [uscan --force-download]: Good
5. Build Twice [sudo pbuilder build --twice <package>.dsc]: Good
6. Reproducible builds [5]: Good
7. Install [No previous installs]: Good
8. Upgrade [Over previous installs if any]: N/A
Summary...
I believe aiolimiter is ready for review/possible sponsorship. Could a Debian Developer (DD) with available free time, please review this package and upload if you feel it is ready and appropriate for the distribution.
P.S. If you filed an RFS bug, I could mark this package as "confirmed" with other packages from this suite, to make it easier for a sponsoring DD.
Regards
Phil
[1] pbuilder:
* Command: sudo pbuilder build <PACKAGE>.dsc
* Document: https://wiki.ubuntu.com/PbuilderHowto.
* Document: https://wiki.debian.org/PbuilderTricks
[2] sbuild:
* Command: sbuild <PACKAGE>.dsc
* Document: https://wiki.kathenas.org/pmwiki.php/Kathenas/Article00000002
* Document: https://wiki.debian.org/sbuild
[3] lintian:
* Command: lintian -v -i -I -E --pedantic --profile debian (*.dsc, *.changes, *.buildinfo). Each can throw up different results, so be thorough.
* Document: https://wiki.debian.org/Lintian
[4] lrc:
* Command: lrc
* Document: https://wiki.debian.org/CopyrightReviewTools#licenserecon
[5] reprotest
* Command: sudo reprotest --vary=-build_path,domain_host.use_sudo=1 --auto-build <PACKAGE>.dsc -- schroot unstable-amd64-sbuild
* Document: https://wiki.kathenas.org/pmwiki.php/Kathenas/Article00000004
* Document: https://wiki.debian.org/ReproducibleBuilds/
* Document: https://wiki.debian.org/ReproducibleBuilds/Howto#Newer_method
Ready
Phil Wyett at Sept. 7, 2024, 10:43 a.m.
-
Thanks for review! I just filled RFS issue: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1081042
Yuri Konotopov at Sept. 7, 2024, 1:06 p.m.
Upload #1
Information
Changelog
aiolimiter (1.1.0-1) unstable; urgency=low
.
* Initial release, autogenerated by py2dsp/3.20230219; Closes: #1080255
Comments
-
Running lintian...
N:
E: python3-aiolimiter: extended-description-is-empty
N:
N: The extended description (the lines after the first line of the
N: "Description:" field) is empty.
N:
N: Please refer to The description of a package (Section 3.4) in the Debian
N: Policy Manual for details.
N:
N: Visibility: error
N: Show-Always: no
N: Check: fields/description
N:
N:
E: python3-aiolimiter: unknown-file-in-python-module-directory [usr/lib/python3/dist-packages/CHANGELOG.md]
N:
N: This package installs the specified "non-Python" file in the top-level of
N: a Python library directory.
N:
N: This was either a mistake and/or will likely to cause conflicts with other
N: packages.
N:
N: Visibility: error
N: Show-Always: no
N: Check: languages/python
N:
N:
I: aiolimiter source: missing-vcs-browser-field Vcs-Git https://gitlab.com/apt-mirror2/aiofile-debian-packaging
N:
N: A Vcs-* field in this package is pointing to a repository that supports
N: browsing of the repository via a web browser.
N:
N: This is typically a nicer user-experience for developers and avoids
N: unnecessary and time-consuming clones of the repository.
N:
N: Please add a suitable Vcs-Browser field to the package.
N:
N: Visibility: info
N: Show-Always: no
N: Check: fields/vcs
N:
N:
I: aiolimiter source: out-of-date-standards-version 4.6.2.0 (released 2022-12-17) (current is 4.7.0)
N:
N: The source package refers to a Standards-Version older than the one that
N: was current at the time the package was created (according to the
N: timestamp of the latest debian/changelog entry). Please consider updating
N: the package to current Policy and setting this control field
N: appropriately.
N:
N: If the package is already compliant with the current standards, you don't
N: have to re-upload the package just to adjust the Standards-Version control
N: field. However, please remember to update this field next time you upload
N: the package.
N:
N: See /usr/share/doc/debian-policy/upgrading-checklist.txt.gz in the
N: debian-policy package for a summary of changes in newer versions of
N: Policy.
N:
N: Please refer to
N: https://www.debian.org/doc/debian-policy/upgrading-checklist.html for
N: details.
N:
N: Visibility: info
N: Show-Always: no
N: Check: fields/standards-version
N:
N:
I: python3-aiolimiter: package-contains-documentation-outside-usr-share-doc [usr/lib/python3/dist-packages/CHANGELOG.md]
N:
N: This package ships a documentation file outside /usr/share/doc
N: Documentation files are normally installed inside /usr/share/doc.
N:
N: If this file doesn't describe the contents or purpose of the directory it
N: is in, please consider moving this file to /usr/share/doc/ or maybe even
N: removing it. If this file does describe the contents or purpose of the
N: directory it is in, please add a lintian override.
N:
N: Visibility: info
N: Show-Always: no
N: Check: documentation
N:
N: Screen: python/egg/metadata
N: Advocates: "Scott Kitterman" <debian@kitterman.com>
N: Reason: The folders XXX.dist-info/ and XXX.egg-info/ hold metadata for
N: Python modules. Those files are not documentation even though
N: some of their names carry the .txt file extension.
N:
N: Python modules can be both public and private.
N:
N: Read more in
N: https://www.python.org/dev/peps/pep-0427/#the-dist-info-directory,
N: https://www.python.org/dev/peps/pep-0376/#id16,
N: https://www.python.org/dev/peps/pep-0610/,
N: https://www.python.org/dev/peps/pep-0639/,
N: https://setuptools.pypa.io/en/latest/deprecated/python_eggs.html,
N: and Bug#1003913.
N:
N:
X: aiolimiter source: upstream-metadata-file-is-missing
N:
N: This source package is not Debian-native but it does not have a
N: debian/upstream/metadata file.
N:
N: The Upstream MEtadata GAthered with YAml (UMEGAYA) project is an effort to
N: collect meta-information about upstream projects from any source package.
N: This file is in YAML format and it is used in to feed the data in the
N: UltimateDebianDatabase. For example, it can contains the way the authors
N: want their software be cited in publications and some bibliographic
N: references about the software.
N:
N: Please add a debian/upstream/metadata file.
N:
N: Please refer to https://dep-team.pages.debian.net/deps/dep12/ and
N: https://wiki.debian.org/UpstreamMetadata for details.
N:
N: Visibility: pedantic
N: Show-Always: no
N: Check: debian/upstream/metadata
N: This tag is experimental.
N:
E: Lintian run failed (runtime error)
Needs work
Phil Wyett at Sept. 2, 2024, 1:40 p.m.
-
Thanks for your review! I uploaded new version that should fix those issues
Yuri Konotopov at Sept. 6, 2024, 7:23 p.m.
