[Bug 785441] Review Request: php-horde-Horde-Nls - Native Language Support (NLS)

[bugzilla at redhat.com]

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=785441

--- Comment #3 from Remi Collet <fedora at famillecollet.com> 2012-02-01 01:13:37 EST ---
Yes, You must requires it for a single package, but as far as you requires
another package in this channel, you already "implicitly" requires it.

After a look to some packages, as all requires
php-pear(PEAR) >= 1.7.0
php-common >= 5.2.0

You could even add this 2 requirement in the channel, to avoid adding it (BR
and R) in each package. This will make the dependency stack really simpler.

About "locales", any feedback from upstream ?

After discussion on IRC, "all files must be generated from sources", so the .mo
should be created during the rpmbuild, and .po/.pot doesn't need to be packaged
(except if you have a good explanation about why they are required)

This will make the spec a little more complex, but will ensure that the .mo
will be accurate, with the .po, and with the gettext version.

-- 
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.