Closed Bug 938510 Opened 11 years ago Closed 10 years ago

switch to tooltool gcc from bug 913442

Categories

(Firefox Build System :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1075183
mozilla29

People

(Reporter: tbsaunde, Assigned: tbsaunde)

References

Details

Attachments

(1 file, 1 obsolete file)

I'd need to dig through email to get the links but I've tested the following things on try
- try: -b do -p linux,linux64 -u all builds and has no unexpected test issues.
- talos results are the same as trunk, with the exception of cracken (or however its spelled) which seems to have improved? but may just be a fluke.
- 10 or so retriggers don't cause any ICEs

So I guess we might as well try putting this in actual production unless glandium or froydnj have an objection.
Attached patch gcc.patch (obsolete) — Splinter Review
Attachment #832769 - Flags: review?(mh+mozilla)
Attachment #832769 - Flags: review?(mh+mozilla) → review+
backed out in https://hg.mozilla.org/integration/mozilla-inbound/rev/b50d2b2f98e0 because b2g desktop builds don't seem to get tooltool packages
and after adding the manifest to b2g/config/tooltool/linux{32,64}/ we hit https://tbpl.mozilla.org/php/getParsedLog.php?id=30916585&tree=Try because those builds are done in the mozilla-centos-i686 env :( so I guess we can either block on releng  changing that or hack around it somehow.
Depends on: 947462
Blocks: 946639
Comment on attachment 832769 [details] [diff] [review]
gcc.patch

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

::: build/unix/mozconfig.linux
@@ +4,5 @@
> +CXX="$topsrcdir/gcc/bin/g++"
> +
> +# We want to make sure we use binutils and other binaries in the tooltool
> +# package.
> +mk_add_options PATH="$topsrcdir/gcc:$PATH"

while I'm (re)looking at this: there's a missing /bin here.
Attached patch gcctestSplinter Review
turns out in addition to b2g be done on i686 vlagrind jobs don't use tooltool at all :/
Attachment #832769 - Attachment is obsolete: true
Attachment #8368269 - Flags: review?(mh+mozilla)
Comment on attachment 8368269 [details] [diff] [review]
gcctest

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

Note the patch summary refers to the wrong bug ;)

::: build/unix/mozconfig.linux
@@ +1,4 @@
>  . "$topsrcdir/build/mozconfig.common"
>  
> +# some b2g desktop builds still happen on i686 machines, and the tooltool
> +# toolchain is x86_64 only.

b2g desktop should be building on x86-64, but it doesn't use tooltool manifests.

@@ +3,5 @@
> +# some b2g desktop builds still happen on i686 machines, and the tooltool
> +# toolchain is x86_64 only.
> +# We also deal with valgrind builds here, they don't use tooltool manifests at
> +# all yet.
> +if [ -z "$building_b2g" -a -z "$building_valgrind" ]

might as well make that $no_tooltool
Attachment #8368269 - Flags: review?(mh+mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/5509778fb65b
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla29
I believe this broke l10n, see bug 966793
Blocks: 966793
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
No longer blocks: 966793
Depends on: 966793
Let's say this is a dupe of bug 1075183. There's bug 965151 for the valgrind part.
Status: REOPENED → RESOLVED
Closed: 10 years ago10 years ago
Resolution: --- → DUPLICATE
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.