Closed Bug 1329379 Opened 7 years ago Closed 4 years ago

[Tracking Bug] Drop OSX universal builds.

Categories

(SeaMonkey :: Build Config, defect)

defect
Not set
normal

Tracking

(seamonkey2.49esr fixed)

RESOLVED FIXED
seamonkey2.49
Tracking Status
seamonkey2.49esr --- fixed

People

(Reporter: ewong, Assigned: ewong)

References

Details

Attachments

(3 files, 1 obsolete file)

It looks like m-c has dropped Universal builds and has moved to 64 only.

:stefan  should we follow suite?
Flags: needinfo?(stefanh)
(In reply to Edmund Wong (:ewong) from comment #0)
> It looks like m-c has dropped Universal builds and has moved to 64 only.
> 
> :stefan  should we follow suite?

On second thoughts... I could be wrong as I was looking at the tooltool
part when I should've looked at the mozconfig part.  my bad
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(stefanh)
Resolution: --- → INVALID
Have a look at bug 1322402 and its dependencies (yes, m-c dropped support for universal builds in december). Not following would probably mean future breakage (for instance, iirc we have some code specific to 32-bit which probably will be removed in the future)
Status: RESOLVED → REOPENED
Flags: needinfo?(ewong)
Resolution: INVALID → ---
Summary: [Tracking Bug] Drop OSX64 universal builds. → [Tracking Bug] Drop OSX universal builds.
(In reply to Stefan [:stefanh] from comment #2)
> Have a look at bug 1322402 and its dependencies (yes, m-c dropped support
> for universal builds in december). Not following would probably mean future
> breakage (for instance, iirc we have some code specific to 32-bit which
> probably will be removed in the future)

Ah. I thought I was going nuts coz I remembered somewhere that they were
dropping -universal but I noticed the universal mozconfigs still there..
I guess I didn't follow the mozconfig line of usage.

Thanks for reopening this stefan.  Will check those bugs out.
Status: REOPENED → NEW
Flags: needinfo?(ewong)
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Attachment #8825635 - Flags: review?(philip.chee)
buildbot config changes.  we don't need the additional objdir/i386 now.
Attachment #8825636 - Flags: review?(frgrahl)
Just a note (I know it's probably a different bug, but I'm unsure where that should be filed): We need to ensure that updates to users running universal builds are handled correct (there should be a m-c bug about that that will give more info).
Comment on attachment 8825635 [details] [diff] [review]
[mozconfigs] Remove macosx-universal and move the required stuff to macosx64

r=me a=me
Attachment #8825635 - Flags: review?(philip.chee) → review+
Comment on attachment 8825636 [details] [diff] [review]
[configs] drop universal builds.

r=me.
I assume the SeaMonkey buildbot is only using the seamonkey dir? Calendar dir still contains the i386.
Attachment #8825636 - Flags: review?(frgrahl) → review+
(In reply to Frank-Rainer Grahl from comment #8)
> Comment on attachment 8825636 [details] [diff] [review]
> [configs] drop universal builds.
> 
> r=me.
> I assume the SeaMonkey buildbot is only using the seamonkey dir? Calendar
> dir still contains the i386.

Yes..  SeaMonkey buildbot infra uses only the seamonkey directory.
(In reply to Stefan [:stefanh] from comment #6)
> Just a note (I know it's probably a different bug, but I'm unsure where that
> should be filed): We need to ensure that updates to users running universal
> builds are handled correct (there should be a m-c bug about that that will
> give more info).

I'm still gathering the necessary information, and aside for not allowing
updates for users on universal release builds on c-b and c-r, I'm not
entirely sure (atm) how we'll accomplish this with c-c and c-a.

Callek, any ideas?
Flags: needinfo?(bugspam.Callek)
https://hg.mozilla.org/comm-central/rev/becf847f864ca2765095ea5f22a4cd6f99d35729
Bug 1329379 - Drop universal OSX64 builds from the mozconfigs. r+a=RattyAway
Attachment #8840765 - Flags: review?(frgrahl)
Attachment #8840765 - Flags: review?(frgrahl) → review+
Comment on attachment 8825635 [details] [diff] [review]
[mozconfigs] Remove macosx-universal and move the required stuff to macosx64

[Approval Request Comment]
Regression caused by (bug #): 1322402
User impact if declined: No OSX builds
Testing completed (on m-c, etc.): c-c
Risk to taking this patch (and alternatives if risky): none already broken
String changes made by this patch:
Attachment #8825635 - Flags: approval-comm-aurora?
Comment on attachment 8825635 [details] [diff] [review]
[mozconfigs] Remove macosx-universal and move the required stuff to macosx64

ewong the old release-l10n in macosx64 which doesn't get changed by the bug does not look right compared to the deleted one and the current one in mail. Broken anyway because of l10n but could you check this.
Flags: needinfo?(ewong)
Comment on attachment 8825635 [details] [diff] [review]
[mozconfigs] Remove macosx-universal and move the required stuff to macosx64

a=me
Attachment #8825635 - Flags: approval-comm-aurora? → approval-comm-aurora+
This builds upon the previous patch which includes the correct non-universal
mozconfig for the regular build as well as setting the proper objdir
depending on the Gecko version.
Attachment #8840765 - Attachment is obsolete: true
(In reply to Frank-Rainer Grahl from comment #15)
> Comment on attachment 8825635 [details] [diff] [review]
> [mozconfigs] Remove macosx-universal and move the required stuff to macosx64
> 
> ewong the old release-l10n in macosx64 which doesn't get changed by the bug
> does not look right compared to the deleted one and the current one in mail.
> Broken anyway because of l10n but could you check this.

Sure.  I'll take a look
Flags: needinfo?(ewong)
Attachment #8841386 - Flags: review?(frgrahl)
Depends on: 1342826
Blocks: SM2.48b1
Comment on attachment 8841386 [details] [diff] [review]
[configs] forgot regular build and switch back to universal build for seamonkey versions <= 2.50

r+ to get the beta going but as stated might need a change next week after merge day when 2.48 beta branch and 2.49 beta are both in comm-beta and need to be build.
Attachment #8841386 - Flags: review?(frgrahl) → review+
https://hg.mozilla.org/build/buildbot-configs/rev/ccc49f7b5a34bfad22f30a088542a9e6a58ed693
Bug 1329379 - Drop universal builds for SeaMonkey version >= 2.51 r=frg
Blocks: SM2.48
No longer blocks: SM2.48
Flags: needinfo?(bugspam.Callek)

We are done here. 2.49.5 dropped the universal builds.

Status: ASSIGNED → RESOLVED
Closed: 7 years ago4 years ago
Depends on: SM2.49.5
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.49
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: