21 Feb
2007
21 Feb
'07
1:15 a.m.
Stefan O'Rear wrote:
Besides, remember that 50 patches is 50x the documentation :)
Yes, but it's 50x the blundering about when trying to review the code :-) Here's a seemingly reasonable compromise. I've attached a single unified diff that contains all of my changes in a single lump. If the code looks sane, you can then "darcs pull" the real changes and see if the history "smells right". There shouldn't be any review surprises. The only "unrelated" change it makes is to check the return code from tar in SrcDist, something that must have been overlooked. The actual darcs history is pretty clean, too. Small, self-contained changes, etc, etc.