[Bug 1032108] Review Request: yarock - A Lightweight and Beautiful Music Player

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1032108



--- Comment #51 from Ralf Corsepius <rc040203@xxxxxxxxxx> ---
(In reply to James Abtahi from comment #50)
> > /usr/share/locale/yarock is a bug. Package simply MUST not install i18n files there in.
> 
> reference?
This is common sense ever since i18n exists and is hard-wired into many tools.

> I thought Kevin Kofler said:
> 
> > The gettext scheme is /usr/share/locale/%{locale}/%{name}.mo, but Qt doesn't 
> > support that scheme, it expects *_%{locale}.qm (or just %{locale}.qm, which 
> > some apps try to use and which is NOT supported by %find_lang) files in a
> > common directory.
> -----
> 
> > This package is broken and should to be fixed.
> 
> and what is your solution exactly?
I haven't had a deepper look into this package yet.

I only stumbled over the 
%dir %{datadir}/locale/%{name}
inside of your *spec. A line which is multiply questionable.

> I thought this was a a bug with qt.
I can't tell without having had a deeper look into your package. My guess would
be, this is an bug inside of this package.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]