On 10/01/2014 03:07 PM, Björn Persson wrote: > Kalev Lember <kalevlem...@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 -- devel mailing list devel@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/devel Fedora Code of Conduct: http://fedoraproject.org/code-of-conduct
