Bug 1298644 - Please provide python3-slugify
Summary: Please provide python3-slugify
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: python-slugify
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Eduardo Mayorga
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Keywords:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-01-14 15:52 UTC by Paul Belanger
Modified: 2016-01-18 01:48 UTC (History)
2 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2016-01-18 01:32:56 UTC


Attachments (Terms of Use)

Description Paul Belanger 2016-01-14 15:52:25 UTC
python-slugify support python3, as such we should also ship python3 support. Please enable it for the RPM using the following[1]

Thanks,
PB

[1] https://fedoraproject.org/wiki/Packaging:Python

Comment 1 Paul Belanger 2016-01-14 16:13:05 UTC
Ugh, this RPM is not https://pypi.python.org/pypi/python-slugify. Hmm, is there a specific reason you didn't use this version, which is more mature then the one you packaged?

Comment 2 William Moreno 2016-01-14 16:26:21 UTC
python-frappe requires the older slugify to run.

https://github.com/frappe/frappe/blob/develop/requirements.txt

And for the momment frappe developers do not have plans to move to python3.

I will ask to the packaging mailing list about this to libs packaging.

Comment 3 Paul Belanger 2016-01-14 16:32:47 UTC
Thanks for the fast reply, I also sent you a private email, which just extends some of the topic here. Just a heads up.

I will look into python-frappe and see how they are using it.

Comment 4 William Moreno 2016-01-14 17:34:46 UTC
For the record:

https://lists.fedoraproject.org/archives/list/packaging%40lists.fedoraproject.org/thread/Z35EMXP5SCKJRR2IW2OKZ3QGWNZVNYPL/

https://github.com/frappe/frappe/pull/1521

I will wait some comment upstream but looks like I can retire python-slugify is frappe do not need any more.

Comment 5 Fedora Admin XMLRPC Client 2016-01-17 05:09:45 UTC
This package has changed ownership in the Fedora Package Database.  Reassigning to the new owner of this component.

Comment 6 Eduardo Mayorga 2016-01-17 16:07:28 UTC
Note that the slugify in PyPI is not the same than python-slugify.

slugify: https://pypi.python.org/packages/source/s/slugify/slugify-0.0.1.tar.gz
python-slugify: https://github.com/un33k/python-slugify/archive/0.0.1.tar.gz

These are not the same source, so I cannot update the old slugify. It would make sense to retire this package in Rawhide as nothing requires it and make room for the new python-slugify.

Another solution would be to package the new PyPI's python-slugify as python-python-slugify (a legit name as per our Package Naming Guidelines), but that would only make things confusing and there is no need to keep the old slugify.

Comment 7 Zbigniew Jędrzejewski-Szmek 2016-01-17 16:58:19 UTC
Are they more or less equivalent? If yes, I'd just switch to the other sources after reviewing them for license compatibility. After all, if this one is retired and a package with an identical name is added, for the user it's exactly the same as if the sources were switched. For the packagers it's less work.

Comment 8 Eduardo Mayorga 2016-01-18 01:32:56 UTC
Building in Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=12588750

Comment 9 Eduardo Mayorga 2016-01-18 01:48:31 UTC
Sorry, this is the actual build: http://koji.fedoraproject.org/koji/taskinfo?taskID=12588760


Note You need to log in before you can comment on or make changes to this bug.