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

Re: [tor-bugs] #9022 [Pluggable transport]: Create an XMPP pluggable transport



#9022: Create an XMPP pluggable transport
---------------------------------+------------------------------------------
 Reporter:  asn                  |          Owner:  feynman 
     Type:  task                 |         Status:  accepted
 Priority:  normal               |      Milestone:          
Component:  Pluggable transport  |        Version:          
 Keywords:                       |         Parent:          
   Points:                       |   Actualpoints:          
---------------------------------+------------------------------------------

Comment(by asn):

 hi feynman,

 I think you asked me in IRC what was the purpose of my 'argparse_cli'
 branch.

 I hoped that it would make the code a bit more elegant, and also make the
 CLI parsing more meaningful (for example, the --client switch of the
 current CLI is very weird since it's three things at one). I also added
 some verbose argparse help strings and some examples.

 For example, the argparse configuration in the current code is not
 entirely correct I think. For example, the '--login' switch should be
 marked as mandatory (both client and server always use it), but it's not.
 This means that if I omit the '--login' switch, I will get a python error
 in `while index<len(args.login):` instead of a helpful argparse message.

 In any case, it's perfectly fine if you don't merge my changes.

 BTW, I'm extremely busy these days. However, I hope I can find some time
 next week (after the weekend) to look at the deployability of hexchat and
 also check out your last two messages.

 Thanks!

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/9022#comment:65>
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