[Bug 226433] Merge Review: stardict

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

Summary: Merge Review: stardict


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





------- Additional Comments From panemade at gmail.com  2007-09-03 07:06 EST -------
hey you forgot to change %files
mock build gave me
error: Installed (but unpackaged) file(s) found:
   /usr/share/applications/fedora-stardict.desktop
When using syntax to install desktop files from 
http://fedoraproject.org/wiki/Packaging/Guidelines#head-d559ee7363418a5840ce63090c608c991cd39ce6
desktop file name got renamed to fedora-%{desktopfilename}.desktop

Also, I think you should use following syntax for installation of desktop file
desktop-file-install --vendor fedora --delete-original  \
  --dir $RPM_BUILD_ROOT%{_datadir}/applications         \
  $RPM_BUILD_ROOT%{_datadir}/applications/%{name}.desktop


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.