Spec URL: https://raw.githubusercontent.com/siwinski/rpms/db67e4b7451d1c963dc4437170aa77a1ce69ab00/php-patchwork-utf8/php-patchwork-utf8.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-patchwork-utf8-1.2.3-1.fc22.src.rpm Description: Portable and performant UTF-8, Unicode and Grapheme Clusters for PHP Fedora Account System Username: siwinski
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.