-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Hi, I have a patch to improve thread safety in jtorctl. How should I submit it? Cheers, Michael -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iQEcBAEBCAAGBQJTPT1vAAoJEBEET9GfxSfM0CMH+wais6E2jteMA0EnrAsV9+K1 jx08ypBNtE7qTYehe3m4s6vFwEGmw6lu2qYqi8XwPlD6PY/x1OtnDWjrTUsMfyWe OMZF2kqtMcpk47Y+SAL2c8P/BGYk88Wb2UMTZEGo3AjBZS2wQyHD+FMaSoDP02R9 MvCfwON+sn4a+AJxrnX/NsrI5FivJWR8obWX2sYpfCQaNIub/G/SGh9DtMbJFEjD tLc+MHyrrmmEmM1LN8kQp6QdBP7xIn/Cv+St3vzG920UuvQD5ZmHCYqiD3ddp74f vqfuKZs8GvnDFnND9mfy16l9lUu8dME1GWnoAO0l4jRmhBKwUCZMnjaLZvBVZiE= =6inv -----END PGP SIGNATURE-----
Hi Michael, glad you want to help! JTorCtl is pretty much unmaintained (it was last committed to in 2009), but if you file a ticket on https://trac.torproject.org we might be able to get it merged. Karsten, you're the last person to touch that project. Thoughts on the component for the ticket? Cheers! -Damian
On 04/04/14 04:02, Damian Johnson wrote:
Hi Michael, glad you want to help! JTorCtl is pretty much unmaintained (it was last committed to in 2009), but if you file a ticket on https://trac.torproject.org we might be able to get it merged.
Karsten, you're the last person to touch that project. Thoughts on the component for the ticket?
Wow, JTorCtl is probably not just unmaintained, but also probably doesn't have a single application using it since 2009. There's a reason why it's sorted into the "Attic" group on https://gitweb.torproject.org/. Michael, are you working on JTorCtl because you want to help out with writing some Java code? If so, I might have some other fine Java code for you to hack on that is in active use: https://gitweb.torproject.org/metrics-lib.git https://gitweb.torproject.org/onionoo.git https://gitweb.torproject.org/exonerator.git https://gitweb.torproject.org/metrics-db.git https://gitweb.torproject.org/metrics-web.git Or are you using JTorCtl for some project of yours? In that case it might be easier to keep your own JTorCtl branch for now and ask for your changes to be merged when you publish your application. Of course, you could also open a Trac ticket and ask for review, but that might take between a while and forever. All the best, Karsten
On 04/04/2014 02:54 AM, Karsten Loesing wrote:
Wow, JTorCtl is probably not just unmaintained, but also probably doesn't have a single application using it since 2009. There's a reason why it's sorted into the "Attic" group on https://gitweb.torproject.org/.
*ahem* Orbot represents 2 million users of jtorctl, and it works just fine for us (in our limited use of it). https://gitweb.torproject.org/orbot.git/tree/HEAD:/external +n
*ahem* Orbot represents 2 million users of jtorctl, and it works just fine for us (in our limited use of it).
Yay, sounds like we found a new owner for JTorCtl! ;) Nathan, should Michael go ahead and file his pull request with the Orbot component then?
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 On 04/04/14 15:44, Nathan Freitas wrote:
*ahem* Orbot represents 2 million users of jtorctl, and it works just fine for us (in our limited use of it).
Hehe, I was just this second cloning the Orbot repo to see if you were still using jtorctl. :-) Briar also uses it, as does Ploggy. On 04/04/14 07:54, Karsten Loesing wrote:
Michael, are you working on JTorCtl because you want to help out with writing some Java code? If so, I might have some other fine Java code for you to hack on that is in active use:
I'd love to help, but right now I've got too much on my plate to take on any new projects. Sorry!
Or are you using JTorCtl for some project of yours? In that case it might be easier to keep your own JTorCtl branch for now and ask for your changes to be merged when you publish your application. Of course, you could also open a Trac ticket and ask for review, but that might take between a while and forever.
I'm happy to maintain a fork - that's what I've been doing until now, but I thought it might be useful to push my changes upstream. Here's a summary of the changes: 1. Make TorControlConnection thread-safe 2. Convert TorControlError and TorControlSyntaxError into checked exceptions 3. Reduce the visibility of classes and methods 4. Use the Java 1.5 foreach operator for readability 5. Fix formatting of javadoc comments Points 2 and 3 might affect existing users of the library. I'm happy to post five small patches or one big patch for eventual review if you think Trac's the way to go. Cheers, Michael -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iQEcBAEBCAAGBQJTPtFHAAoJEBEET9GfxSfMMjAIAK9ajy2Efe1m+hYp6sw9EuJV +KhoUD3UMCq9GWsq93hATZVhrRbV5bpOJvEjaikd9/Ux6TA6888Ngi23LWh7D/Er aaVjm87AoAhMOeaOEXXsmlIqAi8+DRA7jZbE2zHB3TYAjVEU7/KKWY+1J57+j3nu n01Yfr2bp0XeEFJ1a9Q6RPZdhYGo6u0OMsjh9etQoEoDdE+0NR2PYRpUz1sigi/q DYEdxeX66ZTVaEOm4KndSEjmacxt86xQrbsaCGovLb97iUxXa1gv4W66ATTC5aIi A+9Y9v2RmfxynyhQh8QGCd8edxGJ5vQ8M+ISinDIBeq4UiY225r2MWt5GWUQrHc= =4UPM -----END PGP SIGNATURE-----
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 On 04/04/14 16:45, Nathan Freitas wrote:
On 04/04/2014 11:34 AM, Damian Johnson wrote:
Nathan, should Michael go ahead and file his pull request with the Orbot component then?
Sure! We can keep the JtorCtl repo on its own, and perhaps move it out of the attic.
I will review patches, test and merge.
Great, thanks! I'll create some tickets for the Orbot component. Cheers, Michael -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iQEcBAEBCAAGBQJTPtfYAAoJEBEET9GfxSfM6b0H/2CfE09vv8oJxufa3vcRWXYy oB4RyNMim8foZgTx+DBDt3QGERG1VSwWDTXbbAYh/F+r0U/9q5eszPZs2Q7hO9Zy 6HG+YtrHaC37tNZ+cNzTwd0S5FLhoX7j3YL4p+OSynn3yyRUp0ga/gFpmlLjh8XE xxadEEDiOjlWeZS8oy+LZ+i9PvAms4yzrfpxlEPSahv/qjNSXt943uZ/xtrQdbQb EfaZ1tpPGCimNZcnAAjXQPhk49sESNqlKTBnM7WZwgC3w9CUbQ6r5nXzL+1K5gyy QPvwYkoi8Oc/tzvfZULI86Z5nh2zhJDdwmnnQ2FELvF0wCDAiRip8xchKcnxqO4= =m5rB -----END PGP SIGNATURE-----
participants (4)
-
Damian Johnson -
Karsten Loesing -
Michael Rogers -
Nathan Freitas