On 10/01/2014 03:07 PM, Björn Persson wrote:
Kalev Lember kalevlember@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