Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hugin - Frontend for Panorama Tools, similar to PTAssembler, PTGui or Open for Windows https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=206814 ------- Additional Comments From peter@xxxxxxxxxxxxxxxx 2007-03-15 19:11 EST ------- (In reply to comment #26) > > - includes a %{name}.desktop file, and that file must be properly installed > > with desktop-file-install in the %install section. > > Ok I'm a little confused, hugin already installs > /usr/share/applications/hugin.desktop > > Should I delete this and reinstall it with something like this in %install?: > > rm %{buildroot}/%{_datadir}/applications/%{name}.desktop > desktop-file-install --vendor="fedora" \ > --dir=${buildroot}/%{_datadir}/applications \ > src/hugin/hugin.desktop You don't have too. Call desktop-file-install in %install, passing it the in-buildroot .desktop file as the source, and use the "--delete-original" option. Something like the following should do it (adapt as needed): %install # ... desktop-file-install --vendor fedora \ --dir %{buildroot}%{_datadir}/applications \ --delete-original \ %{buildroot}/%{_datadir}/applications/%{name}.desktop Hope that helps. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review