[Bug 1901000] Review Request: python-graphql-server - GraphQL Server tools for powering your server

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=1901000



--- Comment #2 from Mattia Verga <mattia.verga@xxxxxxxxxxxxxx> ---
(In reply to Andy Mender from comment #1)
> I noticed that the extra subpackages use "+" as a separator. Shouldn't they
> use "-" instead or are these flavors of the -server package?
> 
That's the way Fedora has chosen to name the extras provided by the package.
Everything is made automatically by the macro:
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_python_extras

> > Name:           python-%{srcname}
> > Version:        3.0.0
> > Release:        1.b2%{?dist}
> 
> The "b2" part should be in the Version field, I think. It's not a part of
> the Fedora package Release tag, but rather of the version string of the
> upstream package.
> 
No, 3.0.0b2 does not sort properly, since 3.0.0 will be lower:
https://docs.fedoraproject.org/en-US/packaging-guidelines/Versioning/#_unsortable_versions
'b2' must go in <extraver>.

> Upstream provides tests in the PyPi tarball. Can these be added to the SPEC
> file under %check or do they require external binaries, like a server daemon?
> 
Can't run tests, since sanic package isn't available

> The rest looks okay. Package approved.
Thank you!


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux