Bug 1262469
Summary: | Review Request: php-patchwork-utf8 - Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Shawn Iwinski <shawn> | ||||||
Component: | Package Review | Assignee: | Remi Collet <fedora> | ||||||
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | medium | ||||||||
Version: | rawhide | CC: | fedora, package-review | ||||||
Target Milestone: | --- | Flags: | fedora:
fedora-review+
gwync: fedora-cvs+ |
||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2015-10-05 18:14:56 UTC | Type: | --- | ||||||
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: | 1262185 | ||||||||
Attachments: |
|
Description
Shawn Iwinski
2015-09-11 20:02:25 UTC
Created attachment 1073081 [details]
phpci.log
phpCompatInfo version 4.4.0 DB built Jul 20 2015 15:09:02 CEST static analyze results
Created attachment 1073082 [details]
phpci.log
Generated by fedora-review 0.5.3 (bcf15e3) last change: 2015-05-04
[!]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. => mandatory for ASL 2.0 Please file an upstream bug and add link in .spec Notice: as you have a link to both license texts in file headers, I think it is ok to use it as a "source" for the package ("Package does not include license text files separate from upstream." doesn't apply in this case). But of course, habing upstream file seems the best solution. IN composer.json "classmap": ["src/Normalizer.php"] I think this is only a compat file, seems fine to ignore this one. (In reply to Remi Collet from comment #3) > [!]: If the source package does not include license text(s) as a separate > file from upstream, the packager SHOULD query upstream to include it. > => mandatory for ASL 2.0 > > Please file an upstream bug and add link in .spec > > Notice: as you have a link to both license texts in file headers, I think it > is ok to use it as a "source" for the package ("Package does not include > license text files separate from upstream." doesn't apply in this case). But > of course, habing upstream file seems the best solution. Thanks! Updated. Upstream request: https://github.com/tchwork/utf8/pull/50 Spec diff: https://github.com/siwinski/rpms/commit/ff7ce3ee4ddc259f69bd62083087f9f2cf2116cc#diff-1 > IN composer.json > "classmap": ["src/Normalizer.php"] > I think this is only a compat file, seems fine to ignore this one. Yes, that is just a compat file for when the intl extension is not available but it is specified as a requirement of this package. Spec URL: https://raw.githubusercontent.com/siwinski/rpms/ff7ce3ee4ddc259f69bd62083087f9f2cf2116cc/php-patchwork-utf8/php-patchwork-utf8.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-patchwork-utf8-1.2.3-2.fc22.src.rpm [x]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. === APPROVED === THANKS for the review! New Package SCM Request ======================= Package Name: php-patchwork-utf8 Short Description: Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP Upstream URL: https://github.com/tchwork/utf8 Owners: siwinski Branches: f21 f22 f23 epel7 InitialCC: Git done (by process-git-requests). php-patchwork-utf8-1.2.3-3.fc21 has been submitted as an update to Fedora 21. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16597 php-patchwork-utf8-1.2.3-3.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16598 php-patchwork-utf8-1.2.3-3.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-8219 php-patchwork-utf8-1.2.3-3.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2015-16599 php-patchwork-utf8-1.2.3-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. If you want to test the update, you can install it with $ su -c 'yum --enablerepo=epel-testing update php-patchwork-utf8' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2015-8219 php-patchwork-utf8-1.2.3-3.fc21 has been pushed to the Fedora 21 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with $ su -c 'dnf --enablerepo=updates-testing update php-patchwork-utf8' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16597 php-patchwork-utf8-1.2.3-3.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with $ su -c 'dnf --enablerepo=updates-testing update php-patchwork-utf8' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16599 php-patchwork-utf8-1.2.3-3.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with $ su -c 'dnf --enablerepo=updates-testing update php-patchwork-utf8' You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-16598 php-patchwork-utf8-1.2.3-3.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report. php-patchwork-utf8-1.2.3-3.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report. php-patchwork-utf8-1.2.3-3.fc21 has been pushed to the Fedora 21 stable repository. If problems still persist, please make note of it in this bug report. php-patchwork-utf8-1.2.3-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. |