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

Tomas Orsava <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]



--- Comment #4 from Tomas Orsava <[email protected]> ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #3)
> The name should have a dash, not an underscore.
> https://docs.fedoraproject.org/en-US/packaging-guidelines/Python/#_naming
> doesn't say this very clearly, but it says that the package name should use
> the canonical name, and the canonical name is described as the project name
> with
> some canonicalization, and the project name is described as the name used on
> pypi. And on pypi we have a dash: https://pypi.org/project/netapp-ontap/

I agree that the name change is required. From the old Python Fedora packaging
guidelines [0]:

> The canonical name is obtained by switching the standardized name to lower 
> case and converting all runs of non-alphanumeric characters to single “-” 
> characters. Example: “The $$$ Tree” becomes “the-tree”.

[0]
https://docs.fedoraproject.org/en-US/packaging-guidelines/Python_201x/#_automatic_provides_with_a_standardized_name

I agree that it's not well described in the new guidelines, I'll try to get
that fixed.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2139873
_______________________________________________
package-review mailing list -- [email protected]
To unsubscribe send an email to [email protected]
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/[email protected]
Do not reply to spam, report it: 
https://pagure.io/fedora-infrastructure/new_issue

Reply via email to