On 27 April 2013 00:57, Ted Unangst <[email protected]> wrote:
> On Fri, Apr 26, 2013 at 20:17, Eitan Adler wrote:
>
>> Ah, I see. I sent this patch because { Free, Net, DragonFly } BSD all
>> have this feature. After some private discussion I found out that
>> OpenBSD has a much strong aversion to non-POSIX features than the
>> others.
>
> fwiw, "they did this, you should too" isn't a great way to sell a patch. it's
> helpful to mention, but the merits of the patch should stand on their own, and
> you didn't really explain how it's better than the alternatives. it was nice
> to
> get a patch though.
I think its faster, easier to use/remember, avoids PATH_MAX issues,
and possibly avoids certain races.
The new syntax is simple and clear and looks nicer than
find -x . -execdir rm {} \;
That said, I'll leave it to y'all to decide if you want the
functionality or not. I just offered the code. :)
--
Eitan Adler
