Closed
Bug 1197829
Opened 9 years ago
Closed 9 years ago
[mach try] Don't invoke build system if no paths or tags specified
Categories
(Testing :: General, defect)
Testing
General
Tracking
(firefox43 fixed)
RESOLVED
FIXED
mozilla43
Tracking | Status | |
---|---|---|
firefox43 | --- | fixed |
People
(Reporter: ahal, Assigned: chmanchester)
References
Details
Attachments
(1 file)
The biggest downside to |mach try| is that you need to have an objdir to make use of the test path feature. This negates one of the benefits of try, the fact that you don't need to build locally to test things out.
In the long run, it would be cool if we could somehow resolve tests just using a srcdir (not sure if this is possible or not). In the short term, it would be nice if invocations that just use vanilla try parameters don't interact with the build system at all.
Comment 1•9 years ago
|
||
I just hit this today, this would have been useful! I think you *could* resolve tests using a srcdir if you were willing to evaluate all the moz.build files locally. That would be faster than doing an actual build, so might be worthwhile. In the short term just skipping this step if you don't specify test paths would be great.
Assignee | ||
Comment 2•9 years ago
|
||
Resolving tests without a build is something I tried to solve over in bug 1184405 without much success, but should ultimately be doable. Having a way to run tests without re-building has been requested before.
But I've been burned by this a few times too, let's go with the straightforward patch for now.
Assignee: nobody → cmanchester
Assignee | ||
Comment 3•9 years ago
|
||
Bug 1197829 - Don't invoke the build system if paths or tags aren't passed to mach try. r=ahal
Attachment #8652623 -
Flags: review?(ahalberstadt)
Reporter | ||
Comment 4•9 years ago
|
||
Comment on attachment 8652623 [details]
MozReview Request: Bug 1197829 - Don't invoke the build system if paths or tags aren't passed to mach try. r=ahal
https://reviewboard.mozilla.org/r/17237/#review15359
Ship It!
Attachment #8652623 -
Flags: review?(ahalberstadt) → review+
Comment 6•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox43:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla43
You need to log in
before you can comment on or make changes to this bug.
Description
•