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

2016-10-31 Thread Chet Ramey
On 10/31/16 5:44 PM, Eric Blake wrote: > On 10/31/2016 04:14 PM, Chet Ramey wrote: > Nice, thanks for the modifications. >> Here's the modified version. >> >> Chet >> -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars >> longa, vita brevis'' - Hippocrates Chet Ramey, UTech,

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

2016-10-31 Thread Eric Blake
On 10/31/2016 04:14 PM, Chet Ramey wrote: >> > Nice, thanks for the modifications. > Here's the modified version. > > Chet > -- ``The lyf so short, the craft so long to lerne.'' - Chaucer ``Ars > longa, vita brevis'' - Hippocrates Chet Ramey, UTech, CWRU c...@case.edu >

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

2016-10-31 Thread Chet Ramey
On 10/31/16 4:40 PM, Tim Rühsen wrote: > On Montag, 31. Oktober 2016 13:09:15 CET Chet Ramey wrote: >> On 10/31/16 7:59 AM, Tim Ruehsen wrote: >>> Added the 'rm' command to examples/loadables. >>> >>> It accepts -r and -f. >> >> Thanks for the submission. I modified it pretty heavily, and it will

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

2016-10-31 Thread Tim Rühsen
On Montag, 31. Oktober 2016 13:09:15 CET Chet Ramey wrote: > On 10/31/16 7:59 AM, Tim Ruehsen wrote: > > Added the 'rm' command to examples/loadables. > > > > It accepts -r and -f. > > Thanks for the submission. I modified it pretty heavily, and it will be > in the next release of bash and the

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

2016-10-31 Thread Cedric Blancher
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

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

2016-10-31 Thread Tim Ruehsen
On Monday, October 31, 2016 7:08:51 AM CET John McKown wrote: > On Mon, Oct 31, 2016 at 6:59 AM, Tim Ruehsen wrote: > > Added the 'rm' command to examples/loadables. > > > > It accepts -r and -f. > > Enable it with 'enable -f /examples/loadables/rm rm'. > > > > Tested on

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

2016-10-31 Thread John McKown
On Mon, Oct 31, 2016 at 6:59 AM, Tim Ruehsen wrote: > Added the 'rm' command to examples/loadables. > > It accepts -r and -f. > Enable it with 'enable -f /examples/loadables/rm rm'. > > Tested on wget configure script where it increases overall execution time > by > 6-7%. >

[PATCH] Fix-gcc-warnings-in-examples-loadables-cat.c

2016-10-31 Thread Tim Ruehsen
Just saw and fixed few warnings on 'make others' in examples/loadable. Tim From dd84f7ab279f6d2ad38618280ed45eff21fab07e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim Rühsen?= Date: Mon, 31 Oct 2016 12:43:42 +0100 Subject: [PATCH 1/2] Fix gcc warnings in

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

2016-10-31 Thread Tim Ruehsen
Added the 'rm' command to examples/loadables. It accepts -r and -f. Enable it with 'enable -f /examples/loadables/rm rm'. Tested on wget configure script where it increases overall execution time by 6-7%. Tim From 5f1a18e570e63c9f010c48aca3a3484f56045fd5 Mon Sep 17 00:00:00 2001 From: