| Summary: | tesseract-langpack to tesseract merge made langpacks nonfuctional | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Parag Nemade <pnemade> | ||||||
| Component: | tesseract | Assignee: | Sandro Mani <manisandro> | ||||||
| Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
| Severity: | unspecified | Docs Contact: | |||||||
| Priority: | medium | ||||||||
| Version: | rawhide | CC: | karlikt, manisandro | ||||||
| Target Milestone: | --- | ||||||||
| Target Release: | --- | ||||||||
| Hardware: | Unspecified | ||||||||
| OS: | Unspecified | ||||||||
| Whiteboard: | |||||||||
| Fixed In Version: | tesseract-3.04.00-3.fc23 tesseract-3.04.00-3.el6 tesseract-3.04.00-3.el7 | Doc Type: | Bug Fix | ||||||
| Doc Text: | Story Points: | --- | |||||||
| Clone Of: | Environment: | ||||||||
| Last Closed: | 2016-02-03 20:50:53 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: | |||||||
| Bug Depends On: | |||||||||
| Bug Blocks: | 1314406 | ||||||||
| Attachments: |
|
||||||||
|
Description
Parag Nemade
2016-01-22 16:22:11 UTC
Hi I'm the author of that change. First, couldn't the requested virtual provides just be added manually to the existing approach? I.e. extend the macro to also take the ISO639-1 language code as an argument and add the corresponding virtual provides. The reason for the macro based approach is that it considerably decreases the maintenance burden - the number of languages available upstream has increased drammatically. I don't see the advantage of having an extra src.rpm for the langpacks, and I definitely would not want to write out the subpackage entries for all the langpacks. Second, I'm not sure why you say that changes were blindly merged from master to all other branches. This work was introduced for F23 and is available only in F23 onwards, plus as newpackage in el6 and el7. I am okay with any way to implement virtual provides. Oh I take back then what I said. I just didn't like the way virtual provides removed and then just looking at koji I saw same n-v-r built for F23+ and EPEL branches and concluded that master changes backported to EPEL as well. Ok, I'll add the virtual provides then. I suppose in the case of %lang_subpkg aze Azerbaijani %lang_subpkg aze_cyrl "Azerbaijani (Cyrilic)" I'd use az and az_cyrl ? I also want to add here we are currently working on using RPM weak dependencies to enhance the langpacks installation by adding Supplements: tag. I may open one more bugzilla for adding Supplements tag then. We are waiting for approval of https://fedorahosted.org/fpc/ticket/593. I think its good to provide langpack subpackages for only those languages where glibc locale is available. E.g. See /usr/share/i18n/locales. I am not sure how can we use other languages that tesseract is providing where glibc locale is not available actually. Yes use az and az_cyrl. But az_cyrl is not locale code. I will look more into this and can provide you patch on Monday. I need to check what changes happened from 3.02 for languages. Ok, I'll wait for your patch then. Thanks! Created attachment 1117977 [details]
langpacks guidelines patch
If this patch looks okay to you, then please apply it to tesseract package git.
The only question I have: are there consequences in having duplicate provides? I.e. %lang_subpkg deu_frak deu_frak "German (Fraktur)" would have an implicit and an explicit provide on tesseract-langpack-deu_frak Otherwise looks good, many thanks! Created attachment 1118062 [details]
langpacks guidelines patch
Does this looks good now compared to previous patch?
There are few messages when you do fedpkg local but I can't think any solution to silent them as langcode macro need to be defined at top in lang_subpkg() macro.
tesseract-3.04.00-3.el6 has been submitted as an update to Fedora EPEL 6. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-362b4a67ec tesseract-3.04.00-3.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-37131e3b31 tesseract-3.04.00-3.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-f5743f41b4 tesseract-3.04.00-3.el6 has been pushed to the Fedora EPEL 6 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-2016-362b4a67ec smani, You are awesome. Thank you for accepting my patch. tesseract-3.04.00-3.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-2016-f5743f41b4 tesseract-3.04.00-3.fc23 has been pushed to the Fedora 23 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-2016-37131e3b31 tesseract-3.04.00-3.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report. tesseract-3.04.00-3.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report. tesseract-3.04.00-3.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report. |