big find:
find: missing argument to `-exec'
we now say
find: -exec requires an argument
instead of the cryptic message we emitted previously.
Additional benefit is that it's now 23 bytes smaller.
Add TODO to add '-exec {} +' support
Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com>
ap->exec_argv = ++argv; /* first arg after -exec */
ap->exec_argc = 0;
while (1) {
- if (!*argv) /* did not see ';' until end */
- bb_error_msg_and_die("-exec CMD must end by ';'");
+ if (!*argv) /* did not see ';' or '+' until end */
+ bb_error_msg_and_die(bb_msg_requires_arg, "-exec");
if (LONE_CHAR(argv[0], ';'))
break;
+ //TODO: implement {} + (like xargs)
+ // See:
+ // find findutils/ -exec echo ">"{}"<" \;
+ // find findutils/ -exec echo ">"{}"<" +
argv++;
ap->exec_argc++;
}