[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #30166 [Applications/Tor Browser]: If custom bridges are specified, only use those bridges for connecting
#30166: If custom bridges are specified, only use those bridges for connecting
-------------------------------------------------+-------------------------
Reporter: gk | Owner: tbb-
| team
Type: defect | Status:
| needs_revision
Priority: High | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Resolution:
Keywords: tbb-mobile, tbb-8.5-must, | Actual Points:
TorBrowserTeam201905 |
Parent ID: #27609 | Points:
Reviewer: | Sponsor:
-------------------------------------------------+-------------------------
Comment (by gk):
Replying to [comment:14 sisbell]:
> Replying to [comment:13 gk]:
>
>
> > Alright here come some questions:
> >
> > 1) It seems de13672fe1ef6eb180edd4ec80f3f416f5d5c6b1 and
30a7a83c5e715eb7ef739870bc0ff615ad52daeb fix the same bug (this one).
Could you prepare a commit with both of those changes as the first of
those commits does not fully fix the problem?
> Sure I'll get that fixed. I have been working off master, so I didn't
want to rebase in case anyone else was using the library but now that we
have brought in the code, I'll work off the branches with rebases.
Thanks.
> >
> > 2) What's the need for commit
7e77a5cd6c9ec609963e3b60b7838116cb8b37e2? (I guess we should at least
mention the reason for that in the commit message)
> This may be an issue for further discussion. My idea has been to create
a library for general use in the community so I wanted to keep the Android
versions updated. Otherwise, we lock developers into older versions of
Android Studio and dependencies, which makes adoption more difficult.
>
> The versions are isolated to the properties files so changes are easy.
As long our patches for the versions still work, this seems a good
compromise.
Works for me. I'd like to have in the commit message a hint, though, what
makes the update necessary or why we are doing it (maybe it's a new SDK we
want to support, maybe a new Android version, maybe...), just so that one
has something to reason about whether the actual change is good/necessary.
> >
> > 3) What's the deal with the TOPL libs in the repo given that we build
those ourselves during the build?
> This is also related to general community use. I want this to work
outside of tor-browser-build. Ideally, when things are stable (which looks
very close), we will get the TOPL libraries pushed to a maven repo, rather
than including them in the libs directory.
Okay, yes, please. Meanwhile I am fine having them in the repo, although
I'd like to avoid that as a permanent solution.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/30166#comment:15>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs