> Usually when I replace a patch, I click the little box that says > "replace earlier version?" (There may be pros and cons for this.)
The first time, I forgot to click the box - and then I thought it might be good to keep track of the file history... > The other thing I do (since once someone complained) is to make > it very clear in the description what needs to be applied, so when > I reviewed the patch yesterday I also modified the description to say > that trac_9648 was a prerequisite. You're right, when I first wrote the patch, it depended on 9648, but you could still have apply it without. This changed later and I missed to update the description to make this clear. Thanks for the positive review, Christian -- You received this message because you are subscribed to the Google Groups "sage-combinat-devel" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/sage-combinat-devel?hl=en.
