Closed
Bug 538511
Opened 14 years ago
Closed 14 years ago
release automation updates for maemo5
Categories
(Release Engineering :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Assigned: mozilla)
References
Details
(Whiteboard: [maemo5][q2goal])
Attachments
(3 files)
1023 bytes,
patch
|
jhford
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
29.18 KB,
patch
|
jhford
:
review+
mozilla
:
checked-in+
|
Details | Diff | Splinter Review |
2.26 KB,
text/x-python-script
|
Details |
Hoping this is close to Maemo4, but it won't be free.
Assignee | ||
Updated•14 years ago
|
Assignee: aki → nobody
Priority: -- → P3
Updated•14 years ago
|
Priority: P3 → --
Updated•14 years ago
|
Priority: -- → P3
Assignee | ||
Updated•14 years ago
|
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → aki
Assignee | ||
Updated•14 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•14 years ago
|
||
Tested my patches on sm02; I'll clean those up for review after jhford lands his massive patch tomorrow. One thing I noticed, in the partner repack (which is busted for maemo5-gtk atm because of this) is that maemo5-gtk puts its bits in /usr/local/lib instead of /opt/mozilla. I think this is a Fennec bug, but I need to confirm.
Comment 2•14 years ago
|
||
(In reply to comment #1) > Tested my patches on sm02; I'll clean those up for review after jhford lands > his massive patch tomorrow. > > One thing I noticed, in the partner repack (which is busted for maemo5-gtk atm > because of this) is that maemo5-gtk puts its bits in /usr/local/lib instead of > /opt/mozilla. > > I think this is a Fennec bug, but I need to confirm. Looks like the mozconfigs are missing the install location lines, so we are defaulting: http://hg.mozilla.org/build/buildbot-configs/file/a05483b93162/mozilla2/mobile/maemo4/mobile-browser/nightly/mozconfig#l7 You need both of those lines
Assignee | ||
Comment 3•14 years ago
|
||
(In reply to comment #2) > Looks like the mozconfigs are missing the install location lines, so we are > defaulting: > http://hg.mozilla.org/build/buildbot-configs/file/a05483b93162/mozilla2/mobile/maemo4/mobile-browser/nightly/mozconfig#l7 > > You need both of those lines Aha, nice catch. Thanks, I'll add that to my patch.
Assignee | ||
Comment 4•14 years ago
|
||
Attachment #453798 -
Flags: review?(jhford)
Assignee | ||
Comment 5•14 years ago
|
||
1) add prefix+libdir to all maemo5 mozconfigs 2) add maemo5-qt/mobile-browser/release/mozconfig that is the same as the nightly, but with official branding 3) partner repack builders per platform at this stage, since trying to do maemo5-gtk and maemo4 in the same pass had some issues (same filename) Looks like I did most of the heavy lifting pre-1.1rc1; it's largely a matter of just turning these on :)
Attachment #453802 -
Flags: review?(jhford)
Assignee | ||
Comment 6•14 years ago
|
||
For reference+posterity. This is the release-fennec-2.0.py that I used for my testing. It's got Android in it, so requires the patches that I'll be posting to bug 561901 (or just remove android-r7 from the platform list). Since Android and Maemo5-qt are non-1.9.2 platforms, I built off of trunk. QT was not a partner repack (and most likely we'll only deliver one deb to partners, so the partnerRepackPlatforms will stay at 1 platform for the time being). android-r7 was left out of the l10nPlatforms til we have localized Android builds.
Updated•14 years ago
|
Attachment #453798 -
Flags: review?(jhford) → review+
Comment 7•14 years ago
|
||
Comment on attachment 453802 [details] [diff] [review] maemo5-gtk, maemo5-qt release builder configs Looks good. My only concern is the 4 (?) places where this line occurs. >+ sb_target=pf.get('sb_target', 'CHINOOK-ARMEL-2007'), I don't think we should be using a default or {}.get in this case. I would much rather find out that i didn't set a scratchbox target on checkconfig than find out we shipped a FREMANTLE_ARMEL browser to chinook users. If we really do need to have a default for some reason, I think that FREMANTLE_ARMEL is a more appropriate default. r+ if the .get( is replaced with pf[.
Attachment #453802 -
Flags: review?(jhford) → review+
Assignee | ||
Comment 8•14 years ago
|
||
Comment on attachment 453802 [details] [diff] [review] maemo5-gtk, maemo5-qt release builder configs checked in, with pf['sb_target']. http://hg.mozilla.org/build/buildbot-configs/rev/3ba766d66489
Attachment #453802 -
Flags: checked-in+
Assignee | ||
Comment 9•14 years ago
|
||
Comment on attachment 453798 [details] [diff] [review] maemo5 release repacks -- more qt tar.gz http://hg.mozilla.org/build/buildbotcustom/rev/642e73401a4a
Attachment #453798 -
Flags: checked-in+
Assignee | ||
Updated•14 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•