[Bug 206814] Review Request: hugin - Frontend for Panorama Tools, similar to PTAssembler, PTGui or Open for Windows

[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: 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 at thecodergeek.com  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.