Closed Bug 523330 Opened 13 years ago Closed 13 years ago

default symbols path busted for mochitest/reftest


(Testing :: Mochitest, defect)

Not set


(Not tracked)



(Reporter: ted, Assigned: ted)




(1 file)

When I landed a fix for bug 519194, I fixed the packaged tests case and broke the in-tree case. I guess that was an ok tradeoff at the time because we're not running in-tree tests on tinderbox anymore (but still not good!)

The problem is that SYMBOLS_PATH as preprocessed into is a path relative to the script directory (where winds up), but we resolve it relative to the cwd (because we want to handle paths passed to --symbols-path, which would be relative to the cwd).

I think I might fix this by just removing SYMBOLS_PATH, and having the "mochitest/reftest" makefile targets pass --symbols-path in.
This makes SYMBOLS_PATH no longer a hardcoded symbol from the preprocessor, instead forcing everyone to pass --symbols-path in explicitly. It also makes all the mochitest-*, reftest etc targets pass that argument in. The only negative impact here is that people who want to run the python scripts directly and want crash stacks will have to explicitly pass the argument, but I don't think that's a big deal. We should be discouraging running the python scripts directly anyway, in favor of the makefile targets.
Attachment #407508 - Flags: review?(benjamin)
Attachment #407508 - Flags: review?(benjamin) → review+
This patch works great for me in bug 483111, for the in-tree case.
Pushed to m-c:
Closed: 13 years ago
Resolution: --- → FIXED
Depends on: 524125
You need to log in before you can comment on or make changes to this bug.