Closed
Bug 599693
Opened 14 years ago
Closed 14 years ago
Allow developers to run mochitest-1 ... mochitest-5 by simply specifying those as a make target
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla2.0b7
People
(Reporter: Callek, Assigned: Callek)
Details
(Keywords: dev-doc-complete)
Attachments
(2 files, 4 obsolete files)
610 bytes,
patch
|
kairo
:
review+
|
Details | Diff | Splinter Review |
783 bytes,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
allow developers to specify: |make mochitest-1| |make mochitest-2| etc. to mimic the buildbot behavior rather than needing to run the whole suite or dig up the buildbot runtests.py syntax when they need to reproduce an issue found by try.
Attachment #478613 -
Flags: review?(khuey)
Comment on attachment 478613 [details] [diff] [review] v1 I think we should make the total number of chunks a variable that can be overriden (TOTAL_CHUNKS ?= 5) so that in the future it'll be trivial to move from 5 chunks to 6 on buildbot.
Attachment #478613 -
Flags: review?(khuey) → review+
Comment on attachment 478613 [details] [diff] [review] v1 Actually, r-. This will break mochitest-chrome, etc.
Attachment #478613 -
Flags: review+ → review-
Callek points out on IRC that I'm wrong about that.
Assignee | ||
Comment 4•14 years ago
|
||
address an IRC nit
Attachment #478613 -
Attachment is obsolete: true
Attachment #478615 -
Flags: review?(khuey)
Assignee | ||
Comment 5•14 years ago
|
||
Yea, this won't break mochitest-chrome as explicit targets override any wildcard ones. So we are safe there.
Assignee | ||
Comment 6•14 years ago
|
||
and after doing qref on the right patch
Attachment #478615 -
Attachment is obsolete: true
Attachment #478617 -
Flags: review?(khuey)
Attachment #478615 -
Flags: review?(khuey)
Attachment #478617 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 7•14 years ago
|
||
and let comm-central use the magic from root @objdir@
Attachment #478618 -
Flags: review?(kairo)
Assignee | ||
Comment 8•14 years ago
|
||
better patch (and it doesn't error out)
Attachment #478618 -
Attachment is obsolete: true
Attachment #478619 -
Flags: review?(kairo)
Attachment #478618 -
Flags: review?(kairo)
Comment 9•14 years ago
|
||
Comment on attachment 478619 [details] [diff] [review] comm-central patch v1.1 I wonder a bit why we can't have those targets in one list, but the main thing is that it works. :)
Attachment #478619 -
Flags: review?(kairo) → review+
Assignee | ||
Comment 10•14 years ago
|
||
(In reply to comment #9) > Comment on attachment 478619 [details] [diff] [review] > comm-central patch v1.1 > http://hg.mozilla.org/comm-central/rev/e68e000957ed I pushed this before the m-c patch, it won't break anything and doing the mochitest-1 .. 5 attempt prior to m-c patch landing will now just error in the m-c makefile rather than the c-c one. That said, the reason this needs a new target-line is that we can't mix explicit and implicit rules. (using % makes this an implicit rule)
Comment 11•14 years ago
|
||
The only downside here is that this will let you run "make mochitest-anything" and it will use that rule. You could write this as a static pattern rule, instead, which would only match explicitly what you tell it to, like: mochitest-1 mochitest-2 mochitest-3 mochitest-4 mochitest-5: mochitest-%: echo "mochitest: $* / 5" then only mochitest-1 to mochitest-5 would be accepted.
Comment 12•14 years ago
|
||
(In reply to comment #10) > http://hg.mozilla.org/comm-central/rev/e68e000957ed Backed out in http://hg.mozilla.org/comm-central/rev/25d8cbc32bcd
Assignee | ||
Comment 13•14 years ago
|
||
Address teds nits. Since ted wanted me to hard-code the acceptable "this chunk" part, I took out the total-chunks var. (no sense in allowing someone to set 6 chunks, but only go up to 5). Requesting a formal review. [the concept/patch still has r=khuey]
Attachment #478617 -
Attachment is obsolete: true
Attachment #478993 -
Flags: review?(ted.mielczarek)
Assignee | ||
Comment 14•14 years ago
|
||
Comment on attachment 478619 [details] [diff] [review] comm-central patch v1.1 re-landed as: http://hg.mozilla.org/comm-central/rev/74ab84b7da92
Assignee | ||
Updated•14 years ago
|
Keywords: dev-doc-needed
Comment 15•14 years ago
|
||
Comment on attachment 478993 [details] [diff] [review] v3 You don't need the echo, that was just me being lazy and not wanting to copy the full command there.
Attachment #478993 -
Flags: review?(ted.mielczarek) → review+
Assignee | ||
Comment 16•14 years ago
|
||
(In reply to comment #15) > Comment on attachment 478993 [details] [diff] [review] > v3 http://hg.mozilla.org/mozilla-central/rev/decd6313265b [per IRC keeping the echo was ok]
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Target Milestone: --- → mozilla2.0b8
Updated•14 years ago
|
Severity: normal → enhancement
Flags: in-testsuite-
OS: Windows XP → All
Hardware: x86 → All
Updated•14 years ago
|
Target Milestone: mozilla2.0b8 → mozilla2.0b7
Comment 17•13 years ago
|
||
Added a section to the documentation here: https://developer.mozilla.org/en/Mochitest#Running_specific_tests_in_Firefox_4_or_later Would appreciate a quick review; it's only a couple of short paragraphs.
LGTM. Thanks Sheppy!
Keywords: dev-doc-needed → dev-doc-complete
I s/Firefox 4/Gecko 2/'d since this is core functionality.
Assignee | ||
Comment 20•13 years ago
|
||
sheppy, unrelated but your template stuff seems to have broke the anchor link...
Comment 21•13 years ago
|
||
That's only because the name changed; the link is now: https://developer.mozilla.org/en/Mochitest#Running_specific_tests_in_Gecko_2.0_or_later
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•