--- php-horde-Horde-Nls.spec.old	2012-01-30 19:38:29.000000000 +0100
+++ php-horde-Horde-Nls.spec	2012-01-30 19:46:15.000000000 +0100
@@ -12,14 +12,15 @@
 Source0:        http://pear.horde.org/get/%{pear_name}-%{version}.tgz
 
 BuildArch:      noarch
-BuildRequires:  php-pear >= 1:1.4.9-1.2
+BuildRequires:  php-pear(PEAR) >= 1.7.0
+BuildRequires:  php-channel(pear.horde.org)
+
 Requires(post): %{__pear}
 Requires(postun): %{__pear}
-Requires:       php-pear(pear.horde.org/Horde_Translation) <= 2.0.0, php-pear(pear.horde.org/Horde_Util) <= 2.0.0, php-pear(PEAR) >= 1.7.0
-Conflicts:      php-pear(pear.horde.org/Horde_Translation) = 2.0.0, php-pear(pear.horde.org/Horde_Util) = 2.0.0
-Provides:       php-pear(pear.horde.org/Horde_Nls) = %{version}
-BuildRequires:  php-channel(pear.horde.org)
-Requires:       php-channel(pear.horde.org)
+Requires:       php-pear(pear.horde.org/Horde_Util) >= 1.0.0
+Requires:       php-pear(pear.horde.org/Horde_Util) <  2.0.0
+Requires:       php-pear(PEAR) >= 1.7.0
+Provides:       php-pear(pear.horde.org/%{Horde_Nls}) = %{version}
 
 

I propose to remove Horde_Translation, already required by dependency of Horde_Util (you could keep it, if you think "versionning" have some value)

Requiring the channel not needed (I miss to drop this on other packages, except Translation which don't requires other horde package)

+ must handle locales

NB optional dep on Net_DNS2 (not yet available in fedora) and geoip (available). As rpm doesn't handle optional dep, I always think of adding this are normal dep (when not pull to much things => but this is your choice

I'm confused.  According to http://fedoraproject.org/wiki/Packaging:PHP#PEAR_Packages_from_a_non_standard_channel.2Frepository it seems to indicate that I must require the channel?

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.

I believe all outstanding issues are fixed now.

> NB optional dep on Net_DNS2 (not yet available in fedora) and geoip
> (available). As rpm doesn't handle optional dep, I always think of adding
> this as normal dep (when not pull to much things) => but this is your
> choice

This one is also ok now.

APPROVED

New Package SCM Request
=======================
Package Name: php-horde-Horde-Nls
Short Description: Common methods for handling language data, timezones, and
hostname->country lookups
Owners: nb
Branches: f16 f17 el6
InitialCC:

Git done (by process-git-requests).