June 2017 - tor-commits - lists.torproject.org

[tor/maint-0.2.6] Add -dev to version number.
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit ec3ce773ad1394441049337befc8ff7e0867b093 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:08 2017 -0400 Add -dev to version number. --- configure.ac | 2 +- contrib/tor-mingw.nsi.in | 2 +- src/win32/orconfig.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 6b4d009..fe830ab 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ dnl Copyright (c) 2004-2006, Roger Dingledine, Nick Mathewson dnl Copyright (c) 2007-2013, The Tor Project, Inc. dnl See LICENSE for licensing information -AC_INIT([tor],[0.2.4.29]) +AC_INIT([tor],[0.2.4.29-dev]) AC_CONFIG_SRCDIR([src/or/main.c]) AC_CONFIG_MACRO_DIR([m4]) AM_INIT_AUTOMAKE diff --git a/contrib/tor-mingw.nsi.in b/contrib/tor-mingw.nsi.in index 4e00cb7..537dec5 100644 --- a/contrib/tor-mingw.nsi.in +++ b/contrib/tor-mingw.nsi.in @@ -8,7 +8,7 @@ !include "LogicLib.nsh" !include "FileFunc.nsh" !insertmacro GetParameters -!define VERSION "0.2.4.29" +!define VERSION "0.2.4.29-dev" !define INSTALLER "tor-${VERSION}-win32.exe" !define WEBSITE "https://www.torproject.org/" !define LICENSE "LICENSE" diff --git a/src/win32/orconfig.h b/src/win32/orconfig.h index 5468455..d2bb090 100644 --- a/src/win32/orconfig.h +++ b/src/win32/orconfig.h @@ -241,7 +241,7 @@ #define USING_TWOS_COMPLEMENT /* Version number of package */ -#define VERSION "0.2.4.29" +#define VERSION "0.2.4.29-dev"

1 0

[tor/maint-0.2.7-redux] Add -dev to version number.
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit f3804f5999c7eb5cdfddf039f623bcffb89732b8 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:13 2017 -0400 Add -dev to version number. --- configure.ac | 2 +- contrib/win32build/tor-mingw.nsi.in | 2 +- src/win32/orconfig.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index da797e5..4eeb34d 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ dnl Copyright (c) 2004-2006, Roger Dingledine, Nick Mathewson dnl Copyright (c) 2007-2013, The Tor Project, Inc. dnl See LICENSE for licensing information -AC_INIT([tor],[0.2.5.14]) +AC_INIT([tor],[0.2.5.14-dev]) AC_CONFIG_SRCDIR([src/or/main.c]) AC_CONFIG_MACRO_DIR([m4]) AM_INIT_AUTOMAKE diff --git a/contrib/win32build/tor-mingw.nsi.in b/contrib/win32build/tor-mingw.nsi.in index 3e55030..af09326 100644 --- a/contrib/win32build/tor-mingw.nsi.in +++ b/contrib/win32build/tor-mingw.nsi.in @@ -8,7 +8,7 @@ !include "LogicLib.nsh" !include "FileFunc.nsh" !insertmacro GetParameters -!define VERSION "0.2.5.14" +!define VERSION "0.2.5.14-dev" !define INSTALLER "tor-${VERSION}-win32.exe" !define WEBSITE "https://www.torproject.org/" !define LICENSE "LICENSE" diff --git a/src/win32/orconfig.h b/src/win32/orconfig.h index d45f51e..5e670c3 100644 --- a/src/win32/orconfig.h +++ b/src/win32/orconfig.h @@ -241,7 +241,7 @@ #define USING_TWOS_COMPLEMENT /* Version number of package */ -#define VERSION "0.2.5.14" +#define VERSION "0.2.5.14-dev"

1 0

[tor/maint-0.2.7-redux] Add -dev to version number.
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit 95a88f72832d9385ef8e4199d29f8f46090a4584 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:16 2017 -0400 Add -dev to version number. --- configure.ac | 2 +- contrib/win32build/tor-mingw.nsi.in | 2 +- src/win32/orconfig.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 55e59e9..63f4f1f 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ dnl Copyright (c) 2004-2006, Roger Dingledine, Nick Mathewson dnl Copyright (c) 2007-2015, The Tor Project, Inc. dnl See LICENSE for licensing information -AC_INIT([tor],[0.2.6.12]) +AC_INIT([tor],[0.2.6.12-dev]) AC_CONFIG_SRCDIR([src/or/main.c]) AC_CONFIG_MACRO_DIR([m4]) AM_INIT_AUTOMAKE diff --git a/contrib/win32build/tor-mingw.nsi.in b/contrib/win32build/tor-mingw.nsi.in index 4de6b1b..576abd2 100644 --- a/contrib/win32build/tor-mingw.nsi.in +++ b/contrib/win32build/tor-mingw.nsi.in @@ -8,7 +8,7 @@ !include "LogicLib.nsh" !include "FileFunc.nsh" !insertmacro GetParameters -!define VERSION "0.2.6.12" +!define VERSION "0.2.6.12-dev" !define INSTALLER "tor-${VERSION}-win32.exe" !define WEBSITE "https://www.torproject.org/" !define LICENSE "LICENSE" diff --git a/src/win32/orconfig.h b/src/win32/orconfig.h index 74a27f4..4a9ad29 100644 --- a/src/win32/orconfig.h +++ b/src/win32/orconfig.h @@ -232,7 +232,7 @@ #define USING_TWOS_COMPLEMENT /* Version number of package */ -#define VERSION "0.2.6.12" +#define VERSION "0.2.6.12-dev"

1 0

[tor/maint-0.2.7-redux] Add -dev to version number.
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit 44c8cb986d8a95e3eaf8e3ec40307a7d7bb65197 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:18 2017 -0400 Add -dev to version number. --- configure.ac | 2 +- contrib/win32build/tor-mingw.nsi.in | 2 +- src/win32/orconfig.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index f754942..42e5fa7 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ dnl Copyright (c) 2004-2006, Roger Dingledine, Nick Mathewson dnl Copyright (c) 2007-2015, The Tor Project, Inc. dnl See LICENSE for licensing information -AC_INIT([tor],[0.2.7.8]) +AC_INIT([tor],[0.2.7.8-dev]) AC_CONFIG_SRCDIR([src/or/main.c]) AC_CONFIG_MACRO_DIR([m4]) AM_INIT_AUTOMAKE diff --git a/contrib/win32build/tor-mingw.nsi.in b/contrib/win32build/tor-mingw.nsi.in index 32c9731..7aae273 100644 --- a/contrib/win32build/tor-mingw.nsi.in +++ b/contrib/win32build/tor-mingw.nsi.in @@ -8,7 +8,7 @@ !include "LogicLib.nsh" !include "FileFunc.nsh" !insertmacro GetParameters -!define VERSION "0.2.7.8" +!define VERSION "0.2.7.8-dev" !define INSTALLER "tor-${VERSION}-win32.exe" !define WEBSITE "https://www.torproject.org/" !define LICENSE "LICENSE" diff --git a/src/win32/orconfig.h b/src/win32/orconfig.h index 46714ef..f1caed6 100644 --- a/src/win32/orconfig.h +++ b/src/win32/orconfig.h @@ -232,7 +232,7 @@ #define USING_TWOS_COMPLEMENT /* Version number of package */ -#define VERSION "0.2.7.8" +#define VERSION "0.2.7.8-dev"

1 0

[tor/maint-0.2.4] Add -dev to version number.
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit ec3ce773ad1394441049337befc8ff7e0867b093 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:08 2017 -0400 Add -dev to version number. --- configure.ac | 2 +- contrib/tor-mingw.nsi.in | 2 +- src/win32/orconfig.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 6b4d009..fe830ab 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ dnl Copyright (c) 2004-2006, Roger Dingledine, Nick Mathewson dnl Copyright (c) 2007-2013, The Tor Project, Inc. dnl See LICENSE for licensing information -AC_INIT([tor],[0.2.4.29]) +AC_INIT([tor],[0.2.4.29-dev]) AC_CONFIG_SRCDIR([src/or/main.c]) AC_CONFIG_MACRO_DIR([m4]) AM_INIT_AUTOMAKE diff --git a/contrib/tor-mingw.nsi.in b/contrib/tor-mingw.nsi.in index 4e00cb7..537dec5 100644 --- a/contrib/tor-mingw.nsi.in +++ b/contrib/tor-mingw.nsi.in @@ -8,7 +8,7 @@ !include "LogicLib.nsh" !include "FileFunc.nsh" !insertmacro GetParameters -!define VERSION "0.2.4.29" +!define VERSION "0.2.4.29-dev" !define INSTALLER "tor-${VERSION}-win32.exe" !define WEBSITE "https://www.torproject.org/" !define LICENSE "LICENSE" diff --git a/src/win32/orconfig.h b/src/win32/orconfig.h index 5468455..d2bb090 100644 --- a/src/win32/orconfig.h +++ b/src/win32/orconfig.h @@ -241,7 +241,7 @@ #define USING_TWOS_COMPLEMENT /* Version number of package */ -#define VERSION "0.2.4.29" +#define VERSION "0.2.4.29-dev"

1 0

[tor/maint-0.2.6] Add -dev to version number.
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit 95a88f72832d9385ef8e4199d29f8f46090a4584 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:16 2017 -0400 Add -dev to version number. --- configure.ac | 2 +- contrib/win32build/tor-mingw.nsi.in | 2 +- src/win32/orconfig.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index 55e59e9..63f4f1f 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ dnl Copyright (c) 2004-2006, Roger Dingledine, Nick Mathewson dnl Copyright (c) 2007-2015, The Tor Project, Inc. dnl See LICENSE for licensing information -AC_INIT([tor],[0.2.6.12]) +AC_INIT([tor],[0.2.6.12-dev]) AC_CONFIG_SRCDIR([src/or/main.c]) AC_CONFIG_MACRO_DIR([m4]) AM_INIT_AUTOMAKE diff --git a/contrib/win32build/tor-mingw.nsi.in b/contrib/win32build/tor-mingw.nsi.in index 4de6b1b..576abd2 100644 --- a/contrib/win32build/tor-mingw.nsi.in +++ b/contrib/win32build/tor-mingw.nsi.in @@ -8,7 +8,7 @@ !include "LogicLib.nsh" !include "FileFunc.nsh" !insertmacro GetParameters -!define VERSION "0.2.6.12" +!define VERSION "0.2.6.12-dev" !define INSTALLER "tor-${VERSION}-win32.exe" !define WEBSITE "https://www.torproject.org/" !define LICENSE "LICENSE" diff --git a/src/win32/orconfig.h b/src/win32/orconfig.h index 74a27f4..4a9ad29 100644 --- a/src/win32/orconfig.h +++ b/src/win32/orconfig.h @@ -232,7 +232,7 @@ #define USING_TWOS_COMPLEMENT /* Version number of package */ -#define VERSION "0.2.6.12" +#define VERSION "0.2.6.12-dev"

1 0

[tor/maint-0.2.5] Merge branch 'maint-0.2.4' into maint-0.2.5
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit 1a7af7470fdd6173f1b876fc6fee2b171174b4ff Merge: f3804f5 ec3ce77 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:37 2017 -0400 Merge branch 'maint-0.2.4' into maint-0.2.5 "ours" merge to avoid version bump.

1 0

[tor/maint-0.2.5] Add -dev to version number.
by nickm＠torproject.org 08 Jun '17

08 Jun '17

commit f3804f5999c7eb5cdfddf039f623bcffb89732b8 Author: Nick Mathewson <nickm(a)torproject.org> Date: Thu Jun 8 14:05:13 2017 -0400 Add -dev to version number. --- configure.ac | 2 +- contrib/win32build/tor-mingw.nsi.in | 2 +- src/win32/orconfig.h | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index da797e5..4eeb34d 100644 --- a/configure.ac +++ b/configure.ac @@ -3,7 +3,7 @@ dnl Copyright (c) 2004-2006, Roger Dingledine, Nick Mathewson dnl Copyright (c) 2007-2013, The Tor Project, Inc. dnl See LICENSE for licensing information -AC_INIT([tor],[0.2.5.14]) +AC_INIT([tor],[0.2.5.14-dev]) AC_CONFIG_SRCDIR([src/or/main.c]) AC_CONFIG_MACRO_DIR([m4]) AM_INIT_AUTOMAKE diff --git a/contrib/win32build/tor-mingw.nsi.in b/contrib/win32build/tor-mingw.nsi.in index 3e55030..af09326 100644 --- a/contrib/win32build/tor-mingw.nsi.in +++ b/contrib/win32build/tor-mingw.nsi.in @@ -8,7 +8,7 @@ !include "LogicLib.nsh" !include "FileFunc.nsh" !insertmacro GetParameters -!define VERSION "0.2.5.14" +!define VERSION "0.2.5.14-dev" !define INSTALLER "tor-${VERSION}-win32.exe" !define WEBSITE "https://www.torproject.org/" !define LICENSE "LICENSE" diff --git a/src/win32/orconfig.h b/src/win32/orconfig.h index d45f51e..5e670c3 100644 --- a/src/win32/orconfig.h +++ b/src/win32/orconfig.h @@ -241,7 +241,7 @@ #define USING_TWOS_COMPLEMENT /* Version number of package */ -#define VERSION "0.2.5.14" +#define VERSION "0.2.5.14-dev"

