Josh Triplett wrote: > On Fri, 2007-07-13 at 16:50 +0100, Rob Taylor wrote: >> Any followups on this? > > I actually committed the first three patches this morning, before you > sent this. I would like to commit the fourth patch this evening. I > want to think a bit about how to address one issue: I'd really love to > avoid constructs specific to GNU make, such as ifeq and $(shell). I > managed to find a way to do that for the patch adding clean kernel-style > build output, by using the text substitution feature of make variable > expansion. However, I haven't yet figured out a way to avoid $(shell) > with this patch. The c2xml target can just always exist, so no > conditionals needed there. However, without $(shell), I don't see any > way to handle adding c2xml to PROGRAMS and INST_PROGRAMS without > $(shell); backquotes will only work in the commands of a target, not the > prerequisites. (Obviously, just leaving c2xml out of the all and > install targets would solve the problem, but that seems quite > suboptimal.) I want to think about this problem for a bit, and if I > don't come up with anything and don't get any good suggestions, I may > just go ahead and require GNU make. OK, forget it. Anyone who wants Sparse to work with non-GNU make gets to rack *their* brain thinking about how to do without useful GNU make features. Patch applied... > Also, you didn't update the dtd for the changes to the position > handling; it doesn't have end-file, and it still has start and end > rather than {start,end}-{line,col}. ... and I fixed this myself. Thanks again for some most impressive work. - Josh Triplett
Attachment:
signature.asc
Description: OpenPGP digital signature