Closed Bug 599653 Opened 14 years ago Closed 14 years ago

xpcshell-tests target should run extensions/mdn test

Categories

(MailNews Core :: Build Config, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.3a1

People

(Reporter: bugzilla, Assigned: bugzilla)

References

Details

Attachments

(1 file)

When the xpcshell-test target was added in bug 491376, the test in extensions/mdn has been forgotten.
Attachment #478562 - Attachment is patch: true
Attachment #478562 - Attachment mime type: application/octet-stream → text/plain
Attachment #478562 - Flags: review?(bugzilla)
Attachment #478562 - Flags: review?(bugzilla) → review+
Keywords: checkin-needed
Assignee: nobody → aqualon
Status: NEW → ASSIGNED
Comment on attachment 478562 [details] [diff] [review]
add extensions/mdn to TEST_DIRS
[Checked in: Comment 6]

http://hg.mozilla.org/comm-central/rev/ff2d6bce87ff
Attachment #478562 - Attachment description: add extensions/mdn to TEST_DIRS → add extensions/mdn to TEST_DIRS [Checked in: Comment 1]
Blocks: 491376
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3.3a1
(In reply to comment #1)
> http://hg.mozilla.org/comm-central/rev/ff2d6bce87ff

This got backed out again - please be careful with other people's bugs.

http://hg.mozilla.org/comm-central/rev/25d8cbc32bcd
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(In reply to comment #2)
> This got backed out again - please be careful with other people's bugs.

Sorry, I missed that two bugs were backed out in that changeset :-(
Status: REOPENED → ASSIGNED
Then, iiuc, this patch is fine, but needs bug 599809 workaround/fix when it is pushed...
Depends on: 599809
(In reply to comment #4)
> Then, iiuc, this patch is fine, but needs bug 599809 workaround/fix when it is
> pushed...

Or to be pushed along with any patch that will force the Makefile to be generated from c-c, such as a configure change [like one I just reviewed] ;-)
Comment on attachment 478562 [details] [diff] [review]
add extensions/mdn to TEST_DIRS
[Checked in: Comment 6]

http://hg.mozilla.org/comm-central/rev/024092e0b589
Attachment #478562 - Attachment description: add extensions/mdn to TEST_DIRS [Checked in: Comment 1] → add extensions/mdn to TEST_DIRS [Checked in: Comment 6]
Status: ASSIGNED → RESOLVED
Closed: 14 years ago14 years ago
No longer depends on: 599809
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.