1 0

[stem/master] Parallelize longest process test
by atagar＠torproject.org 08 Jun '17

08 Jun '17

commit de97e243e241ee34df6333e5557f4a3fcea9900d Author: Damian Johnson <atagar(a)torproject.org> Date: Sun Jun 4 16:52:41 2017 -0700 Parallelize longest process test I plan to do 'em all but starting with one as a proof of concept. This drops the test runtime from 48s to 36s on my system (25% faster). --- run_tests.py | 9 ++++- test/integ/installation.py | 1 + test/integ/process.py | 99 ++++++++++++++++++++++++++++------------------ 3 files changed, 69 insertions(+), 40 deletions(-) diff --git a/run_tests.py b/run_tests.py index a174f65..2798fd4 100755 --- a/run_tests.py +++ b/run_tests.py @@ -29,6 +29,7 @@ import stem.version import test import test.arguments import test.integ.installation +import test.integ.process import test.output import test.runner import test.task @@ -232,8 +233,12 @@ def main(): skipped_tests = 0 - if args.run_integ and (not args.specific_test or 'test.integ.installation'.startswith(args.specific_test)): - test.integ.installation.setup() + if args.run_integ: + if not args.specific_test or 'test.integ.installation'.startswith(args.specific_test): + test.integ.installation.setup() + + if not args.specific_test or 'test.integ.process'.startswith(args.specific_test): + test.integ.process.setup(args.tor_path) if args.run_unit: test.output.print_divider('UNIT TESTS', True) diff --git a/test/integ/installation.py b/test/integ/installation.py index 8b39e9d..2b58e1d 100644 --- a/test/integ/installation.py +++ b/test/integ/installation.py @@ -12,6 +12,7 @@ import unittest import stem import stem.util.system +import stem.util.test_tools import test INSTALL_MISMATCH_MSG = "Running 'python setup.py sdist' doesn't match our git contents in the following way. The manifest in our setup.py may need to be updated...\n\n" diff --git a/test/integ/process.py b/test/integ/process.py index 207d4b4..5677be1 100644 --- a/test/integ/process.py +++ b/test/integ/process.py @@ -18,6 +18,7 @@ import stem.process import stem.socket import stem.util.str_tools import stem.util.system +import stem.util.test_tools import stem.util.tor_tools import stem.version import test.require @@ -38,6 +39,64 @@ PublishServerDescriptor 0 DataDirectory %s """ +TEST_TAKE_OWNERSHIP_BY_PID = None + + +def setup(tor_cmd): + global TEST_TAKE_OWNERSHIP_BY_PID + + TEST_TAKE_OWNERSHIP_BY_PID = stem.util.test_tools.AsyncTestResult(_test_take_ownership_via_pid, tor_cmd) + + +def _test_take_ownership_via_pid(tor_cmd): + """ + Checks that the tor process quits after we do if we set take_ownership. To + test this we spawn a process and trick tor into thinking that it is us. + """ + + if not stem.util.system.is_available('sleep'): + raise stem.util.test_tools.SkipTest('(sleep unavailable)') + elif test.tor_version() < stem.version.Requirement.TAKEOWNERSHIP: + raise stem.util.test_tools.SkipTest('(requires )' % stem.version.Requirement.TAKEOWNERSHIP) + + data_directory = tempfile.mkdtemp() + + try: + sleep_process = subprocess.Popen(['sleep', '60']) + + tor_process = stem.process.launch_tor_with_config( + tor_cmd = tor_cmd, + config = { + 'SocksPort': '2779', + 'ControlPort': '2780', + 'DataDirectory': data_directory, + '__OwningControllerProcess': str(sleep_process.pid), + }, + completion_percent = 5, + ) + + # Kill the sleep command. Tor should quit shortly after. + + sleep_process.kill() + sleep_process.communicate() + + # tor polls for the process every fifteen seconds so this may take a + # while... + # + # https://trac.torproject.org/projects/tor/ticket/21281 + + start_time = time.time() + + while time.time() - start_time < 30: + if tor_process.poll() == 0: + return # tor exited + + time.sleep(0.01) + + raise AssertionError("tor didn't quit after the process that owned it terminated") + finally: + shutil.rmtree(data_directory) + class TestProcess(unittest.TestCase): def setUp(self): @@ -425,49 +484,13 @@ class TestProcess(unittest.TestCase): if not (runtime > 0.05 and runtime < 1): self.fail('Test should have taken 0.05-1 seconds, took %0.1f instead' % runtime) - @test.require.only_run_once - @test.require.command('sleep') - @test.require.version(stem.version.Requirement.TAKEOWNERSHIP) - @patch('os.getpid') - def test_take_ownership_via_pid(self, getpid_mock): + def test_take_ownership_via_pid(self): """ Checks that the tor process quits after we do if we set take_ownership. To test this we spawn a process and trick tor into thinking that it is us. """ - sleep_process = subprocess.Popen(['sleep', '60']) - getpid_mock.return_value = str(sleep_process.pid) - - tor_process = stem.process.launch_tor_with_config( - tor_cmd = test.runner.get_runner().get_tor_command(), - config = { - 'SocksPort': '2777', - 'ControlPort': '2778', - 'DataDirectory': self.data_directory, - }, - completion_percent = 5, - take_ownership = True, - ) - - # Kill the sleep command. Tor should quit shortly after. - - sleep_process.kill() - sleep_process.communicate() - - # tor polls for the process every fifteen seconds so this may take a - # while... - # - # https://trac.torproject.org/projects/tor/ticket/21281 - - start_time = time.time() - - while time.time() - start_time < 30: - if tor_process.poll() == 0: - return # tor exited - - time.sleep(0.01) - - self.fail("tor didn't quit after the process that owned it terminated") + TEST_TAKE_OWNERSHIP_BY_PID.result(self) @test.require.only_run_once @test.require.version(stem.version.Requirement.TAKEOWNERSHIP)

