[Rpm-maint] Add appdata() and application() auto-provides
Michael Schroeder
mls at suse.de
Fri Oct 4 12:01:26 UTC 2013
On Wed, Oct 02, 2013 at 07:32:26PM -0400, Colin Walters wrote:
> + echo "application()"
> + echo "application(${instfile##*/applications/})"
> if ! grep -q '^Type=Application$' "$instfile"; then continue; fi
> if ! grep -q '^Exec=' "$instfile"; then continue; fi
>
> Shouldn't these pairs of lines be swapped, so that we don't
> emit provides for things that aren't actually Type=Application for
> example?
I also saw that "if" statement, but decided to put the echo statements
before it. My reasoning is that the application() marker is for
.desktop files in the "/usr/share/applications" directory, independent
of the type. (I must admit that I have no clue about what other types
are allowed in those desktop files.)
Cheers,
Michael.
--
Michael Schroeder mls at suse.de
SUSE LINUX Products GmbH, GF Jeff Hawn, HRB 16746 AG Nuernberg
main(_){while(_=~getchar())putchar(~_-1/(~(_|32)/13*2-11)*13);}
More information about the Rpm-maint
mailing list