Proposal: Increasing application icon sizes to 64px

Kalev Lember kalevlember at gmail.com
Wed Oct 1 13:19:31 UTC 2014


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

-- 
Thanks,
Kalev


More information about the devel mailing list