From: Bron Gondwana (no email)
Date: Wed Nov 21 2007 - 04:11:56 EST
On Wed, 21 Nov 2007 07:40:21 +0100 (CET), "Simon Matter" <> said:
> Hi Bron,
>
> Did you consider this one
> http://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3006 in the patch above?
> From a quick look it seems both patches conflict, is #3006 obsolete now?
You're right, they probably do conflict and Ken has applied 3006. I should
have checked on that. Hmm...
Actually, my patch doesn't even fix #3006. It does sort of suck to be working
from a stable past point rather than the latest CVS actually when I do these
patches - though it's nice from a stability point of view.
OK - Ken, would you like me to re-build this patch on top of current CVS, or
would you like to do it yourself? My patch has the replication bugfixes which
are still worth doing, but Simon's logic is actually correct for the test
while mine isn't (though I renamed the function he's using...)
Bron.
-- Bron Gondwana
|
|
|