Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/sgmanager.spec SRPM URL: https://ignatenkobrain.fedorapeople.org/for-review/sgmanager-2.0.0~git+116.a420e38-1.fc30.src.rpm Description: OpenStack Security Groups Management Tool. Fedora Account System Username: ignatenkobrain
Taking this review.
A couple of issues: > Version: 2.0.0~git+%{revnum}.%{shortcommit} This versioning scheme is not permitted currently. Please change to the current scheme or package a current release. > BuildRequires: /usr/bin/flit I'd prefer if it was "%{_bindir}/flit" instead.
> %{?python_enable_dependency_generator} Please move this to the top of the spec to avoid any potential weirdness.
New Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/sgmanager.spec New SRPM URL: https://ignatenkobrain.fedorapeople.org/for-review/sgmanager-2.0.0-0+20180906git+116.a420e38.fc30.src.rpm
Review notes: - Package is named appropriately - Package builds and installs - Follows Fedora packaging guidelines - Follows Python packaging guidelines - Licensing is correct, license file is installed correctly PACKAGE APPROVED.
(fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/sgmanager