<div>Hi,</div>
<div> </div>
<div>I'm not seeing a response to this.  May I check in this hack, or get enough info to do a better one?</div>
<div> </div>
<div>Without it, I get about 22 test failures on Windows.</div>
<div> </div>
<div>Thanks.</div>
<div> </div>
<div>-John<br><br></div>
<div class="gmail_quote">On Wed, Mar 16, 2011 at 11:40 AM, John Thompson <span dir="ltr"><<a href="mailto:john.thompson.jtsoftware@gmail.com">john.thompson.jtsoftware@gmail.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">
<div>Sorry, I should have looked into it further, but I didn't want to start hacking yet.  The real problem is that the path doesn't have the .exe suffix, which if checked by adding this hack, seems to work:</div>

<div> </div>
<div>Index: utils/not/not.cpp<br>===================================================================<br>--- utils/not/not.cpp (revision 127522)<br>+++ utils/not/not.cpp (working copy)<br>@@ -15,6 +15,11 @@<br> int main(int argc, const char **argv) {<br>
   sys::Path Program = sys::Program::FindProgramByName(argv[1]);<br> <br>+#if defined(_MSC_VER) <br>+  if (strstr(Program.c_str(), ".exe") == NULL)<br>+    Program.appendSuffix("exe");<br>+#endif<br>+<br>
   std::string ErrMsg;<br>   int Result = sys::Program::ExecuteAndWait(Program, argv + 1, 0, 0, 0, 0,<br>                                             &ErrMsg);<br></div>
<div>What should the real fix be?  Is it a problem of something higher in the test system or cmake not passing in the .exe extension, or should something inside Program be doing it?</div>
<div> </div>
<div>Without the extension, the canExecute call in Program.inc fails, and commenting that out, other stuff does too.</div>
<div> </div><font color="#888888">
<div>-John<br><br></div></font>
<div>
<div></div>
<div class="h5">
<div class="gmail_quote">On Wed, Mar 16, 2011 at 2:26 AM, Csaba Raduly <span dir="ltr"><<a href="mailto:rcsaba@gmail.com" target="_blank">rcsaba@gmail.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">Hi John,<br>
<div><br>On Wed, Mar 16, 2011 at 12:43 AM, John Thompson  wrote:<br>> I'm trying to figure out why a bunch of tests are failing on Windows, and<br>> the first thing I see is that the "not" program from utils/not reports that<br>
> the path given to it is not executable, which is because of forward slashes<br>> in the Path object passed around, which nearly every Windows call that takes<br>> a path will choke on.<br><br></div>That is not quite true. The Windows _API_ can cope just fine with<br>
forward slashes. It is cmd.exe which uses forward slashes for options<br>and therefore requires backslashes in the paths.<br><br><br>Csaba<br><font color="#888888">--<br>GCS a+ e++ d- C++ ULS$ L+$ !E- W++ P+++$ w++$ tv+ b++ DI D++ 5++<br>
The Tao of math: The numbers you can count are not the real numbers.<br>Life is complex, with real and imaginary parts.<br>"Ok, it boots. Which means it must be bug-free and perfect. " -- Linus Torvalds<br>"People disagree with me. I just ignore them." -- Linus Torvalds<br>
</font></blockquote></div><br><br clear="all"><br></div></div>
<div>
<div></div>
<div class="h5">-- <br>John Thompson<br><a href="mailto:John.Thompson.JTSoftware@gmail.com" target="_blank">John.Thompson.JTSoftware@gmail.com</a><br><br></div></div></blockquote></div><br><br clear="all"><br>-- <br>John Thompson<br>
<a href="mailto:John.Thompson.JTSoftware@gmail.com">John.Thompson.JTSoftware@gmail.com</a><br><br>