All looks good expect following minor comment
On Wed, 2009-07-29 at 17:12 +0800, Ke Wang wrote:
base-specs/libcanberra.spec
---------------
> +%if %build_l10n
> +%else
> +# REMOVE l10n FILES
> +rm -rf $RPM_BUILD_ROOT%{_datadir}/locale
> +%endif
I do not say any l10n files built out for libcanberra, so removing above
commands should be safe.-Halton
