> On March 9, 2015, 3:40 p.m., Alan Conway wrote: > > trunk/qpid/cpp/src/tests/CMakeLists.txt, line 149 > > <https://reviews.apache.org/r/31812/diff/1/?file=887906#file887906line149> > > > > It would be easy to modify the sh script to accept -option instead of > > --option which would avoid these shenanigens. I would personally prefer > > that but it might start some sort of religious war. At your discretion :)
Meant to say "ship it" stupid reviewboard has too many buttons that look like you should press them next. - Alan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31812/#review75705 ----------------------------------------------------------- On March 6, 2015, 11:02 p.m., Steve Huston wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31812/ > ----------------------------------------------------------- > > (Updated March 6, 2015, 11:02 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 1664535 > trunk/qpid/cpp/src/tests/run_test.ps1 1664535 > trunk/qpid/cpp/src/tests/test_env.ps1.in 1664535 > > 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. > > > Thanks, > > Steve Huston > >
