Closed Bug 503886 Opened 10 years ago Closed 10 years ago

Allow non-mozilla central applications to use automation-build.mk

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(status1.9.1 .2-fixed)

RESOLVED FIXED
mozilla1.9.2a1
Tracking Status
status1.9.1 --- .2-fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(2 files)

I am trying to hook up a new makefile driver test harness (at least initially) for Thunderbird. As part of that I want to use automation.py - which means the best way is for me to include automation-build.mk.

Unfortunately automation-build.mk is using $(topsrcdir) which means I can't include it. As it is always used after rules.mk, we can swap it to use $(MOZILLA_DIR) which will point to the mozilla source directory whether in the mozilla-central or comm-central build systems.

Trunk and branch are a bit different so I'll attach a patch for branch in a bit.
Attachment #388235 - Flags: review?(benjamin)
(In reply to comment #0)
> Created an attachment (id=388235) [details]
> Trunk patch

Forgot to say, I ran this patch through try server with no problems.
Branch version of the patch.
Attachment #388244 - Flags: review?(benjamin)
Attachment #388235 - Flags: review?(benjamin) → review+
Attachment #388244 - Flags: review?(benjamin) → review+
Attachment #388235 - Attachment description: Trunk patch → [checked in] Trunk patch
Comment on attachment 388235 [details] [diff] [review]
[checked in] Trunk patch

Checked into trunk:

http://hg.mozilla.org/mozilla-central/rev/465f87d82960
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Comment on attachment 388244 [details] [diff] [review]
1.9.1 branch patch

Not 100% sure if I need approval of this or not, so requesting just in case.

This patch changes the file that generates automation.py so that it can be generated and used non-mozilla-central specific testing code.

Thunderbird wants this for some of its testing code.

AFAIK this code is only used in testing the trunk builders have been unaffected by the trunk version of the patch.
Attachment #388244 - Flags: approval1.9.1.2?
Comment on attachment 388244 [details] [diff] [review]
1.9.1 branch patch

This is NPOTB, doesn't need approval from drivers.
What is the best/simplest way for QA to verify this for Firefox 3.5.2?
I guess all the test suites that use it are still running e.g. mochitest xpcshell etc etc (oh and that Thunderbird is able to run MozMill which uses it, which can be found here: http://tinderbox.mozilla.org/showbuilds.cgi?tree=ThunderbirdTest, ignore the red/orange, there's some passed tests on there somewhere!).
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.