[tbb-bugs] #21723 [Applications/Tor Browser]: Fix inconsistent generation of MOZ_MACBUNDLE_ID

Tor Bug Tracker & Wiki blackhole at torproject.org
Wed Mar 15 15:50:34 UTC 2017

#21723: Fix inconsistent generation of MOZ_MACBUNDLE_ID
 Reporter:  teor                      |          Owner:  tbb-team
     Type:  defect                    |         Status:  needs_review
 Priority:  Medium                    |      Milestone:
Component:  Applications/Tor Browser  |        Version:
 Severity:  Normal                    |     Resolution:
 Keywords:  TorBrowserTeam201703R     |  Actual Points:
Parent ID:  #17670                    |         Points:
 Reviewer:                            |        Sponsor:

Comment (by mcs):

 Replying to [comment:6 teor]:
 > The build change `--with-distribution-id=org.torproject` changes the
 distribution ID on all platforms. I'm not sure how the distribution ID is
 used on Linux or Windows, but we should be consistent.

 I agree that we should be consistent (unless we find a good reason not to
 be). Here are the references to MOZ_DISTRIBUTION_ID (which is what --with-
 distribution-id sets):

 Files such as browser/branding/aurora/locales/moz.build use
 see where that is used.

 In old-configure.in, MOZ_DISTRIBUTION_ID is used to define
 MOZ_MACBUNDLE_ID, as teor pointed out.

 The code in toolkit/xre/nsAppRunner.cpp makes the MOZ_DISTRIBUTION_ID
 value available via nsIXULRuntime.distributionID. Chasing that via:
 shows that it is used in telemetry and in the search engine code. We don't
 care about telemetry, but the search engine code will insert the
 distribution ID into search parameters via `{moz:distributionID}`. But the
 only place I see that technique used is in browser/locales/searchplugins
 /yahoo-jp-auctions.xml, which I assume we will not ship with Tor Browser.

 So... I think it is okay to change the distribution ID on all platforms.

Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21723#comment:7>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online

More information about the tbb-bugs mailing list