Proposal: Increasing application icon sizes to 64px

Björn Persson bjorn at xn--rombobjrn-67a.se
Wed Oct 1 14:01:42 UTC 2014


Kalev Lember <kalevlember at gmail.com> wrote:
> On 10/01/2014 03:07 PM, Björn Persson wrote:
> > Kalev Lember <kalevlember at gmail.com> wrote:
> >> # If no desktop files are installed, return immediately
> >> if ! ls -A "$RPM_BUILD_ROOT"/usr/share/applications/ 2>/dev/null; then
> >> 	exit 0
> >> fi
> > 
> > That tests whether the directory exists, not whether it contains
> > desktop files. If that's what you want, then
> > «if [ -d "$RPM_BUILD_ROOT"/usr/share/applications/ ]; then» would be
> > much less confusing code.
> 
> Yes indeed, I seem to have gotten it wrong. What I wanted to test was
> not for the directory, but for the files inside. Would something like
> this work better?
> 
> if ! ls "$RPM_BUILD_ROOT"/usr/share/applications/*.desktop &>/dev/null; then

That should work, if you want to ignore files in subdirectories such
as /usr/share/applications/kde4. Files without the .desktop suffix will
of course also be ignored, should there be any.

Björn Persson
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.fedoraproject.org/pipermail/devel/attachments/20141001/03ac644d/attachment.sig>


More information about the devel mailing list