On Mon, Sep 08, 2008 at 05:05:29PM -0700, Jason Dagit wrote: > Due to the amount of work required to make a clean set of patches I > won't do that again until you're happy with the changes I have > locally. It's just too time consuming to keep doing it that way. > Sorry, but it literally takes me 2-3 days to do the full pull, > unrecord, fixup, re-record, re-test cycle for all these changes. In > the mean time, it's easy and quick to fix things, record over top and > then give you demo of what it should look like and ask you questions > about the remaining tricky spots. I could unrecord everything and > make one big patch if that's easier to review than this non-linear > bundle. I just want to avoid going to all the trouble to make another > 'clean patchset' until you're ready to accept this stuff. I know > you're busy, but making the clean patchsets seems like silly busy work > if we're still discussing the details here. Please suggest something > else if the above plan doesn't fit your style. I found that an > amend-record style is painful due to patch dependencies, hence the > unrecord/record workflow I described.
This really would be much easier if you stopped sending everything at once, then you wouldn't have to take so long to record everything, just fix part of the code, and record that. e.g. it would be great to have a patch that just switched to RIO in Darcs.Repository, but which actually will compile with all of darcs (and pass tests). Keeping this separate from switching to using type witnesses in all the commands would make it a patch that I can both review and apply, which would allow us to move forward. I know this will mean re-doing some work you've already done, but right now we're in this horrible cycle where (1) you do a huge amount of work, (2) I can't possibly review this work because it's too huge (and it's got problems because any huge change has problems), so I make one little suggestion, and you again have to do a huge amount of work. It's counterproductive. David _______________________________________________ darcs-users mailing list [email protected] http://lists.osuosl.org/mailman/listinfo/darcs-users
