Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1282099 - Python3 port of python-django-admin-honeypot
Summary: Python3 port of python-django-admin-honeypot
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: python-django-admin-honeypot
Version: 24
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Eduardo Echeverria
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: PYTHON3 PY3PATCH-PUSH
TreeView+ depends on / blocked
 
Reported: 2015-11-14 22:27 UTC by Adolfo Fitoria
Modified: 2023-09-12 00:51 UTC (History)
7 users (show)

Fixed In Version: python-django-admin-honeypot-1.0.0-1.fc26
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2016-09-07 10:51:25 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Update to 1.0.0 including tests and Python 3 subpackage (4.53 KB, patch)
2016-08-05 14:26 UTC, Jan Beran
no flags Details | Diff
Updated spec file (4.70 KB, patch)
2016-08-17 15:33 UTC, Jan Beran
no flags Details | Diff

Description Adolfo Fitoria 2015-11-14 22:27:15 UTC
Python3 version of python-progressbar is available at pypi 

https://pypi.python.org/pypi/django-admin-honeypot

Please add python3 support.

Thanks.

Comment 1 Petr Viktorin 2016-02-01 12:29:19 UTC
Here's a more fleshed out report:

Upstream, this software supports Python 3. Please provide a Python 3
package for Fedora.


According to the Python packaging guidelines [0], software must be
packaged for Python 3 if upstream supports it.
The guidelines give detailed information on how to do this, and even
provide an example spec file [1].

The current best practice is to provide subpackages for the two Python
versions (called "Common SRPM" in the guidelines). Alternatively, if
nothing depends on your Python2 package, you can just switch to Python 3
entirely.

It's fine to do this in Rawhide only.


If anything is unclear, or if you need any kind of assistance with the
porting, you can ask on IRC (#fedora-python on Freenode), or reply here.
We'll be happy to help!


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

Comment 2 Charalampos Stratakis 2016-02-15 15:37:36 UTC
Ping?

No response as of yet from maintainer

Comment 3 Jan Kurik 2016-02-24 13:57:42 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 24 development cycle.
Changing version to '24'.

More information and reason for this action is here:
https://fedoraproject.org/wiki/Fedora_Program_Management/HouseKeeping/Fedora24#Rawhide_Rebase

Comment 4 Petr Viktorin 2016-04-19 13:43:56 UTC
Eduardo, do you need some help with updating, or with porting to Python 3?

Comment 5 Jan Beran 2016-08-05 14:26:17 UTC
Created attachment 1187932 [details]
Update to 1.0.0 including tests and Python 3 subpackage

I have prepared Python 3 patch which includes

- update to version 1.0.0
- source update
- tests are included now
- modernized specfile with Python 3 packaging
- changelog correction to avoid a following error from rpmlint:

$ rpmlint python-django-admin-honeypot.spec
python-django-admin-honeypot.spec: E: specfile-error warning: bogus date in %changelog: Tue Feb 22 2013 Eduardo Echeverria  <echevemaster> - 0.2.4-3
0 packages and 1 specfiles checked; 1 errors, 0 warnings.

The patch follows the recommendation of Matthias (bug #1311552).

Comment 6 Eduardo Echeverria 2016-08-07 05:17:51 UTC
Sorry for  the delay, I'll include your patch. thanks.

Comment 7 Lumír Balhar 2016-08-15 10:51:33 UTC
Hello.

Thank you for new specfile but build fails because there are missing build requires python2-coverage & python2-pytest and python3-coverage & python3-pytest for Python 3 subpackage. Also, more specific python[23]-coverage should be used in test execution command.

Comment 8 Jan Beran 2016-08-17 15:33:48 UTC
Created attachment 1191667 [details]
Updated spec file

Hi Lumír,

thank you for the review. I attach the spec file, updated according to you guidance.

Comment 9 Lumír Balhar 2016-08-18 07:46:54 UTC
Hello.

Thank you very much for the update. Specfile looks perfect to me now.

If maintainer has nothing against, we can push this changes after a week.

Have a nice day.

Comment 10 Lumír Balhar 2016-08-18 07:48:33 UTC
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=15293594

Comment 11 Red Hat Bugzilla 2023-09-12 00:51:38 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 1000 days


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