Spec URL: https://davdunc.fedorapeople.org/awscli-2-rpms/awscli-2.spec SRPM URL: https://davdunc.fedorapeople.org/awscli-2-rpms/awscli-2-2.4.12-2.fc35.src.rpm Description: This package provides version 2 of the unified command line interface to Amazon Web Services. Fedora Account System Username: davdunc
This depends on awscrt, marked it as a blocker. Meanwhile: - why rename this to awscli-2 ? If the intention is that both are parallel installable, this should not obsolete the other. If not, then you can just call this awscli (and actually, don't need a review request) - missing %license LICENSE.txt - with auto build requirement, you only need python3-devel. Drop toml and wheel, they should be automatically picked up - drop python_provide, it's deprecated: https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_deprecated_macros - there are tests, you probably want a %check section. If invoking %tox is too much of a work, invoking %pytest should work. See https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_tests
(In reply to Michel Alexandre Salim from comment #1) > This depends on awscrt, marked it as a blocker. Meanwhile: > > - why rename this to awscli-2 ? If the intention is that both are parallel > installable, this should not obsolete the other. If not, then you can just > call this awscli (and actually, don't need a review request) it's a separate package. We have to maintain the original awscli package because it is in RHEL and needs to stay consistent for the foreseeable future. We will need to maintain the two tracks because they function in very different ways. > - missing %license LICENSE.txt whoops! > - with auto build requirement, you only need python3-devel. Drop toml and > wheel, they should be automatically picked up > - drop python_provide, it's deprecated: > https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/ > #_deprecated_macros Thank you. some careless copying of old process. > - there are tests, you probably want a %check section. If invoking %tox is > too much of a work, invoking %pytest should work. See > https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_tests I think kyle can figure this out for me. Thanks.
This is an automatic action taken by review-stats script. The ticket submitter failed to clear the NEEDINFO flag in a month. As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews we consider this ticket as DEADREVIEW and proceed to close it.
This not stalled...
Here are some more important issues: > Name: %{appname}-2 I don't think this is a valid name. Packaging Guidelines say about packages with the same base name [1]: If the base package name does not end with a digit, the version MUST be directly appended to the package name with no intervening separator. > License: ASL 2.0 and MIT This should be: License: Apache-2.0 AND MIT You should also explicitly state in a comment which source files are under MIT (since majority of them are under Apache-2.0). > Patch0: awscli-2.patch Existence of this patch should be justified. The package doesn't build. It does build after updating to the latest upstream version (2.11.6) and removing the downstream patch and version cap of prompt-toolkit, but the test suite ends with: = 96 failed, 64530 passed, 4 skipped, 3 warnings, 97 errors in 518.31s (0:08:38) = [1] https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#multiple
Closing this in favor of bug #2189420. *** This bug has been marked as a duplicate of bug 2189420 ***