On Tue, May 08, 2012 at 07:47:48AM +0200, Steffen Mueller wrote: > On 05/06/2012 05:58 AM, Christopher J. Madsen via RT wrote: > > On Sat May 05 20:53:41 2012, jkeenan wrote: > >> Please review the patch attached. > > > > Well, I can confirm that it solves my problem, but I don't know enough > > about ExtUtils::ParseXS to be sure it's the right solution, or if it's > > just masking the real problem. > > That is precisely why I hadn't included a simple change like the > proposed patch yet -- I am also not entirely certain about whether it's > a fix or just plaster. Jim's finding of key (var name) mismatches does > make it sound like there's a deeper issue. > > I'm on the fence on whether it's better to apply the patch to avoid > confusing users or whether it's better not to in the hope that one of > the affected users cares as much as Christopher and figures it out. > Alas, I don't currently have the time to dig in myself. I fear that it's plaster, and hence applying it does nothing to solve the actual problem. Nicholas ClarkThread Previous | Thread Next