Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 1470358 (php-jms-serializer)
Summary: | Review Request: php-jms-serializer - Library for (de-)serializing data of any complexity | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
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+
|
||||||||
Target Release: | --- | ||||||||||
Hardware: | All | ||||||||||
OS: | Linux | ||||||||||
Whiteboard: | |||||||||||
Fixed In Version: | Doc Type: | If docs needed, set a value | |||||||||
Doc Text: | Story Points: | --- | |||||||||
Clone Of: | Environment: | ||||||||||
Last Closed: | 2017-09-28 16:55:08 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: | 1458576, 1483262 | ||||||||||
Attachments: |
|
Description
Shawn Iwinski
2017-07-12 19:50:45 UTC
DEBUG util.py:450: No matching package to install: 'php-JMSParser >= 1.0.0-7' DEBUG util.py:450: No matching package to install: 'php-composer(jms/metadata) >= 1.6.0' DEBUG util.py:450: No matching package to install: 'php-composer(jms/metadata) < 2.0' DEBUG util.py:450: No matching package to install: 'php-composer(jms/parser-lib) < 2.0' DEBUG util.py:450: No matching package to install: 'php-composer(phpcollection/phpcollection) >= 0.5.0' DEBUG util.py:450: No matching package to install: 'php-composer(phpcollection/phpcollection) < 1.0' Missing packages in rawhide... probably we need a successful "compose" Update to 1.8.1 Spec URL: https://raw.githubusercontent.com/siwinski/rpms/c95392c98f697c8df3ed110ef44872581cd335cf/php-jms-serializer/php-jms-serializer.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-jms-serializer-1.8.1-1.fc26.src.rpm Still same error... :( Looking at https://src.fedoraproject.org/rpms/php-JMSParser/c/2ce3bc7de8cef3c4873f53f305109b37523d6d87?branch=master https://src.fedoraproject.org/rpms/php-Metadata/c/9ce5dab2be8eb460b7d47d054207e6739338fa02?branch=master https://src.fedoraproject.org/rpms/php-PhpCollection/c/576548c4d44a27b0091f0b06f1a9edfee0874498?branch=master These 3 packages have been retired so are blocked, and don't go in the repository. You have to have them unblocked first. Despite a rawhide repository update (Fri, 25 Aug 2017 13:26:56 +0200) Missing packages sare still not there Ex: https://rpms.remirepo.net/rpmphp/zoom.php?rpm=php-PhpCollection Scratch builds: * rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=22013039 * f27: https://koji.fedoraproject.org/koji/taskinfo?taskID=22013041 * f26: https://koji.fedoraproject.org/koji/taskinfo?taskID=22013043 * f25: https://koji.fedoraproject.org/koji/taskinfo?taskID=22013045 Sadly, the needed packages are still not available in the repository, including rawhide... BTW I will run the review against local packages, and let you manage this issue (perhaps only have to wait for a new "compose") Minor test failure, only with PHP 5.6 and Twig 2 (ok with other PHP versions or with Twig 1) There were 2 errors: 1) JMS\Serializer\Tests\Twig\SerializerExtensionTest::testSerialize Argument 1 passed to Twig_Filter::__construct() must be an instance of string, string given, called in /dev/shm/extras/BUILDROOT/php-jms-serializer-1.8.1-1.fc25.remi.x86_64/usr/share/php/JMS/Serializer/Twig/SerializerExtension.php on line 46 and defined /usr/share/php/Twig2/Filter.php:35 /dev/shm/extras/BUILDROOT/php-jms-serializer-1.8.1-1.fc25.remi.x86_64/usr/share/php/JMS/Serializer/Twig/SerializerExtension.php:46 /dev/shm/extras/BUILD/serializer-ce65798f722c836f16d5d7d2e3ca9d21e0fb4331/tests/Twig/SerializerExtensionTest.php:40 2) JMS\Serializer\Tests\Twig\SerializerExtensionTest::testRuntimeSerializerExtension Argument 1 passed to Twig_Filter::__construct() must be an instance of string, string given, called in /dev/shm/extras/BUILD/serializer-ce65798f722c836f16d5d7d2e3ca9d21e0fb4331/tests/Twig/SerializerExtensionTest.php on line 70 and defined /usr/share/php/Twig2/Filter.php:35 /dev/shm/extras/BUILD/serializer-ce65798f722c836f16d5d7d2e3ca9d21e0fb4331/tests/Twig/SerializerExtensionTest.php:70 Notice: "php56" should be last case of the loop, as it disable some tests (sed) BTW, not a blocker as Fedora only have PHP 7 Simply, Twig 2 is not compatible with PHP 5 ;) (could be tweak in autoloader, but perhaps doesn't worth the work for Fedora) Created attachment 1329367 [details]
phpci.log
phpCompatInfo version 5.0.8 DB version 1.23.0 built Jul 17 2017
20:33:14 CEST static analyze results
Created attachment 1329368 [details]
phpci.log
Created attachment 1329369 [details]
review.txt
Generated by fedora-review 0.6.1 (f03e4e7) last change: 2016-05-02
Command line :/usr/bin/fedora-review -b 1470358 -L /dev/shm/extras/RPMS/noarch
Buildroot used: fedora-rawhide-x86_64
Active plugins: Generic, PHP, Shell-api
Disabled plugins: Java, C/C++, Python, fonts, SugarActivity, Ocaml, Perl, Haskell, R
Disabled flags: EXARCH, DISTTAG, EPEL5, BATCH, EPEL6
Built with local dependencies:
/dev/shm/extras/RPMS/noarch/php-PhpCollection-0.5.0-4.fc25.remi.noarch.rpm
/dev/shm/extras/RPMS/noarch/php-JMSParser-1.0.0-12.fc25.remi.noarch.rpm
[x]: Package complies to the Packaging Guidelines === APPROVED === THANKS! https://pagure.io/releng/fedora-scm-requests/issue/1535 https://pagure.io/releng/fedora-scm-requests/issue/1536 https://pagure.io/releng/fedora-scm-requests/issue/1537 https://pagure.io/releng/fedora-scm-requests/issue/1538 FYI bug 1483262 is up next :) (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/php-jms-serializer php-jms-serializer-1.8.1-1.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2017-6aea60dd2a php-jms-serializer-1.8.1-1.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-d4d7812e2e php-jms-serializer-1.8.1-1.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-a5510eed4b php-jms-serializer-1.8.1-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-a5510eed4b php-jms-serializer-1.8.1-1.fc25 has been pushed to the Fedora 25 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-d4d7812e2e php-jms-serializer-1.8.1-1.fc27 has been pushed to the Fedora 27 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-6aea60dd2a php-jms-serializer-1.8.1-1.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. php-jms-serializer-1.8.1-1.fc26 has been pushed to the Fedora 26 stable repository. If problems still persist, please make note of it in this bug report. php-jms-serializer-1.8.1-1.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report. |