Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=823071
Remi Collet <fedora(a)famillecollet.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |fedora(a)famillecollet.com
--- Comment #1 from Remi Collet <fedora(a)famillecollet.com> 2012-05-19 02:50:18 EDT
---
Just a quick notes
- LICENSE and README file are installed in /usr/share/pear.
Probably a good idea to ask upstream, this files should be tagged as "doc" and
so installed in /usr/share/doc/pear (and avoid to be duplicated in the package)
- Extension dependencies
# phpci print --recursive --report extension Symfony
-------------------------------------------------------------------------------
PHP COMPAT INFO EXTENSION SUMMARY
-------------------------------------------------------------------------------
EXTENSION PECL VERSION COUNT
-------------------------------------------------------------------------------
Core 4.0.0 309
SPL 0.2 5.0.0 5
ctype 4.0.4 3
date 4.0.0 10
intl 1.1.0 5.2.4 12
pcre 4.0.0 13
session 4.0.0 3
standard 4.0.0 689
-------------------------------------------------------------------------------
So need to requires php-intl
- Other classes dependencies
From package.xml => ok
Requires:
php-pear((%{pear_channel}/EventDispatcher)
Requires: php-pear((%{pear_channel}/Validator)
Requires: php-pear((%{pear_channel}/Locale)
And optionnaly (your decision)
Requires: php-pear((%{pear_channel}/HttpFoundation)
- directory ownership
%dir %{pear_phpdir}/Symfony/Component
%dir %{pear_phpdir}/Symfony
You don't need to own this directories which are already owned by required
packages.
--
Configure bugmail:
https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.