2011/4/2 Jan Dubois <jand@activestate.com>: > On Fri, 01 Apr 2011, Zefram wrote: >> >> Jerry D. Hedden wrote: >> >Yes, the problem still exists. >> >> Unless we have a Win32 linker wizard in the house, I suggest that we >> should just skip that part of the test on Win32. > > I assume you meant Cygwin, not Win32. > > I have fixed this same issue for MSWin32 with commit ad6ab6c50, > and I suspect that the Cygwin side might be fixed by moving > the change from win32/win32.h into perl.h: > > http://perl5.git.perl.org/perl.git/commitdiff/ad6ab6c50 > > of course wrapped with: > > #if defined(__CYGWIN__) || defined(WIN32) > #endif > > It needs confirmation from Jerry and/or Reini that this actually solves > the problem before we bother asking Jesse if he prefers the change, or > the skipping of the test for 5.14. Oops. This is a rather tricky change, which will need some testing time I do not have in the moment. I'm on darwin right now... Jerry, can you do this? We would also need some embed tests, and I'm not sure if the core suite does test it enough. -- Reini UrbanThread Previous | Thread Next