Neil Bird wrote on Tue, Feb 08, 2011 at 08:39:04 +0000:
> Around about 08/02/11 01:58, Daniel Shahaf typed ...
>> I'm concerned; that doesn't sound like a good process to develop
>> a patch.  Normally backporting a patch is a matter of finding
>> N applicable revisions and merging them... but it sounds that here
>> you're re-developing the feature from scratch.
>
>   Which is why this route wasn't my first choice.  The required changes  
> look as if they're too far down to line to apply separately;  they 
> require previous, possibly unrelated, revisions.
>
>   I'll take the time later to see if I can figure out exactly how many  
> trunk revisions I'd need.
>

Thanks.  However, to clarify, I'm not specifically interested in the
"give us N revisions" form; I'm just interested in seeing a coherent
patch at the end, and wanted to ensure you haven't forgotten that in
depth of coding.

>
>> If it won't compile on windows, for example, I'd be concerned that you
>> introduced run-time bugs on unix.
>
>   I poorly phrased it;  all I meant was, I'm likely going to be unable to 
> verify that it builds on Windows.  The code's actually not as ifdef'd as 
> it first appears, and a quick walk-though makes me think it *should* be 
> OK.  I just won't be able to check myself.

No problem here, then.

> 

Good luck / and thanks,

Daniel

Reply via email to