[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

Re: [tor-talk] torsocks is broken and unmaintained



Nick Mathewson:
> On Sat, Nov 3, 2012 at 11:23 PM, Matthew Finkel <matthew.finkel@xxxxxxxxx>wrote:
> 
>> On 11/03/2012 08:38 PM, Nick Mathewson wrote:
>>
>  [...]
> 
>>> Okay, sounds like we've got some enthusiasm.  Let's get started.  I
>>> volunteer to review commits and if people ask me to, and suggest that
>>> asking me to review stuff for a while might be a smart idea.  I just gave
>>> myself commit access to the git@xxxxxxxxxxxxxxxxxxxxx repo too, in case
>>> that helps.  I am not planning to be a primary author here.
>>
>> Thanks for adding one more thing to your plate! I know Jake can handle
>> this but the more eyes we have looking at these initial changes the
>> better it'll be.
>>
>>>
>>> Given the amount of people asking us to apply and/or warning us that we
>>> mustn't apply particular patches, I'm going to suggest the following
>>> principles for a while:
>>>   * LET'S START MINIMAL.  Let's stick to doing only the very major
>> bugfixes
>>> and obvious fixes for at least the next release or two, so that something
>>> usable comes out.
>>
>> Agreed. To be honest, I haven't really looked at the code too much, so
>> I'll start diving into that in a bit. (If there isn't one already...I
>> haven't checked) Can we get a trac component added so we can track
>> progress and such?
> 
> 
> Done.  At some point we should migrate issues from google code, but IMO
> that's best done once we have something nontrivial to show for our efforts.
> 

I'm on the google code project, I'd like to close out each of the issues
and open new issues on Tor's trac. That will allow us to ensure each
reporter will receive a desired response.

> 
>>>   * NO ARCHITECTURAL ASTRONAUTICS. I'm always tempted when I come to a
>>> codebase for the first time to refactor the heck out of it.  Let's avoid
>>> doing that till we have a little experience with this codebase.  There
>>> isn't all that much here: let's
>>
>> Yes...let's! :)
>>
>> Was there supposed to be more to that sentence?
> 
> 
> Yeah; sometimes I start a sentence, then I think of something to write
> elsewhere and start another sentence, but then by the time I'm done with
> that one I don't remember the first sentence any more, so it
> 
> That one should end with "There isn't all that much code here; let's make
> sure we understand it pretty thoroughly before we complexify it in the name
> of some half-glimpsed vision."
> 

Agreed.

> 
> 
>>>   * LOVE MEANS GET TESTED. If at all possible, we should make this
>> codebase
>>> easier to test (right now it wants you to install before testing), and
>>> improve the coverage of the tests so that (if as people suspect) we're
>>> likely to break things on one platform when we fix them on another, we
>> can
>>> at least find out fast whether a patch works everywhere.
>>>
>>
>> Certainly sounds like a good idea. I'm going to have to familiarize
>> myself with some of the other *nix platforms it does/should support.
>> Just looking through the current issues on google code, for example, I
>> don't know the internals of OSX well enough *yet* to know if [1] is even
>> possible. But once we've compiled a list of all the current critical
>> patches, Debian and others (assuming such a list doesn't exist already),
>> then we start applying, testing, revising, etc. :)
>>
>> [1] https://code.google.com/p/torsocks/issues/detail?id=41
> 
> 
> Hm. Supposedly, it's _supposed_ to work on OSX.  It has a lot of code for
> OSX support.  I just tried it with curl on my osx laptop, and it seemed to
> work okay.
> 

It should work on OS X and if it doesn't, we likely have OS X related
bugs. :)

All the best,
Jake
_______________________________________________
tor-talk mailing list
tor-talk@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-talk