| 1 |
On 04/27/2012 01:17 PM, Mike Gilbert wrote: |
| 2 |
> On 04/27/2012 12:35 PM, Dirkjan Ochtman wrote: |
| 3 |
>>>> 3. Added 61_all_process_data.patch, for which the goal seems somewhat unclear. |
| 4 |
>>> This is some logic for python-wrapper that was in the 2.7.2 patchset as |
| 5 |
>>> well. If you want to drop it, I'm sure that will require some |
| 6 |
>>> re-engineering of python-wrapper. |
| 7 |
>> |
| 8 |
>> Right. Do you know what this fixes, and how/why it will be |
| 9 |
>> re-engineered? As long as we don't know these things, I'd prefer to |
| 10 |
>> leave the patch out rather than include it. |
| 11 |
>> |
| 12 |
> |
| 13 |
> I have been bitten in the past by removing things that I don't fully |
| 14 |
> understand, so I'm trying to avoid doing that. I guess I can drop it and |
| 15 |
> see what happens. |
| 16 |
> |
| 17 |
|
| 18 |
I will add investigating this patch to my todo list. Hopefully Arfrever |
| 19 |
can explain the rationale to me, and I can relay that. |
| 20 |
|
| 21 |
Otherwise, I will have to figure it out the hard way. |
Archives