[comp.os.minix] [bugfix] Fix to Clam's alias/pipe bug?

wkt@ccadfa.adfa.oz.au (Warren Toomey) (09/28/90)

David Feynes reported Clam's alias/pipe bug, which has been known for
some time. Here's a one-line _hack_ which fixes it. I've tested it
and it appears to work, but it may have some side effects, I don't know.

Could David and as many others test this patch & report success/failure?!

The patch is to the Oct'89 posting of the source, i.e to the source as
it appears on plains/sirius/ccadfa.

	Warren

P.S Note the crcs in the patch - I propose cdiff.c be changed to add
    these. Votes for and against, please!


echo x - exec.cdif
sed '/^X/s///' > exec.cdif << '/'
X*** 23687  20924 oexec.c	Fri Sep 28 10:28:45 1990
X--- 14056  20939 exec.c		Fri Sep 28 10:48:49 1990
X***************
X*** 177,182 ****
X--- 177,183 ----
X    int i,pos,ppos,pid=0,oldnumargtemp,retval;
X    bool skip_to_semi;
X  
X+   bckgnd=TRUE;
X    if (bckgnd==TRUE) NoIoctl=1;
X    if (alias_depth==0)	/* are we at the bottom level? */
X    {
/


-- 
Warren Toomey VK2XWT, not flying this w/e?
 Deep in the bowels of ADFA Comp Science.
 `At least he didn't bring in a bucket!'

cagney@chook.ua.oz (Andrew Cagney - aka Noid) (09/28/90)

wkt@ccadfa.adfa.oz.au (Warren Toomey):
> P.S Note the crcs in the patch - I propose cdiff.c be changed to add
>     these. Votes for and against, please!

Interesting idea. I don't know about the format. What about instead
adding a line of the form:
        Crc: 23687  20924 oexec.c
or just
        23687  20924 oexec.c
before the patch? Pam (1) looks for these & normal patch would ignore them.

                                        Andrew


(1) Pam, patch for minix, a moronic rewrite of patch that
        1. runs faster :-)
        2. Doesn't have memory problems :-)
but     3. has no fuzzy apply logic :-(