1 0

[stem/master] Support pid arguments in stem.util.system's is_running()
by atagar＠torproject.org 08 Jun '17

08 Jun '17

commit e212e4182bd74a7fbf8ccbca6625c6762f0d4b49 Author: Damian Johnson <atagar(a)torproject.org> Date: Sun Jun 4 11:08:13 2017 -0700 Support pid arguments in stem.util.system's is_running() We have an is_running() function that accepts process names. Expanding it to be able to check for pids too... https://stackoverflow.com/a/568285 --- docs/change_log.rst | 1 + stem/util/system.py | 14 ++++++++++++-- test/integ/util/system.py | 12 ++++++++++-- 3 files changed, 23 insertions(+), 4 deletions(-) diff --git a/docs/change_log.rst b/docs/change_log.rst index 926e7ea..84f7493 100644 --- a/docs/change_log.rst +++ b/docs/change_log.rst @@ -69,6 +69,7 @@ The following are only available within Stem's `git repository * Added timeout argument to :func:`~stem.util.system.call` * Added cwd argument to :func:`~stem.util.system.call` * Added :class:`~stem.util.test_tools.TimedTestRunner` and :func:`~stem.util.test_tools.test_runtimes` + * Supporing pid arguments in :func:`~stem.util.system.is_running` * **Interpreter** diff --git a/stem/util/system.py b/stem/util/system.py index 69ec979..6d96672 100644 --- a/stem/util/system.py +++ b/stem/util/system.py @@ -257,14 +257,24 @@ def is_available(command, cached=True): def is_running(command): """ - Checks for if a process with a given name is running or not. + Checks for if a process with a given name or pid is running. - :param str command: process name to be checked + .. versionchanged:: 1.6.0 + Added support for pid arguments. + + :param str,int command: process name if a str or pid if an int to be checked :returns: **True** if the process is running, **False** if it's not among ps results, and **None** if ps can't be queried """ + if isinstance(command, int): + try: + os.kill(command, 0) + return True + except OSError: + return False + # Linux and the BSD families have different variants of ps. Guess based on # the is_bsd() check which to try first, then fall back to the other. # diff --git a/test/integ/util/system.py b/test/integ/util/system.py index 07d7d75..818b639 100644 --- a/test/integ/util/system.py +++ b/test/integ/util/system.py @@ -87,10 +87,18 @@ class TestSystem(unittest.TestCase): self.assertFalse(stem.util.system.is_available('blarg_and_stuff')) + def test_is_running_by_pid(self): + """ + Checks the stem.util.system.is_running function with a pid. + """ + + self.assertTrue(stem.util.system.is_running(test.runner.get_runner().get_pid())) + self.assertFalse(stem.util.system.is_running(528955)) + @test.require.command('ps') - def test_is_running(self): + def test_is_running_by_name(self): """ - Checks the stem.util.system.is_running function. + Checks the stem.util.system.is_running function with a process name. """ # Check to see if the command we started tor with is running. The process

1 0