Closed Bug 856763 Opened 11 years ago Closed 11 years ago

Build and deploy gcc 4.5.4 on all linux/linux64 build slaves

Categories

(Release Engineering :: General, defect, P2)

x86_64
Windows 7
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: RyanVM, Assigned: rail)

References

Details

Attachments

(2 files, 1 obsolete file)

We currently use gcc 4.5.2 as our default compiler for linux/linux64 builds. However, this is causing is frequent pain in bug 820796 as it leads to burned nightlies and PGO builds. In dev-platform, Mike suggested that we try upgrading to gcc 4.5.4 to see if the situation improves since gcc 4.7 still has some issues that prevent us from switching over to it.

Note that this bug is not currently about switching over to version 4.5.4, only that it be made available for testing.
As much as I love seeing hours of builder time get wasted every day due to bug 820796, can we please up the priority on this to see if it helps?
Flags: needinfo?(rail)
Sure, I can generate a package, but it would be great to have a person who can validate the binaries. maybe glandium? ;)
Assignee: nobody → rail
Flags: needinfo?(rail)
Priority: -- → P2
I think he already volunteered himself in the dev.platform thread about this :)
Attached patch gcc 4.5.4 spec file (obsolete) — Splinter Review
Renames, cleanup and DRY.
Attachment #733703 - Flags: review?(bugspam.Callek)
Oops, s/0moz4/0moz1/
Attachment #733703 - Attachment is obsolete: true
Attachment #733703 - Flags: review?(bugspam.Callek)
Attachment #733704 - Flags: review?(bugspam.Callek)
Attached patch configsSplinter Review
Once I have the packages ready I'll test if they don't conflict.
Attachment #733707 - Flags: review?(bugspam.Callek)
Comment on attachment 733707 [details] [diff] [review]
configs

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

Only comment here is, since we now know 4.7 is broken why don't we just remove it while here? (not blocking)
Attachment #733707 - Flags: review?(bugspam.Callek) → review+
Comment on attachment 733704 [details] [diff] [review]
gcc 4.5.4 spec file

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

As long as you test for no conflicts, r=me

Before switch we'll definitely need assurances that any compat changes (due to gcc switch) are understood and signed off on, and figure out if we can (and want to) limit updates based on that :-)
Attachment #733704 - Flags: review?(bugspam.Callek) → review+
(In reply to Justin Wood (:Callek) from comment #7)
> Comment on attachment 733707 [details] [diff] [review]
> configs
> 
> Review of attachment 733707 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Only comment here is, since we now know 4.7 is broken why don't we just
> remove it while here? (not blocking)

That would instantly break dxr and asan. And it's still useful.
(In reply to Mike Hommey [:glandium] from comment #9)
> (In reply to Justin Wood (:Callek) from comment #7)
> > Only comment here is, since we now know 4.7 is broken why don't we just
> > remove it while here? (not blocking)
> 
> That would instantly break dxr and asan. And it's still useful.

Ahhh, was under the impression it was only there for try, so no worries then.
Comment on attachment 733707 [details] [diff] [review]
configs

http://hg.mozilla.org/build/buildbot-configs/rev/78b3db20dd24

all 3 gcc packages live together very friendly :)
Attachment #733707 - Flags: checked-in+
Will be available on try after next reconfig. /tools/gcc-4.5.4-0moz1 is the new prefix.
in production
Al done here. Feel free to reopen if we need to change something in the paackage.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
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: