bug-bash
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Add-rm-rf-as-examples-loadables-rm.c


From: Cedric Blancher
Subject: Re: [PATCH] Add-rm-rf-as-examples-loadables-rm.c
Date: Mon, 31 Oct 2016 15:30:26 +0100

Some comments, since ksh93 already has the same (POSIX utilities as builtins):
1. All builtin code should be reentrant (you'll get to that point at
*least* with find(1) -exec, and at that point it will bite you into
your *ARSE*)
2. All builtin code must not use global variables or any global
resources "write" (see (1), same warning, bloody *arse* if you try)
3. Be aware of locale changes (also global resource)

Ced


On 31 October 2016 at 12:59, Tim Ruehsen <address@hidden> wrote:
> Added the 'rm' command to examples/loadables.
>
> It accepts -r and -f.
> Enable it with 'enable -f <path-to-bash>/examples/loadables/rm rm'.
>
> Tested on wget configure script where it increases overall execution time by
> 6-7%.
>
> Tim



-- 
Cedric Blancher <address@hidden>
[https://plus.google.com/u/0/+CedricBlancher/]
Institute Pasteur



reply via email to

[Prev in Thread] Current Thread [Next in Thread]