Closed Bug 664836 Opened 13 years ago Closed 13 years ago

Remove old mobile code.

Categories

(Release Engineering :: General, defect)

defect
Not set
minor

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jhford, Assigned: jhford)

References

Details

Attachments

(2 files)

Once fennec 5 ships, we don't need to have fennec <=4 automation on production-0.8 or in the /mozilla/ configs.
Kill!
Attached patch buildbot-configsSplinter Review
woo
Attachment #540799 - Flags: review?(aki)
Attached patch buildbotcustomSplinter Review
hoo
Attachment #540801 - Flags: review?(aki)
Attachment #540799 - Flags: review?(aki) → review+
Attachment #540801 - Flags: review?(aki) → review+
I think this broke mobile builds on mozilla-2.0. My push (the first one since this was merged on the 23rd) failed all of them:
http://tbpl.mozilla.org/?tree=Firefox4.0&rev=5216dd412535
Specifically, they failed with:
cp: cannot stat `configs/mozilla2/linux-mobile/mozilla-2.0/nightly/mozconfig': No such file or directory
Actually, you didn't touch those mozconfigs...filed bug 667918.
(In reply to comment #4)
> I think this broke mobile builds on mozilla-2.0. My push (the first one
> since this was merged on the 23rd) failed all of them:
> http://tbpl.mozilla.org/?tree=Firefox4.0&rev=5216dd412535

There should be no mobile builds on mozilla-2.0.
(In reply to comment #7)
> (In reply to comment #4)
> > I think this broke mobile builds on mozilla-2.0. My push (the first one
> > since this was merged on the 23rd) failed all of them:
> > http://tbpl.mozilla.org/?tree=Firefox4.0&rev=5216dd412535
> 
> There should be no mobile builds on mozilla-2.0.

yes, there was a bug in the removal patch where the locked_platforms variable was commented out.  it is fixed on default and the fix is being deployed now.
aside from fallout addressed in 667918, this work is done.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Backed out the mozconfig removal, only, since this was breaking 6.0b1 0.7 release builds.
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: