Bug 1680187
Summary: | Add Python EPEL 7 branch for snowballstemmer | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Scott K Logan <logans> | ||||||||
Component: | python-snowballstemmer | Assignee: | Julien Enselme <jujens> | ||||||||
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||||
Severity: | unspecified | Docs Contact: | |||||||||
Priority: | unspecified | ||||||||||
Version: | rawhide | CC: | jujens | ||||||||
Target Milestone: | --- | ||||||||||
Target Release: | --- | ||||||||||
Hardware: | All | ||||||||||
OS: | Linux | ||||||||||
Whiteboard: | |||||||||||
Fixed In Version: | python-snowballstemmer-1.2.1-9.el7 | Doc Type: | If docs needed, set a value | ||||||||
Doc Text: | Story Points: | --- | |||||||||
Clone Of: | Environment: | ||||||||||
Last Closed: | 2019-03-12 21:08:03 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: | |||||||||||
Attachments: |
|
I asked for the branch creation https://pagure.io/releng/fedora-scm-requests/issue/9930. I'll track this and push the package once it is done. I have an issue with your patch. When I try to build it, I get this error:
warning: line 96: Possible unexpanded macro in: %package -n python%{python3_other_pkgversion}-snowballstemmer
error: line 99: Unknown tag: %python_provide: ERROR: python%{python3_other_pkgversion}-snowballstemmer not recognized.
error: query of specfile /home/jenselme/fedora-scm/python-snowballstemmer/python-snowballstemmer.spec failed, can't parse
And this is the first time I see python3_other_pkgversion. Perhaps it should be removed or only present on the EPEL7 branch?
> If you wouldn't like to maintain an EPEL branch yourself, please assign maintainership to me.
I'll gladly given you maintainership on this package. Can you give me your pagure username so I can add you?
Created attachment 1537984 [details]
Updated patch to parse on Fedora
Created attachment 1537985 [details]
Patch for 'master' to build for EPEL 7
Thanks for engaging me on this, I really appreciate it.
The spec errors are my fault. The patch doesn't parse on Fedora because python_provide gets grumpy when python3_other_pkgversion isn't defined. It is on EPEL 7, so the patch would likely work, but it is super inconvenient not to be able to work with the spec on Fedora. I added an updated patch.
My FAS/Pagure username is `cottsay`.
python-snowballstemmer-1.2.1-9.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2019-b38dc90ff5 I just pushed the package to EPEL. I also added you as an admin for this package. Thanks Julien, I really appreciate it. python-snowballstemmer-1.2.1-9.el7 has been pushed to the Fedora EPEL 7 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-EPEL-2019-b38dc90ff5 python-snowballstemmer-1.2.1-9.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report. |
Created attachment 1537652 [details] Patch for 'master' to build for EPEL 7 Description of problem: Despite the prerequisites being met to build 'snowballstemmer' on EPEL 7, the branch does not exist and the package is not built. This package is used by other packages which are desired in EPEL, and is therefore blocking their packaging efforts. I have attached a patch which can be applied on top of `master`, and makes the package buildable on EPEL 7. Please create an epel7 branch [1] from `master` and apply that patch onto it. If you wouldn't like to maintain an EPEL branch yourself, please assign maintainership to me. Thanks!