[webkit-reviews] review requested: [Bug 45651] jscPath() is incorrect in Windows' cmd.exe shell : [Attachment 67413] proposed fix

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Sep 13 08:24:59 PDT 2010


Csaba Osztrogonac <ossy at webkit.org> has asked  for review:
Bug 45651: jscPath() is incorrect in Windows' cmd.exe shell
https://bugs.webkit.org/show_bug.cgi?id=45651

Attachment 67413: proposed fix
https://bugs.webkit.org/attachment.cgi?id=67413&action=review

------- Additional Comments from Csaba Osztrogonac <ossy at webkit.org>
proposed fix:
+    $jscName .= ".exe" if (isWindows() || isCygwin());

The root of the problem was that ' if -e "executableBinary" ' is false 
with ActiveState perl on Windows in cmd.exe shell, but true in CygWin 
shell if executableBinary.exe exist, but executableBinary doesn't.

It is a misleading perl feature what we should avoid, so I 
propose to add ".exe" suffix for Windows and CygWin platform too.


More information about the webkit-reviews mailing list