Eric Kow <[EMAIL PROTECTED]> writes:

> Hi David,
>
> On Fri, Nov 07, 2008 at 07:35:58 -0500, David Roundy wrote:
>> Here are some more patches I had lying around (I just amended them to
>> resolve some conflicts).  Not important stuff, just type witness cleanups.
>> At least the cutting of dead code should be accepted.
>
> Nice to see our witness coverage creeping forward.  Having realised that
> Jason has a thesis to work on and that I shouldn't be asking any patch
> review from him, I've taken a look and applied these, thanks!
>
> enable type witnesses for show commands.
> ----------------------------------------
>> +    src/Darcs/Commands/Show.o \

It has been suggested that the "witnesses" target's dependencies be
automatically generated (as is currently done for DARCS_FILES), so that
the makefile doesn't need to be edited every time witnesses are added to
another file.

Something like this could be added to witnessed files:

    #define HAS_WITNESSES

And then in GNUmakefile, something like

    witnesses: $(shell egrep -rl define[[:space:]]+HAS_WITNESSES src/)

Does anyone want to comment on the idea?

_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to