----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31812/#review76732 -----------------------------------------------------------
Ship it! Ship It! - Andrew Stitcher On March 16, 2015, 9:39 p.m., Steve Huston wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31812/ > ----------------------------------------------------------- > > (Updated March 16, 2015, 9:39 p.m.) > > > Review request for qpid, Alan Conway and Cliff Jansen. > > > Bugs: QPID-6399 > https://issues.apache.org/jira/browse/QPID-6399 > > > Repository: qpid > > > Description > ------- > > As noted in QPID-6399, the way run_tests takes in test info and runs test > programs has changed and run_test.ps1 needs to be changed to match. To do > this I: > - Added parameters to the run_test.ps1 script to easily take the command line > arguments. The way powershell does this is, of course, different from > anything else. They use one dash instead of two and there's no concept of > long vs. short names. Thus, the way to specify the arguments is different > between Linux and Windows. > - Changed CMakeLists.txt to adapt to the different argument-passing > mechanisms. > - Added some help to test_env.ps1.in to help set up run-time paths. > > > Diffs > ----- > > trunk/qpid/cpp/src/tests/CMakeLists.txt 1666768 > trunk/qpid/cpp/src/tests/run_test 1666768 > trunk/qpid/cpp/src/tests/run_test.ps1 1666768 > trunk/qpid/cpp/src/tests/test_env.ps1.in 1666768 > > Diff: https://reviews.apache.org/r/31812/diff/ > > > Testing > ------- > > Run all the cpp/src/tests on Windows. They all fail, but they do at least run. > > Ran new run_test script on Linux also (RHEL 5) > > > Thanks, > > Steve Huston > >
