Opened 2 years ago

Closed 17 months ago

#25930 closed task (fixed)

Update gcc to 8.X

Reported by: cypherpunks Owned by: tbb-team
Priority: Medium Milestone:
Component: Applications/Tor Browser Version:
Severity: Normal Keywords: tbb-rbm, TorBrowserTeam201905R, GeorgKoppen201905
Cc: Actual Points:
Parent ID: #30321 Points:
Reviewer: Sponsor:


We should build Tor Browser using gcc 7.3.0.
Firefox meta is
The only thing to check is
Porting guide is
LRA is the default
-mindirect-branch=thunk is available.

Child Tickets

#30377closedtbb-teamRemove Selfrando from our build systemApplications/Tor Browser

Change History (8)

comment:1 Changed 2 years ago by gk

Parent ID: #24631

That's nothing for ESR 60. We stick to 6.x for Tor Browser 8.

comment:2 Changed 21 months ago by gk

#29335 is a duplicate.

comment:3 Changed 21 months ago by gk

Summary: Update gcc to 7.3.0Update gcc to 7.X

comment:4 Changed 20 months ago by gk

We could get -fwrapv back for our debug builds if we switched to GCC 8.x (see: #27597). As a data point: I've been compiling (and using) some custom Tor Browser with GCC 8.x for a while now and did not run into any issues. Thus, we might consider 8.x sufficiently stable to switch to it in particular when preparing for eser68.

comment:5 Changed 18 months ago by gk

Parent ID: #30321

comment:7 Changed 18 months ago by gk

Keywords: TorBrowserTeam201905R GeorgKoppen201905 added
Status: newneeds_review
Summary: Update gcc to 7.XUpdate gcc to 8.X

I think it's good to be conservative here and not switching to GCC 9.1 right away. It seems using GCC 8.3.0 is a decent solution for our Tor Browser 9. bug_25930_v3( has the proposed fix. Note: I am leaving mingw-w64/gcc untouched for now. All things seem to build fine, just our spec hack for Firefox is breaking. But that should get fixed by mingw-w64/clang anyway, so we bump the GCC part for mingw-w64/gcc after we switched Firefox to mingw-w64/clang (#28238).

comment:8 Changed 17 months ago by boklm

Resolution: fixed
Status: needs_reviewclosed

This looks good to me. I merged this to master with commit ae8f6de9939dd372945bea063245ab7123019cb6.

Note: See TracTickets for help on using tickets.