I stated that the code only worked for Linux so far. Porting this to Windows up front would have a) made the example less clear and b) delayed getting initial feedback.
I already know about the ';', a bad call to env.py, a portability bug in env.py (after fixing the former), and the need to deal with different library "locations" such as RelWithDebInfo. Presumably after addressing those, a little more quicksand will need wading through too. I'm working on it. On Thu, Feb 14, 2013 at 10:54 AM, Mary Hinton <m.hin...@nc.rr.com> wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9433/#review16596 > ----------------------------------------------------------- > > > I put your changes in my code and ran it. > I was trying to figure out why it is failing, but haven't had any success, so > I'll just include the result here. > c:\proton\trunk>ctest -V > UpdateCTestConfiguration from :C:/proton/trunk/DartConfiguration.tcl > Parse Config file:C:/proton/trunk/DartConfiguration.tcl > UpdateCTestConfiguration from :C:/proton/trunk/DartConfiguration.tcl > Parse Config file:C:/proton/trunk/DartConfiguration.tcl > Test project C:/proton/trunk > Constructing a list of tests > Done constructing a list of tests > Checking test dependency graph... > Checking test dependency graph end > test 1 > Start 1: proton-c > > 1: Test command: c:\Python27\python.exe "C:/proton/trunk/proton-c/env.py" > "PYTHONPATH=C:/proton/trunk/te > ts/python:C:/proton/trunk/proton-c/bindings/python:C:/proton/trunk/proton-c/bindings/python" > "C:/proton/ > runk/tests/python/proton-test" > 1: Test timeout computed to be: 1500 > 1: Traceback (most recent call last): > 1: File "C:/proton/trunk/proton-c/env.py", line 64, in <module> > 1: sys.exit(main()) > 1: File "C:/proton/trunk/proton-c/env.py", line 60, in main > 1: os.execvpe(args[0], args, new_env) > 1: File "c:\Python27\lib\os.py", line 353, in execvpe > 1: _execvpe(file, args, env) > 1: File "c:\Python27\lib\os.py", line 368, in _execvpe > 1: func(file, *argrest) > 1: OSError: [Errno 2] No such file or directory > 1/1 Test #1: proton-c .........................***Failed Required regular > expression not found.Regex=[T > tals:.*, 0 failed > ] 0.44 sec > > 0% tests passed, 1 tests failed out of 1 > > Total Test time (real) = 0.69 sec > > The following tests FAILED: > 1 - proton-c (Failed) > Errors while running CTest > > I tried the python line substituting a ';' for the ':' in the PYTHONPATH, but > same results. > Not sure what is missing. > > > - Mary Hinton > > > On Feb. 13, 2013, 7:41 p.m., Cliff Jansen wrote: >> >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> https://reviews.apache.org/r/9433/ >> ----------------------------------------------------------- >> >> (Updated Feb. 13, 2013, 7:41 p.m.) >> >> >> Review request for qpid and Mary Hinton. >> >> >> Description >> ------- >> >> The smallest useful example I could devise to illustrate the possibilities. >> >> See https://issues.apache.org/jira/browse/PROTON-238 for more info >> >> >> This addresses bug PROTON-238. >> https://issues.apache.org/jira/browse/PROTON-238 >> >> >> Diffs >> ----- >> >> http://svn.apache.org/repos/asf/qpid/proton/trunk/CMakeLists.txt 1445761 >> http://svn.apache.org/repos/asf/qpid/proton/trunk/proton-c/CMakeLists.txt >> 1445761 >> >> Diff: https://reviews.apache.org/r/9433/diff/ >> >> >> Testing >> ------- >> >> linux only so far. Minimally the PYTHONPATH will require a ';' separator in >> Windows. >> >> >> Thanks, >> >> Cliff Jansen >> >> > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org
