From: Reini U. <ru...@x-...> - 2006-07-01 10:17:34
Attachments:
Win32-GUI-Scintilla.diff
|
Attached are some minor cygwin path fixes for Scintilla -- Reini Urban http://phpwiki.org/ http://murbreak.at/ http://helsinki.at/ http://spacemovie.mur.at/ |
From: Reini U. <ru...@x-...> - 2006-07-01 10:26:19
Attachments:
Win32-GUI-Scintilla.diff
|
Attached are some minor cygwin path fixes for Scintilla. Note that Include/SciLexer.dll must have the executable +x flag in the filesystem, otehrwise LoadLibrary in cygwin will fail. I did cvs up, so it's wrong in cvs at sf.net. But not that important. Just be sure that the final .tar.gz will have the +x bit set for this file. -- Reini Urban http://phpwiki.org/ http://murbreak.at/ http://helsinki.at/ http://spacemovie.mur.at/ |
From: Robert M. <rm...@po...> - 2006-07-16 13:11:55
|
Reini Urban wrote: > Attached are some minor cygwin path fixes for Scintilla. Thanks for the pointers. I didn't like the call to the external process, or having to remove the taint check from the test, so I've just committed a different fix for this problem. Win32::GUI::LoadLibrary() has been enhanced to: (1) Convert '/' to '\' - as (on win98 at least) LoadLibrary doesn't like Unix separators. (2) call cygwin_convert_to_win32_path() under Cygwin (this is pretty much what cygpath does with the -w option) (3) now returns undef (rather than 0) on error > Note that Include/SciLexer.dll must have the executable +x flag in the > filesystem, otehrwise LoadLibrary in cygwin will fail. > > I did cvs up, so it's wrong in cvs at sf.net. But not that important. > Just be sure that the final .tar.gz will have the +x bit set for this file. I don't know how to fix this is CVS, but I can confirm that if I do nmake tardist and then extract the resulting tar.gz file in my cygwin environment that the execute bit is correctly set on SciLexer.dll Regards, Rob. -- Robert May Win32::GUI, a perl extension for native Win32 applications http://perl-win32-gui.sourceforge.net/ |
From: Reini U. <ru...@x-...> - 2006-07-17 06:36:35
|
2006/7/16, Robert May <rm...@po...>: > Reini Urban wrote: > > Note that Include/SciLexer.dll must have the executable +x flag in the > > filesystem, otehrwise LoadLibrary in cygwin will fail. > > > > I did cvs up, so it's wrong in cvs at sf.net. But not that important. > > Just be sure that the final .tar.gz will have the +x bit set for this file. > > I don't know how to fix this is CVS, The only way to fix wrong file perms in CVS is to remove the file and resubmit with the correct perms. Or ask the cvs admin to fix in the filesystem. A nuisance. It's not that important. I just wanted to tell you to watch perms before cvs add a new file. > but I can confirm that if I do > nmake tardist > and then extract the resulting tar.gz file in my cygwin environment that > the execute bit is correctly set on SciLexer.dll Thanks. |
From: Robert M. <rm...@po...> - 2006-07-17 23:20:32
|
Reini Urban wrote: > 2006/7/16, Robert May <rm...@po...>: >> Reini Urban wrote: >>> Note that Include/SciLexer.dll must have the executable +x flag in the >>> filesystem, otehrwise LoadLibrary in cygwin will fail. >>> >>> I did cvs up, so it's wrong in cvs at sf.net. But not that important. >>> Just be sure that the final .tar.gz will have the +x bit set for this file. >> I don't know how to fix this is CVS, > > The only way to fix wrong file perms in CVS is to remove the file and resubmit > with the correct perms. Or ask the cvs admin to fix in the filesystem. > A nuisance. It's not that important. I've made a note, and will ask the SF techs to change this next time I have reason to ask them to tinker with our repository. Regards, Rob. |