|
I almost never think it's worth running tests with -O, so I'm all for
removing it. Heikki Toivonen wrote: We've recently had a couple of examples where developers run tests as expected, but they ran the tests with -O (by default release RunChandler includes that). Then when Tbox runs the debug cycle (without -O) it notices some tests break because asserts fire.Bryan suggested that we change do_tests.sh to run without -O. I think the optimal solution with do_tests.sh would be: - if it is running both debug and release tests let it run with defaults (debug runs without -O, release runs with it) - if you test just debug or just release, run without -O This way you will notice AssertionErrors being raised, and assert commands firing in general. |
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ Open Source Applications Foundation "chandler-dev" mailing list http://lists.osafoundation.org/mailman/listinfo/chandler-dev
