Opened 10 months ago

Last modified 9 months ago

#26877 needs_information enhancement

Declare gcc version in rbm.conf

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


We declare the gcc version (currently set to 6.4.0) in multiple places:

projects/gcc/config:version: 6.4.0
projects/mingw-w64/config:  gcc_version: 6.4.0

We should probably define it in rbm.conf instead and then refer to that. (This may also be relevant for #25485).

Child Tickets

Change History (6)

comment:1 Changed 10 months ago by gk

Component: Applications/rbmApplications/Tor Browser
Keywords: tbb-rbm added
Owner: changed from boklm to tbb-team

comment:2 Changed 10 months ago by sukhbir

Status: newneeds_review

comment:3 Changed 10 months ago by boklm

I think the two gcc versions used in projects/gcc and projects/mingw-w64 are independent, and could sometimes be two different versions. So I'm not sure we should define it in rbm.conf. But if we define it in rbm.conf, we should then probably also define the sha256sum there too.

comment:4 Changed 10 months ago by gk

Keywords: TorBrowser201807R added

Hm, I am inclined to think it makes sense to keep those versions separated. Assuming we hit a toolchain bug for Linux which needs to get handled by using a newer compiler it does not seem smart to suddenly ship a new toolchain for Windows, too (or vice versa), especially if the stable series is affected. Thus, I think we should be conservative here and not have both linked together. On the other hand, the question is how often this actually happens and whether we could not work around that ad hoc without the need for bumping the compiler version of the unaffected platform. So, I don't feel too strongly here.

Do we know why we had this separation in the past? I think we did not have that back in the tor-browser-bundle days.

comment:5 Changed 9 months ago by gk

Keywords: TorBrowserTeam201808R added; TorBrowser201807R removed
Status: needs_reviewneeds_information

comment:6 Changed 9 months ago by gk

Keywords: TorBrowserTeam201808 added; TorBrowserTeam201808R removed
Note: See TracTickets for help on using tickets.