Spec URL: http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Security.spec SRPM URL: http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Security-2.0.14-1.fc16.src.rpm Description: Symfony2 Security Component.
Updates per comments in bug 823043 - Removed BuildRoot - Changed php require to php-common - Added the following requires based on phpci results: php-date, php-hash, php-json, php-openssl, php-pcre, php-reflection, php-spl - Removed %defattr from %files section - Removed ownership for directories already owned by required packages Update per https://bugzilla.redhat.com/show_bug.cgi?id=823041#c5 - Moved documentation to correct location SPEC URL: http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Security.spec SRPM URL: http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Security-2.0.14-3.fc16.src.rpm
Updated to upstream version 2.0.15 & updates per bug #817303 - Removed "BuildRequires: php-pear >= 1:1.4.9-1.2" - Updated %prep section - Removed cleaning buildroot from %install section - Removed documentation move from %install section (fixed upstream) - Removed %clean section - Updated %doc in %files section SPEC URL: http://people.redhat.com/siwinski/rpmbuild/SPECS/php-symfony2-Security.spec SRPM URL: http://people.redhat.com/siwinski/rpmbuild/SRPMS/php-symfony2-Security-2.0.15-1.fc16.src.rpm
Added optional requires - Added php-pear(%{pear_channel}/ClassLoader) require - Added php-pear(%{pear_channel}/Finder) require - Added php-pear(%{pear_channel}/Form) require - Added php-pear(%{pear_channel}/Routing) require SPEC URL: http://people.redhat.com/~siwinski/rpmbuild/SPECS/php-symfony2-Security.spec SRPM URL: http://people.redhat.com/~siwinski/rpmbuild/SRPMS/php-symfony2-Security-2.0.15-2.fc17.src.rpm
Please check file placement of *sql (mark as doc). This files seems not used. You probably need php-doctrine-DoctrineCommon and php-doctrine-DoctrineDBAL (not yet packaged). So please 1/ assure this is an optionnal dep 2/ consider submitting a review for this 2packages. In Security-2.0.15/Symfony/Component/Security/Acl/Dbal/AclProvider.php foreach ($stmt->fetchAll(\PDO::FETCH_NUM) as $data) { So also need php-pdo
Created attachment 594258 [details] php-symfony2-Security-review.txt Generated by fedora-review 0.1.3
(In reply to comment #4) > Please check file placement of *sql (mark as doc). > This files seems not used. I will update the spec to modify package.xml and make these files role="php". Upstream v2.1.0-BETA1 has several packaging fixes in it and these files are role="php" there. > You probably need php-doctrine-DoctrineCommon and php-doctrine-DoctrineDBAL > (not yet packaged). > > So please > 1/ assure this is an optionnal dep > 2/ consider submitting a review for this 2packages. It appears that these are optional. Looking through the dependency updates for upstream version 2.1.0 BETA1 (which has several packaging fixes in it), the PEAR dependencies are the same, but the composer dependencies for those packages are "require-dev" which means "packages required for developing this package, or running tests, etc." but not installed as normal required packages. v2.0.15: https://github.com/symfony/Security/blob/v2.0.15/composer.json v2.1.0-BETA1: https://github.com/symfony/Security/blob/v2.1.0-BETA1/composer.json I am working on the packaging of DoctrineCommon and DoctrineDBAL and will add them as requires to this package when they are available. > In Security-2.0.15/Symfony/Component/Security/Acl/Dbal/AclProvider.php > foreach ($stmt->fetchAll(\PDO::FETCH_NUM) as $data) { > > So also need php-pdo I will update the spec with this.
Updates per comment #4 - Added php-pdo require - Updated %description - Changed PEAR role of *.sql files from doc to php (Note: This has been fixed in upstream version 2.1.0 BETA1) SPEC URL: http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-symfony2-Security.spec SRPM URL: http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-symfony2-Security-2.0.15-3.fc17.src.rpm
$ colordiff -u php-symfony2-Security.spec.001 php-symfony2-Security.spec --- php-symfony2-Security.spec.001 2012-06-13 08:31:19.000000000 +0200 +++ php-symfony2-Security.spec 2012-06-30 06:44:46.000000000 +0200 @@ -4,7 +4,7 @@ Name: php-symfony2-Security Version: 2.0.15 -Release: 2%{?dist} +Release: 3%{?dist} Summary: Symfony2 %{pear_name} Component Group: Development/Libraries @@ -22,6 +22,7 @@ Requires: php-json Requires: php-openssl Requires: php-pcre +Requires: php-pdo Requires: php-reflection Requires: php-spl Requires: php-pear(PEAR) @@ -40,7 +41,12 @@ Provides: php-pear(%{pear_channel}/%{pear_name}) = %{version} %description -%{summary}. +Security provides an infrastructure for sophisticated authorization systems, +which makes it possible to easily separate the actual authorization logic from +so called user providers that hold the users credentials. It is inspired by +the Java Spring framework. + +Optional dependencies: DoctrineCommon and DoctrineDBAL %prep @@ -48,6 +54,12 @@ # package.xml is version 2.0 mv package.xml %{pear_name}-%{version}/%{name}.xml +# Change PEAR role of *.sql files from doc to php. +# Fixed in upstream version 2.1.0 BETA1. +sed -i \ + 's#<file *md5sum="\([^"]\+\)" *name="\([^"]\+.sql\)" *role="doc" */>#<file md5sum="\1" name="\2" role="php" />#' \ + %{pear_name}-%{version}/%{name}.xml + %build # Empty build section, most likely nothing required. @@ -84,6 +96,11 @@ %changelog +* Sat Jun 30 2012 Shawn Iwinski <shawn.iwinski> 2.0.15-3 +- Added php-pdo require +- Updated %%description +- Changed PEAR role of *.sql files from doc to php + * Mon Jun 11 2012 Shawn Iwinski <shawn.iwinski> 2.0.15-2 - Added php-pear(%%{pear_channel}/ClassLoader) require - Added php-pear(%%{pear_channel}/Finder) require
$ rpmdiff php-symfony2-Security-2.0.15-2.fc17.remi.noarch.rpm \ php-symfony2-Security-2.0.15-3.fc16.remi.noarch.rpm added REQUIRES php-pdo ... removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources/schema removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources/schema/db2.sql removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources/schema/mssql.sql removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources/schema/mysql.sql removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources/schema/oracle.sql removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources/schema/postgresql.sql removed /usr/share/doc/pear/Security/Symfony/Component/Security/Acl/Resources/schema/sqlite.sql ... added /usr/share/pear/Symfony/Component/Security/Acl/Resources/schema added /usr/share/pear/Symfony/Component/Security/Acl/Resources/schema/db2.sql added /usr/share/pear/Symfony/Component/Security/Acl/Resources/schema/mssql.sql added /usr/share/pear/Symfony/Component/Security/Acl/Resources/schema/mysql.sql added /usr/share/pear/Symfony/Component/Security/Acl/Resources/schema/oracle.sql added /usr/share/pear/Symfony/Component/Security/Acl/Resources/schema/postgresql.sql added /usr/share/pear/Symfony/Component/Security/Acl/Resources/schema/sqlite.sql
All blockers fixed == APPROVED ==
New Package SCM Request ======================= Package Name: php-symfony2-Security Short Description: Symfony2 Security Component Owners: siwinski Branches: f16 f17 el6 InitialCC:
Git done (by process-git-requests).
php-symfony2-Security-2.0.15-3.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/php-symfony2-Security-2.0.15-3.fc17
php-symfony2-Security-2.0.15-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/php-symfony2-Security-2.0.15-3.el6
php-symfony2-Security-2.0.15-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/php-symfony2-Security-2.0.15-3.fc16
php-symfony2-Security-2.0.15-3.fc17 has been pushed to the Fedora 17 testing repository.
php-symfony2-Security-2.0.15-3.fc16 has been pushed to the Fedora 16 stable repository.
php-symfony2-Security-2.0.15-3.fc17 has been pushed to the Fedora 17 stable repository.
php-symfony2-Security-2.0.15-3.el6 has been pushed to the Fedora EPEL 6 stable repository.