On 14.12.2011 10:34, Vincent Untz wrote:
If people are using %suse_update_desktop_file to edit the .desktop file, I recommend to look at using desktop-file-install instead (in desktop-file-utils). It doesn't provide all the options of %suse_update_desktop_file yet, but I can add what is needed. This would move us closer to what other distros do.
Well, this is bizarre anyway. Check blender.spec: # x-blend.desktop %if 0%{?suse_version} install -D -m 0644 "%{S:7}" "%{buildroot}%{_datadir}/applications/x-blend.desktop" %suse_update_desktop_file %{buildroot}/%{_datadir}/applications/x-blend.desktop # blender.desktop %suse_update_desktop_file -i "%{name}" %else %if 0%{?fedora} # Install the .desktop file the Fedora way desktop-file-install --dir=%{buildroot}%{_datadir}/applications %{S:7} # Validate blender.desktop desktop-file-validate %{buildroot}%{_datadir}/applications/blender.desktop %endif %endif I can't say that the desktop-file-install line is nicer than using -i option, but for a spec file that is meant to compile on both, it sounds good to have one way.
There is one problem with removing the buildrequires on update-desktop-files though: this package includes rpm provide triggers for mimehandler(), so don't remove the buildrequire if you have MimeType= in your .desktop file. Or we move that triggers to a package always installed in factory.
We can move that to desktop-file-utils if it helps.
Sounds like a plan - in combination with an rpmlint error if you have MimeType in .desktop and do not provide mimehandler() Greetings, Stephan -- To unsubscribe, e-mail: opensuse-packaging+unsubscribe@opensuse.org To contact the owner, e-mail: opensuse-packaging+owner@opensuse.org