[tbb-bugs] #28051 [Applications/Tor Browser]: Build Orbot into TBA

Tor Bug Tracker & Wiki blackhole at torproject.org
Wed Nov 21 17:14:17 UTC 2018


#28051: Build Orbot into TBA
-------------------------------------------------+-------------------------
 Reporter:  sysrqb                               |          Owner:  sysrqb
     Type:  enhancement                          |         Status:
                                                 |  needs_review
 Priority:  Very High                            |      Milestone:
Component:  Applications/Tor Browser             |        Version:
 Severity:  Normal                               |     Resolution:
 Keywords:  tbb-mobile, tba-a2,                  |  Actual Points:
  TorBrowserTeam201811                           |
Parent ID:                                       |         Points:
 Reviewer:                                       |        Sponsor:
-------------------------------------------------+-------------------------

Comment (by sysrqb):

 Replying to [comment:17 gk]:
 > Okay, starting to review I guess `28051_2`?
 >
 > 1) Can you squash those fixups you made (that is commits
 43936a111d23a89d509e9de75ecf2f28c17dc3c0 and
 c7fa82b84eceacf49d7d9436bfeb530630c9fd03) and base the branch on the
 latest `tor-browser-60.3.0esr-8.5-1`?
 >
 > 2) commit ca62a15e7e8f6781c61e82c4ec64bc4fcfbfe97c
 >
 > {{{
 > +    //implementation 'com.github.apl-devs:appintro:v4.2.2'
 > +    //implementation 'com.github.javiersantos:AppUpdater:2.6.4'
 > }}}
 > Let's omit those if we don't need them. The Orbot integration is already
 complex enough :)
 >
 > 3) commit 5a755dad3b549f5e01b87016f6f4bfbfbd3654cc
 >
 > I doubt we'll ever get back to a model where Tor Browser for Android
 ships without Orbot or something similarly integrated and needs to ask the
 user about installing Orbot first. Thus, can we clean up the
 > `checkStartOrbot()` related logic while we are at it instead of working
 around with `checkStartOrbot2()`?

 Thanks for starting the review. I pushed `28051_3` with the changes.

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


More information about the tbb-bugs mailing list