Extract the DESKTOP_ENTRIES related targets.

[jilles]

Mk/Scripts/check-desktop-entries.sh

74

The previous code using echo|grep matched the suffix case-insensitively.

75–77

The expr syntax is invalid because / is interpreted as a division operator. Perhaps

case ${Icon} in
/*) ;;
*)
    ${dp_ECHO_MSG} "${dp_PKGNAME}:  Makefile warning: in desktop entry ${entry}: field 3 (Icon) should be either absolute path or icon name without extension if installed icons follow Icon Theme Specification" ;;
esac

[mat]

• Fix the absolute path check.

[jilles]

Mk/Scripts/check-desktop-entries.sh

76

This will work with the current implementation of expr, but it is not POSIX-compliant, since POSIX leaves it unspecified whether an initial ^ is special (the expression being implicitly anchored anyway).

[mat]

Mk/Scripts/check-desktop-entries.sh

74

True. On the other hand, I did check beforehand, there are no cases where the files are not lower cased.

76

But we do not run in the os called POSIX, we run on the OS called FreeBSD, where the behavior is currently very well defined.
If the behavior ever changes later, then all the code using it will need to change. We live in the now, with an eye of what is in the work, and not using tools because of a vague possibilty that one day the way they work may change is not how I see doing things. I mean, I could never do anything if it was the case.

[jilles]

Mk/Scripts/check-desktop-entries.sh

76

Sorry for being less constructive here. The desired behaviour can be implemented using POSIX-compliant expr as ${dp_EXPR} "x${Icon}" : x/.

My point is not that we should restrict ourselves to POSIX features at all cost, but that an implementation using only POSIX features is preferable to an implementation using extensions if they are similar in other respects. In some situations, non-standard features unique to a system are more likely to change than standard features common with other systems.

Given that this feature (^ in : regular expression) is implemented the same way by GNU expr, it seems not unlikely to change, though.

[mat]

• Switch this one to grep, expr's posix is harder.

[mat]

• Drop dev/null

[mat]

• Quote many things.

[jilles]

It looks like it will work.

This could be optimized a little but I think it is not worth gating the patch for it.