Closed
Bug 601113
Opened 14 years ago
Closed 14 years ago
Port |Bug 543800 - package chrome style tests into .jar file for local testing| to comm-central
Categories
(MailNews Core :: Build Config, defect)
MailNews Core
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3.3a1
People
(Reporter: sgautherie, Assigned: sgautherie)
References
Details
(Keywords: regression)
Attachments
(2 files, 2 obsolete files)
1.12 KB,
patch
|
Callek
:
review+
|
Details | Diff | Splinter Review |
1.69 KB,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
SeaMonkey hnightlies are failing with { make -C mozilla package-tests make[1]: Entering directory `/e/builds/slave/comm-central-trunk-win32-nightly/build/objdir/mozilla' make[1]: Leaving directory `/e/builds/slave/comm-central-trunk-win32-nightly/build/objdir/mozilla' make[1]: *** No rule to make target `package-tests'. Stop. }
Comment 1•14 years ago
|
||
Callek did a bugfix for that in http://hg.mozilla.org/comm-central/rev/3b49ae48e232, but I don't know why this didn't work for our nightly builders. I can't reproduce this locally.
Assignee | ||
Comment 2•14 years ago
|
||
(In reply to comment #1) > http://hg.mozilla.org/comm-central/rev/3b49ae48e232 "[Attempt to] Fix OSX bustage from m-c Bug 543800" Indeed, it seems to have already fixed usual tinderbox builds :-) But that was only a part of the needed port...
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Flags: in-testsuite-
Assignee | ||
Comment 3•14 years ago
|
||
Untested, but that should be it. (for the common part at least)
Attachment #480098 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 4•14 years ago
|
||
Untested, but that should be it.
Attachment #480102 -
Flags: review?(bugzilla)
Assignee | ||
Comment 5•14 years ago
|
||
Cv1-TB, with testsuite-targets.mk part included.
Attachment #480102 -
Attachment is obsolete: true
Attachment #480104 -
Flags: review?(bugzilla)
Attachment #480102 -
Flags: review?(bugzilla)
Assignee | ||
Updated•14 years ago
|
Blocks: SmTestFail
Comment 7•14 years ago
|
||
(In reply to comment #3) > Created attachment 480098 [details] [diff] [review] > (Bv1) (Missed) Makefile.in part I don't think our problem here isn't above all the Makefile.in, but that we call make package-tests if tests are disabled. With the patch, the call just fails earlier, because we don't have the target in our Makefile anymore. So either we can configure buildbot that the target isn't called at all for disabled tests or we have to patch our Makefile.in to be something like that: ifndef MOZ_THUNDERBIRD package-tests:: $(SUBMAKEFILES) ifdef ENABLE_TESTS $(MAKE) -C mozilla $@ endif endif This would retain the target but only do something, if tests are enabled.
Assignee | ||
Comment 8•14 years ago
|
||
Per http://mxr.mozilla.org/comm-central/search?string=%3A%2F%2Fmochikit%2Fcontent%2Fbrowser&case=1&find=%2Fsuite%2F.*test.*%5C.js%24
Attachment #480187 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 9•14 years ago
|
||
(In reply to comment #7) > we call make package-tests if tests are disabled. Yeah, that's a different issue: > So either we can configure buildbot that the target isn't called at all for That's what I would (have) expect actually! Yet, bug 545172 should enable tests now :-) Then we shouldn't care anymore. But you can file a bug about that if you want. > disabled tests or we have to patch our Makefile.in to be something like that: I fully agree with this workaround, but only if bug 545172 is refused for some reason.
Comment 10•14 years ago
|
||
(In reply to comment #9) > Yet, bug 545172 should enable tests now :-) > Then we shouldn't care anymore. But you can file a bug about that if you want. If we get tests enabled again, that would be the best solution, of course :-)
Assignee | ||
Updated•14 years ago
|
Attachment #480187 -
Attachment description: (Dv1-SM) Fix mochitest-browser-chrome tests: s|chrome://mochikit/content|http://mochi.test:8888| → (Dv1-SM) Fix mochitest-browser-chrome tests: s|chrome://mochikit/content|http://mochi.test:8888|
[Superseded by bug 601069]
Attachment #480187 -
Attachment is obsolete: true
Attachment #480187 -
Flags: review?(bugspam.Callek)
Assignee | ||
Updated•14 years ago
|
No longer blocks: SmTestFail
Updated•14 years ago
|
Attachment #480104 -
Flags: review?(bugzilla) → review+
Assignee | ||
Comment 11•14 years ago
|
||
Comment on attachment 480104 [details] [diff] [review] (Cv1a-TB) Just port it [Checked in: Comment 11] http://hg.mozilla.org/comm-central/rev/7a7aa0052f04
Attachment #480104 -
Attachment description: (Cv1a-TB) Just port it → (Cv1a-TB) Just port it
[Checked in: Comment 11]
Comment 12•14 years ago
|
||
Comment on attachment 480098 [details] [diff] [review] (Bv1) (Missed) Makefile.in part [Checked in: Comment 13] sure. [I'll get Bug 601069 review in a day or two, real life is crowding my time]
Attachment #480098 -
Flags: review?(bugspam.Callek) → review+
Assignee | ||
Comment 13•14 years ago
|
||
Comment on attachment 480098 [details] [diff] [review] (Bv1) (Missed) Makefile.in part [Checked in: Comment 13] http://hg.mozilla.org/comm-central/rev/b776fde61d23
Attachment #480098 -
Attachment description: (Bv1) (Missed) Makefile.in part → (Bv1) (Missed) Makefile.in part
[Checked in: Comment 13]
Assignee | ||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.3a1
You need to log in
before you can comment on or make changes to this bug.
Description
•