Closed Bug 843707 Opened 11 years ago Closed 11 years ago

Force run_jetpack.py to test the SDK modules

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mossop, Assigned: mossop)

Details

Attachments

(1 file)

We're adding support to the SDK so it can use the modules now shipping in Firefox. We want to make sure that run_jetpack.py ignores those modules in Firefox though so we need to add an extra flag to the command line we call cfx with.
Attached patch patch rev 1Splinter Review
Would any patches need to be uplifted to mozilla-aurora?
or the code is already there?
(In reply to Armen Zambrano G. [:armenzg] from comment #2)
> Would any patches need to be uplifted to mozilla-aurora?
> or the code is already there?

run_jetpack.py is only in use on the Jetpack tree now unless I'm forgetting something, and that tree always runs against the latest SDK version so this change should work, in fact it fixes some test failures on that tree where it is attempting to use older APIs from Firefox right now.
Comment on attachment 716668 [details] [diff] [review]
patch rev 1

Can you review this Armen?
Attachment #716668 - Flags: review?(armenzg)
Comment on attachment 716668 [details] [diff] [review]
patch rev 1

Review of attachment 716668 [details] [diff] [review]:
-----------------------------------------------------------------

It looks good to me. Would you like me to test it on staging before landing it?
Attachment #716668 - Flags: review?(armenzg) → review+
(In reply to Armen Zambrano G. [:armenzg] from comment #5)
> Comment on attachment 716668 [details] [diff] [review]
> patch rev 1
> 
> Review of attachment 716668 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> It looks good to me. Would you like me to test it on staging before landing
> it?

I don't think it's worth it, I'll just land it then I have a whitespace change to push to the SDK anyway that will trigger a full set of builds.

Thanks
It won't use the new code until the code merges from default to production on buildbot-configs.
(In reply to Armen Zambrano G. [:armenzg] from comment #7)
> It won't use the new code until the code merges from default to production
> on buildbot-configs.

In the past changes to run_jetpack.py have taken effect immediately.

https://hg.mozilla.org/build/tools/rev/6489651f9a28
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
(In reply to Dave Townsend (:Mossop) from comment #8)
> (In reply to Armen Zambrano G. [:armenzg] from comment #7)
> > It won't use the new code until the code merges from default to production
> > on buildbot-configs.
> 
> In the past changes to run_jetpack.py have taken effect immediately.
> 
> https://hg.mozilla.org/build/tools/rev/6489651f9a28

My bad. I was thinking of the wrong repo. I didn't notice that it was against tools.
Product: mozilla.org → Release Engineering
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: