Re: [9fans] cc and more

["rob pike" <rob@plan9.bell-labs.com>]

Quinn is right; ape/cc and pcc have a nefarious bug.  We never caught
it because it is triggered by specifying libraries explicitly, which is
almost never necessary in Plan 9 - our `autolib' #pragma figures out
which libraries to use.  But if you override that, for whatever reason,
you can overwrite the wrong files.  Quinn did
	ape/cc -v -o test -lc test.c
which caused the output to be written to /$objtype/lib/ape/libc.a.
(The -lc is unnecessary and in fact erroneous).

Here are the patches:

diff -n /n/dump/2000/0708/sys/src/cmd/pcc.c /sys/src/cmd/pcc.c
/n/dump/2000/0708/sys/src/cmd/pcc.c:154 c pcc.c:154,157
< 			append(&cc, objs.strings[i]);
---
> 			if(cflag)
> 				append(&cc, oname);
> 			else
> 				append(&cc, changeext(srcs.strings[i], ot->o));

diff -n /n/dump/2000/0708/sys/src/cmd/pcc.c /sys/src/cmd/pcc.c
/n/dump/2000/0708/sys/src/cmd/pcc.c:52 c /sys/src/cmd/pcc.c:52
< 	int i, cppn, ccn;
---
> 	int i, cppn, ccn, oflag;
/n/dump/2000/0708/sys/src/cmd/pcc.c:53 a /sys/src/cmd/pcc.c:54
> 	oflag = 0;
/n/dump/2000/0708/sys/src/cmd/pcc.c:69 a /sys/src/cmd/pcc.c:71
> 			oflag = 1;
/n/dump/2000/0708/sys/src/cmd/pcc.c:154 c /sys/src/cmd/pcc.c:156,159
< 			append(&cc, objs.strings[i]);
---
> 			if(oflag && cflag)
> 				append(&cc, oname);
> 			else
> 				append(&cc, changeext(srcs.strings[i], ot->o));