Bug 1357690
| Summary: | python3 supackage | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | dima <pronix.service> | ||||||||
| Component: | python-okaara | Assignee: | Jay Dobies <jason.dobies> | ||||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||||
| Severity: | unspecified | Docs Contact: | |||||||||
| Priority: | unspecified | ||||||||||
| Version: | 25 | CC: | awood, cstratak, ishcherb, jason.dobies, lbalhar | ||||||||
| Target Milestone: | --- | ||||||||||
| Target Release: | --- | ||||||||||
| Hardware: | Unspecified | ||||||||||
| OS: | Unspecified | ||||||||||
| Whiteboard: | |||||||||||
| Fixed In Version: | python-okaara-1.0.37-1.fc26 | Doc Type: | If docs needed, set a value | ||||||||
| Doc Text: | Story Points: | --- | |||||||||
| Clone Of: | Environment: | ||||||||||
| Last Closed: | 2017-04-01 17:08:08 UTC | Type: | Bug | ||||||||
| Regression: | --- | Mount Type: | --- | ||||||||
| Documentation: | --- | CRM: | |||||||||
| Verified Versions: | Category: | --- | |||||||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||||||
| Embargoed: | |||||||||||
| Bug Depends On: | |||||||||||
| Bug Blocks: | 1285816, 1312032 | ||||||||||
| Attachments: |
|
||||||||||
|
Description
dima
2016-07-18 22:07:53 UTC
This bug appears to have been reported against 'rawhide' during the Fedora 25 development cycle. Changing version to '25'. Hello. There are some issues in this patch which I mentioned before in previous reviews (missing description in subpackages, release number back to one, global sum seems to be bad, missing changelog ...). Could you please review your patch and rebase it to the latest version of specfile? Also, link to Koji scratch build would be perfect. Have a nice day. Created attachment 1186161 [details]
py3 subpackage, add license
update specfile after review
Hello.
Thank you for the new patch. Unfortunately, there are still some issues.
- Is better to use global macro for name of module and then %{modname} instead of "okaara" in specfile. Also, you have a bug which breaks build - you should change %{name} to something like %{modname}-%{name}-%{version}-%{rel} in %setup.
- You changed version so release number should be changed back to 1.
- Please improve summary to be more specific.
Also, are you sure that this package supports Python3 in upstream? When I fixed all of specfile issues and tried to build this package, build process ended with a lot of SytaxErrors. Fox example:
running install_scripts
File "/usr/lib/python3.5/site-packages/okaara/prompt.py", line 150
except (EOFError, KeyboardInterrupt), e:
Have a nice day.
I am currently working on my own PR for python3 compatibility. I'm gonna link it here when it will be prepared and attach patch for specfile. Then we will have to wait until PR will be merged. Here is my PR for upstream Python 3 compatibility: https://github.com/jdob/okaara/pull/14 I have also prepared a new version of specfile, but it would be better to test it with new release before I attach it here. My PR is merged so now we can wait for a while until maintainer makes new release. If it will take too much time, we can base next version of the package on the latest commit. 1.0.37 version is released with the improved Python 3 support. This is waiting for the GitHub release, as the PyPI tarball does not include the .po files. I've created new issue in upstream: https://github.com/jdob/okaara/issues/17 Created attachment 1263013 [details] Patch to provide Python 3 package Hello, please find a patch attached to update the package to version 1.0.37 and provide a Python 3 subpackage. I have incorporated the changes from the previous patch and added a switch to build Python 3 subpackage only on Fedora. Please review and apply. Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=18382009 python-okaara-1.0.37-1.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-6d64ecb923 python-okaara-1.0.37-1.fc26 has been pushed to the Fedora 26 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-6d64ecb923 python-okaara-1.0.37-1.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report. |