- tor-commits - lists.torproject.org

[nyx/master] Halting trackers when tor disconnects
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit ea4d683b1c46da42b50251c0dcb57f860f4d8609 Author: Damian Johnson <atagar(a)torproject.org> Date: Wed Feb 10 12:17:57 2016 -0800 Halting trackers when tor disconnects The connection tracker did this, but applying it to all trackers. --- nyx/controller.py | 19 +++---------------- nyx/log_panel.py | 8 +++++--- nyx/util/tracker.py | 3 +++ 3 files changed, 11 insertions(+), 19 deletions(-) diff --git a/nyx/controller.py b/nyx/controller.py index dc9f53b..fcc6239 100644 --- a/nyx/controller.py +++ b/nyx/controller.py @@ -8,7 +8,6 @@ import time import curses import threading -import nyx.arguments import nyx.menu.menu import nyx.popups import nyx.header_panel @@ -38,7 +37,6 @@ def conf_handler(key, value): CONFIG = conf.config_dict('nyx', { - 'startup.events': 'N3', 'features.acsSupport': True, 'features.panels.show.graph': True, 'features.panels.show.log': True, @@ -104,8 +102,7 @@ def init_controller(stdscr, start_time): first_page_panels.append(nyx.graph_panel.GraphPanel(stdscr)) if CONFIG['features.panels.show.log']: - expanded_events = nyx.arguments.expand_events(CONFIG['startup.events']) - first_page_panels.append(nyx.log_panel.LogPanel(stdscr, expanded_events)) + first_page_panels.append(nyx.log_panel.LogPanel(stdscr)) if first_page_panels: page_panels.append(first_page_panels) @@ -434,18 +431,8 @@ def heartbeat_check(is_unresponsive): def conn_reset_listener(controller, event_type, _): - """ - Pauses connection resolution when tor's shut down, and resumes with the new - pid if started again. - """ - - resolver = nyx.util.tracker.get_connection_tracker() - - if resolver.is_alive(): - resolver.set_paused(event_type == State.CLOSED) - - if event_type == State.CLOSED: - log.notice('Tor control port closed') + if event_type == State.CLOSED: + log.notice('Tor control port closed') def start_nyx(stdscr): diff --git a/nyx/log_panel.py b/nyx/log_panel.py index 37e9e21..708a679 100644 --- a/nyx/log_panel.py +++ b/nyx/log_panel.py @@ -29,15 +29,16 @@ def conf_handler(key, value): CONFIG = conf.config_dict('nyx', { + 'attr.log_color': {}, + 'cache.log_panel.size': 1000, 'features.logFile': '', 'features.log.showDuplicateEntries': False, 'features.log.prepopulate': True, 'features.log.prepopulateReadLimit': 5000, 'features.log.maxRefreshRate': 300, 'features.log.regex': [], - 'cache.log_panel.size': 1000, 'msg.misc.event_types': '', - 'attr.log_color': {}, + 'startup.events': 'N3', }, conf_handler) # The height of the drawn content is estimated based on the last time we redrew @@ -61,11 +62,12 @@ class LogPanel(panel.Panel, threading.Thread): from tor's log file if it exists. """ - def __init__(self, stdscr, logged_events): + def __init__(self, stdscr): panel.Panel.__init__(self, stdscr, 'log', 0) threading.Thread.__init__(self) self.setDaemon(True) + logged_events = nyx.arguments.expand_events(CONFIG['startup.events']) self._event_log = nyx.util.log.LogGroup(CONFIG['cache.log_panel.size'], group_by_day = True) self._event_types = nyx.util.log.listen_for_events(self._register_tor_event, logged_events) self._log_file = nyx.util.log.LogFileOutput(CONFIG['features.logFile']) diff --git a/nyx/util/tracker.py b/nyx/util/tracker.py index f218630..8aa1181 100644 --- a/nyx/util/tracker.py +++ b/nyx/util/tracker.py @@ -460,6 +460,9 @@ class Daemon(threading.Thread): self._process_pid = tor_pid self._process_name = tor_cmd if tor_cmd else 'tor' + elif event_type == stem.contorl.State.CLOSED: + self._process_pid = None + self._process_name = None def __enter__(self): self.start()

1 0

[nyx/master] Merge init_controller() into controller constructor
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit 5f975c93855f2d53a678eb701ea6229a0f6ece59 Author: Damian Johnson <atagar(a)torproject.org> Date: Wed Feb 10 19:53:07 2016 -0800 Merge init_controller() into controller constructor Now that the init method just calls simple constructors we may as well do this in the Controller class. --- nyx/controller.py | 86 ++++++++++++++++++++----------------------------------- 1 file changed, 31 insertions(+), 55 deletions(-) diff --git a/nyx/controller.py b/nyx/controller.py index 7b0a276..e35c929 100644 --- a/nyx/controller.py +++ b/nyx/controller.py @@ -76,54 +76,6 @@ def stop_controller(): return halt_thread -def init_controller(stdscr): - """ - Spawns the controller, and related panels for it. - - Arguments: - stdscr - curses window - """ - - global NYX_CONTROLLER - - # initializes the panels - - sticky_panels = [ - nyx.header_panel.HeaderPanel(stdscr), - LabelPanel(stdscr), - ] - - page_panels, first_page_panels = [], [] - - # first page: graph and log - if CONFIG['features.panels.show.graph']: - first_page_panels.append(nyx.graph_panel.GraphPanel(stdscr)) - - if CONFIG['features.panels.show.log']: - first_page_panels.append(nyx.log_panel.LogPanel(stdscr)) - - if first_page_panels: - page_panels.append(first_page_panels) - - # second page: connections - if CONFIG['features.panels.show.connection']: - page_panels.append([nyx.connection_panel.ConnectionPanel(stdscr)]) - - # third page: config - - if CONFIG['features.panels.show.config']: - page_panels.append([nyx.config_panel.ConfigPanel(stdscr)]) - - # fourth page: torrc - - if CONFIG['features.panels.show.torrc']: - page_panels.append([nyx.torrc_panel.TorrcPanel(stdscr)]) - - # initializes the controller - - NYX_CONTROLLER = Controller(stdscr, sticky_panels, page_panels) - - class LabelPanel(panel.Panel): """ Panel that just displays a single line of text. @@ -158,21 +110,43 @@ class Controller: Tracks the global state of the interface """ - def __init__(self, stdscr, sticky_panels, page_panels): + def __init__(self, stdscr): """ Creates a new controller instance. Panel lists are ordered as they appear, top to bottom on the page. Arguments: stdscr - curses window - sticky_panels - panels shown at the top of each page - page_panels - list of pages, each being a list of the panels on it """ - self.quit_signal = False self._screen = stdscr - self._sticky_panels = sticky_panels - self._page_panels = page_panels + + self._sticky_panels = [ + nyx.header_panel.HeaderPanel(stdscr), + LabelPanel(stdscr), + ] + + self._page_panels, first_page_panels = [], [] + + if CONFIG['features.panels.show.graph']: + first_page_panels.append(nyx.graph_panel.GraphPanel(stdscr)) + + if CONFIG['features.panels.show.log']: + first_page_panels.append(nyx.log_panel.LogPanel(stdscr)) + + if first_page_panels: + self._page_panels.append(first_page_panels) + + if CONFIG['features.panels.show.connection']: + self._page_panels.append([nyx.connection_panel.ConnectionPanel(stdscr)]) + + if CONFIG['features.panels.show.config']: + self._page_panels.append([nyx.config_panel.ConfigPanel(stdscr)]) + + if CONFIG['features.panels.show.torrc']: + self._page_panels.append([nyx.torrc_panel.TorrcPanel(stdscr)]) + + self.quit_signal = False self._page = 0 self._is_paused = False self._force_redraw = False @@ -408,7 +382,9 @@ def start_nyx(stdscr): stdscr - curses window """ - init_controller(stdscr) + global NYX_CONTROLLER + + NYX_CONTROLLER = Controller(stdscr) control = get_controller() if not CONFIG['features.acsSupport']:

1 0

[nyx/master] Header panel doesn't need the start time
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit d280b81cf55acbc792615613bd84096816a64ede Author: Damian Johnson <atagar(a)torproject.org> Date: Wed Feb 10 19:45:01 2016 -0800 Header panel doesn't need the start time Argument's unused by the constructor. Guess just something that got lost in the shuffle. --- nyx/controller.py | 6 +++--- nyx/header_panel.py | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/nyx/controller.py b/nyx/controller.py index 0f972b3..7b0a276 100644 --- a/nyx/controller.py +++ b/nyx/controller.py @@ -76,7 +76,7 @@ def stop_controller(): return halt_thread -def init_controller(stdscr, start_time): +def init_controller(stdscr): """ Spawns the controller, and related panels for it. @@ -89,7 +89,7 @@ def init_controller(stdscr, start_time): # initializes the panels sticky_panels = [ - nyx.header_panel.HeaderPanel(stdscr, start_time), + nyx.header_panel.HeaderPanel(stdscr), LabelPanel(stdscr), ] @@ -408,7 +408,7 @@ def start_nyx(stdscr): stdscr - curses window """ - init_controller(stdscr, CONFIG['start_time']) + init_controller(stdscr) control = get_controller() if not CONFIG['features.acsSupport']: diff --git a/nyx/header_panel.py b/nyx/header_panel.py index bbc5ede..0d6591a 100644 --- a/nyx/header_panel.py +++ b/nyx/header_panel.py @@ -34,7 +34,7 @@ class HeaderPanel(panel.Panel, threading.Thread): Top area containing tor settings and system information. """ - def __init__(self, stdscr, start_time): + def __init__(self, stdscr): panel.Panel.__init__(self, stdscr, 'header', 0) threading.Thread.__init__(self) self.setDaemon(True)

1 0

[nyx/master] Replace stop_controller() with halt() method
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit 9555a8a14a3940df452d5640d82f6921e17066e5 Author: Damian Johnson <atagar(a)torproject.org> Date: Thu Feb 11 20:37:10 2016 -0800 Replace stop_controller() with halt() method Feels a little better as a Controller method. --- nyx/controller.py | 36 ++++++++++++++++-------------------- nyx/starter.py | 9 +++++---- 2 files changed, 21 insertions(+), 24 deletions(-) diff --git a/nyx/controller.py b/nyx/controller.py index e35c929..e143b49 100644 --- a/nyx/controller.py +++ b/nyx/controller.py @@ -56,26 +56,6 @@ def get_controller(): return NYX_CONTROLLER -def stop_controller(): - """ - Halts our Controller, providing back the thread doing so. - """ - - def halt_controller(): - control = get_controller() - - if control: - for panel_impl in control.get_daemon_panels(): - panel_impl.stop() - - for panel_impl in control.get_daemon_panels(): - panel_impl.join() - - halt_thread = threading.Thread(target = halt_controller) - halt_thread.start() - return halt_thread - - class LabelPanel(panel.Panel): """ Panel that just displays a single line of text. @@ -373,6 +353,22 @@ class Controller: def quit(self): self.quit_signal = True + def halt(self): + """ + Halts curses panels, providing back the thread doing so. + """ + + def halt_panels(): + for panel_impl in self.get_daemon_panels(): + panel_impl.stop() + + for panel_impl in self.get_daemon_panels(): + panel_impl.join() + + halt_thread = threading.Thread(target = halt_panels) + halt_thread.start() + return halt_thread + def start_nyx(stdscr): """ diff --git a/nyx/starter.py b/nyx/starter.py index 6084d04..326bd82 100644 --- a/nyx/starter.py +++ b/nyx/starter.py @@ -265,10 +265,11 @@ def _shutdown_daemons(controller): Stops and joins on worker threads. """ - halt_threads = [ - nyx.controller.stop_controller(), - nyx.util.tracker.stop_trackers(), - ] + halt_threads = [nyx.util.tracker.stop_trackers()] + controller = nyx.controller.get_controller() + + if controller: + halt_threads.append(controller.halt()) for thread in halt_threads: thread.join()

1 0

[nyx/master] Drop unused Panel methods
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit ea87cf573b3a36f38a291ccd4157a1d7c43a78ed Author: Damian Johnson <atagar(a)torproject.org> Date: Sun Feb 14 16:06:06 2016 -0800 Drop unused Panel methods The get_left() and set_left() are just there to match the related methods for the top and bottom (... which are unused outside the Panel too). These too are completely unused so easy to pluck out. Dropping a couple other unused methods too. --- nyx/util/panel.py | 36 +----------------------------------- 1 file changed, 1 insertion(+), 35 deletions(-) diff --git a/nyx/util/panel.py b/nyx/util/panel.py index 283a16a..4a94133 100644 --- a/nyx/util/panel.py +++ b/nyx/util/panel.py @@ -209,13 +209,6 @@ class Panel(object): return self.parent - def is_visible(self): - """ - Provides if the panel's configured to be visible or not. - """ - - return self.visible - def set_visible(self, is_visible): """ Toggles if the panel is visible or not. @@ -325,13 +318,6 @@ class Panel(object): return self.pause_time - def get_top(self): - """ - Provides the position subwindows are placed at within its parent. - """ - - return self.top - def set_top(self, top): """ Changes the position where subwindows are placed within its parent. @@ -344,26 +330,6 @@ class Panel(object): self.top = top self.win = None - def get_left(self): - """ - Provides the left position where this subwindow is placed within its - parent. - """ - - return self.left - - def set_left(self, left): - """ - Changes the left position where this subwindow is placed within its parent. - - Arguments: - left - positioning of top within parent - """ - - if self.left != left: - self.left = left - self.win = None - def get_height(self): """ Provides the height used for subwindows (-1 if it isn't limited). @@ -467,7 +433,7 @@ class Panel(object): # skipped if not currently visible or activity has been halted - if not self.is_visible() or HALT_ACTIVITY: + if not self.visible or HALT_ACTIVITY: return # if the panel's completely outside its parent then this is a no-op

1 0

[nyx/master] Merge text_input into panel
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit bf58d639c25c5d7cf83ad840b68d716c59b81437 Author: Damian Johnson <atagar(a)torproject.org> Date: Sun Feb 14 12:58:30 2016 -0800 Merge text_input into panel Our text_input module was primarily to support the control interpreter. We might re-introduce some of this in the future but for now only the BasicValidator is used by our codebase. --- nyx/util/__init__.py | 2 - nyx/util/panel.py | 80 +++++++++++++++++-- nyx/util/text_input.py | 213 ------------------------------------------------- 3 files changed, 75 insertions(+), 220 deletions(-) diff --git a/nyx/util/__init__.py b/nyx/util/__init__.py index e32a816..1bc3d71 100644 --- a/nyx/util/__init__.py +++ b/nyx/util/__init__.py @@ -17,8 +17,6 @@ from nyx.util import log __all__ = [ 'log', 'panel', - 'text_input', - 'tor_config', 'tracker', 'ui_tools', ] diff --git a/nyx/util/panel.py b/nyx/util/panel.py index bf16422..611a14d 100644 --- a/nyx/util/panel.py +++ b/nyx/util/panel.py @@ -9,7 +9,7 @@ import curses.ascii import curses.textpad from threading import RLock -from nyx.util import text_input, ui_tools +from nyx.util import ui_tools from stem.util import conf, log, str_tools @@ -32,6 +32,8 @@ SPECIAL_KEYS = { 'esc': 27, } +PASS = -1 + def conf_handler(key, value): if key == 'features.torrc.maxLineWrap': @@ -46,6 +48,76 @@ CONFIG = conf.config_dict('nyx', { HALT_ACTIVITY = False +class BasicValidator(object): + """ + Interceptor for keystrokes given to a textbox, doing the following: + - quits by setting the input to curses.ascii.BEL when escape is pressed + - stops the cursor at the end of the box's content when pressing the right + arrow + - home and end keys move to the start/end of the line + """ + + def validate(self, key, textbox): + """ + Processes the given key input for the textbox. This may modify the + textbox's content, cursor position, etc depending on the functionality + of the validator. This returns the key that the textbox should interpret, + PASS if this validator doesn't want to take any action. + + Arguments: + key - key code input from the user + textbox - curses Textbox instance the input came from + """ + + result = self.handle_key(key, textbox) + return key if result == PASS else result + + def handle_key(self, key, textbox): + y, x = textbox.win.getyx() + + if curses.ascii.isprint(key) and x < textbox.maxx: + # Shifts the existing text forward so input is an insert method rather + # than replacement. The curses.textpad accepts an insert mode flag but + # this has a couple issues... + # - The flag is only available for Python 2.6+, before that the + # constructor only accepted a subwindow argument as per: + # https://trac.torproject.org/projects/tor/ticket/2354 + # - The textpad doesn't shift text that has text attributes. This is + # because keycodes read by textbox.win.inch() includes formatting, + # causing the curses.ascii.isprint() check it does to fail. + + current_input = textbox.gather() + textbox.win.addstr(y, x + 1, current_input[x:textbox.maxx - 1]) + textbox.win.move(y, x) # reverts cursor movement during gather call + elif key == 27: + # curses.ascii.BEL is a character codes that causes textpad to terminate + + return curses.ascii.BEL + elif key == curses.KEY_HOME: + textbox.win.move(y, 0) + return None + elif key in (curses.KEY_END, curses.KEY_RIGHT): + msg_length = len(textbox.gather()) + textbox.win.move(y, x) # reverts cursor movement during gather call + + if key == curses.KEY_END and msg_length > 0 and x < msg_length - 1: + # if we're in the content then move to the end + + textbox.win.move(y, msg_length - 1) + return None + elif key == curses.KEY_RIGHT and x >= msg_length - 1: + # don't move the cursor if there's no content after it + + return None + elif key == 410: + # if we're resizing the display during text entry then cancel it + # (otherwise the input field is filled with nonprintable characters) + + return curses.ascii.BEL + + return PASS + + class Panel(object): """ Wrapper for curses subwindows. This hides most of the ugliness in common @@ -555,7 +627,7 @@ class Panel(object): return x, y - def getstr(self, y, x, initial_text = '', text_format = None, max_width = None, validator = None): + def getstr(self, y, x, initial_text = '', text_format = None, max_width = None): """ Provides a text field where the user can input a string, blocking until they've done so and returning the result. If the user presses escape then @@ -572,7 +644,6 @@ class Panel(object): initial_text - starting text in this field text_format - format used for the text max_width - maximum width for the text field - validator - custom TextInputValidator for handling keybindings """ if not text_format: @@ -610,8 +681,7 @@ class Panel(object): textbox = curses.textpad.Textbox(input_subwindow) - if not validator: - validator = text_input.BasicValidator() + validator = BasicValidator() textbox.win.attron(text_format) user_input = textbox.edit(lambda key: validator.validate(key, textbox)).strip() diff --git a/nyx/util/text_input.py b/nyx/util/text_input.py deleted file mode 100644 index 3172750..0000000 --- a/nyx/util/text_input.py +++ /dev/null @@ -1,213 +0,0 @@ -""" -Provides input validators that provide text input with various capabilities. -These can be chained together with the first matching validator taking -precidence. -""" - -import os -import curses - -PASS = -1 - - -class TextInputValidator: - """ - Basic interface for validators. Implementations should override the handle_key - method. - """ - - def __init__(self, next_validator = None): - self.next_validator = next_validator - - def validate(self, key, textbox): - """ - Processes the given key input for the textbox. This may modify the - textbox's content, cursor position, etc depending on the functionality - of the validator. This returns the key that the textbox should interpret, - PASS if this validator doesn't want to take any action. - - Arguments: - key - key code input from the user - textbox - curses Textbox instance the input came from - """ - - result = self.handle_key(key, textbox) - - if result != PASS: - return result - elif self.next_validator: - return self.next_validator.validate(key, textbox) - else: - return key - - def handle_key(self, key, textbox): - """ - Process the given keycode with this validator, returning the keycode for - the textbox to process, and PASS if this doesn't want to modify it. - - Arguments: - key - key code input from the user - textbox - curses Textbox instance the input came from - """ - - return PASS - - -class BasicValidator(TextInputValidator): - """ - Interceptor for keystrokes given to a textbox, doing the following: - - quits by setting the input to curses.ascii.BEL when escape is pressed - - stops the cursor at the end of the box's content when pressing the right - arrow - - home and end keys move to the start/end of the line - """ - - def handle_key(self, key, textbox): - y, x = textbox.win.getyx() - - if curses.ascii.isprint(key) and x < textbox.maxx: - # Shifts the existing text forward so input is an insert method rather - # than replacement. The curses.textpad accepts an insert mode flag but - # this has a couple issues... - # - The flag is only available for Python 2.6+, before that the - # constructor only accepted a subwindow argument as per: - # https://trac.torproject.org/projects/tor/ticket/2354 - # - The textpad doesn't shift text that has text attributes. This is - # because keycodes read by textbox.win.inch() includes formatting, - # causing the curses.ascii.isprint() check it does to fail. - - current_input = textbox.gather() - textbox.win.addstr(y, x + 1, current_input[x:textbox.maxx - 1]) - textbox.win.move(y, x) # reverts cursor movement during gather call - elif key == 27: - # curses.ascii.BEL is a character codes that causes textpad to terminate - - return curses.ascii.BEL - elif key == curses.KEY_HOME: - textbox.win.move(y, 0) - return None - elif key in (curses.KEY_END, curses.KEY_RIGHT): - msg_length = len(textbox.gather()) - textbox.win.move(y, x) # reverts cursor movement during gather call - - if key == curses.KEY_END and msg_length > 0 and x < msg_length - 1: - # if we're in the content then move to the end - - textbox.win.move(y, msg_length - 1) - return None - elif key == curses.KEY_RIGHT and x >= msg_length - 1: - # don't move the cursor if there's no content after it - - return None - elif key == 410: - # if we're resizing the display during text entry then cancel it - # (otherwise the input field is filled with nonprintable characters) - - return curses.ascii.BEL - - return PASS - - -class HistoryValidator(TextInputValidator): - """ - This intercepts the up and down arrow keys to scroll through a backlog of - previous commands. - """ - - def __init__(self, command_backlog = [], next_validator = None): - TextInputValidator.__init__(self, next_validator) - - # contents that can be scrolled back through, newest to oldest - - self.command_backlog = command_backlog - - # selected item from the backlog, -1 if we're not on a backlog item - - self.selection_index = -1 - - # the fields input prior to selecting a backlog item - - self.custom_input = '' - - def handle_key(self, key, textbox): - if key in (curses.KEY_UP, curses.KEY_DOWN): - offset = 1 if key == curses.KEY_UP else -1 - new_selection = self.selection_index + offset - - # constrains the new selection to valid bounds - - new_selection = max(-1, new_selection) - new_selection = min(len(self.command_backlog) - 1, new_selection) - - # skips if this is a no-op - - if self.selection_index == new_selection: - return None - - # saves the previous input if we weren't on the backlog - - if self.selection_index == -1: - self.custom_input = textbox.gather().strip() - - if new_selection == -1: - new_input = self.custom_input - else: - new_input = self.command_backlog[new_selection] - - y, _ = textbox.win.getyx() - _, max_x = textbox.win.getmaxyx() - textbox.win.clear() - textbox.win.addstr(y, 0, new_input[:max_x - 1]) - textbox.win.move(y, min(len(new_input), max_x - 1)) - - self.selection_index = new_selection - return None - - return PASS - - -class TabCompleter(TextInputValidator): - """ - Provides tab completion based on the current input, finishing if there's only - a single match. This expects a functor that accepts the current input and - provides matches. - """ - - def __init__(self, completer, next_validator = None): - TextInputValidator.__init__(self, next_validator) - - # functor that accepts a string and gives a list of matches - - self.completer = completer - - def handle_key(self, key, textbox): - # Matches against the tab key. The ord('\t') is nine, though strangely none - # of the curses.KEY_*TAB constants match this... - - if key == 9: - current_contents = textbox.gather().strip() - matches = self.completer(current_contents) - new_input = None - - if len(matches) == 1: - # only a single match, fill it in - new_input = matches[0] - elif len(matches) > 1: - # looks for a common prefix we can complete - common_prefix = os.path.commonprefix(matches) # weird that this comes from path... - - if common_prefix != current_contents: - new_input = common_prefix - - # TODO: somehow display matches... this is not gonna be fun - - if new_input: - y, _ = textbox.win.getyx() - _, max_x = textbox.win.getmaxyx() - textbox.win.clear() - textbox.win.addstr(y, 0, new_input[:max_x - 1]) - textbox.win.move(y, min(len(new_input), max_x - 1)) - - return None - - return PASS

1 0

[nyx/master] Drop unused no-op function
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit a157afc862a36f146595e98185ec8bc66996e5ea Author: Damian Johnson <atagar(a)torproject.org> Date: Thu Feb 11 22:15:07 2016 -0800 Drop unused no-op function --- nyx/util/panel.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/nyx/util/panel.py b/nyx/util/panel.py index 136fe85..bf16422 100644 --- a/nyx/util/panel.py +++ b/nyx/util/panel.py @@ -42,14 +42,6 @@ CONFIG = conf.config_dict('nyx', { 'features.maxLineWrap': 8, }, conf_handler) - -# tags used by addfstr - this maps to functor/argument combinations since the -# actual values (in the case of color attributes) might not yet be initialized - -def _no_op(arg): - return arg - - # prevents curses redraws if set HALT_ACTIVITY = False

1 0

[nyx/master] Drop redraw() block argument
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit d281f04f7404290a0ef2fb4bba1f5d8fb515faf8 Author: Damian Johnson <atagar(a)torproject.org> Date: Sun Feb 14 15:58:41 2016 -0800 Drop redraw() block argument Unused argument for concurrent use. --- nyx/util/panel.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/nyx/util/panel.py b/nyx/util/panel.py index 611a14d..10727c8 100644 --- a/nyx/util/panel.py +++ b/nyx/util/panel.py @@ -456,16 +456,13 @@ class Panel(object): pass - def redraw(self, force_redraw=False, block=False): + def redraw(self, force_redraw=False): """ Clears display and redraws its content. This can skip redrawing content if able (ie, the subwindow's unchanged), instead just refreshing the display. Arguments: force_redraw - forces the content to be cleared and redrawn if true - block - if drawing concurrently with other panels this determines - if the request is willing to wait its turn or should be - abandoned """ # skipped if not currently visible or activity has been halted @@ -498,7 +495,7 @@ class Panel(object): self.max_y, self.max_x = subwin_max_y, subwin_max_x - if not CURSES_LOCK.acquire(block): + if not CURSES_LOCK.acquire(False): return try:

1 0

[nyx/master] Drop unused getstr() arguments
by atagar＠torproject.org 15 Feb '16

15 Feb '16

commit cc60ec1a0e8be6990ab2499929cb7d0fea9bad8f Author: Damian Johnson <atagar(a)torproject.org> Date: Sun Feb 14 16:01:47 2016 -0800 Drop unused getstr() arguments We only use this method one place so tempted to get rid of it alltogether, but for now dropping the arguments we don't use. We could probably drop these curses.A_NORMAL bits but not sure, so leaving it alone for now. --- nyx/util/panel.py | 18 +++++------------- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/nyx/util/panel.py b/nyx/util/panel.py index 10727c8..283a16a 100644 --- a/nyx/util/panel.py +++ b/nyx/util/panel.py @@ -624,7 +624,7 @@ class Panel(object): return x, y - def getstr(self, y, x, initial_text = '', text_format = None, max_width = None): + def getstr(self, y, x, initial_text = ''): """ Provides a text field where the user can input a string, blocking until they've done so and returning the result. If the user presses escape then @@ -639,13 +639,8 @@ class Panel(object): y - vertical location x - horizontal location initial_text - starting text in this field - text_format - format used for the text - max_width - maximum width for the text field """ - if not text_format: - text_format = curses.A_NORMAL - # makes cursor visible try: @@ -657,20 +652,17 @@ class Panel(object): display_width = self.get_preferred_size()[1] - if max_width: - display_width = min(display_width, max_width + x) - input_subwindow = self.parent.subwin(1, display_width - x, self.top + y, self.left + x) # blanks the field's area, filling it with the font in case it's hilighting input_subwindow.clear() - input_subwindow.bkgd(' ', text_format) + input_subwindow.bkgd(' ', curses.A_NORMAL) # prepopulates the initial text if initial_text: - input_subwindow.addstr(0, 0, initial_text[:display_width - x - 1], text_format) + input_subwindow.addstr(0, 0, initial_text[:display_width - x - 1], curses.A_NORMAL) # Displays the text field, blocking until the user's done. This closes the # text panel and returns user_input to the initial text if the user presses @@ -680,9 +672,9 @@ class Panel(object): validator = BasicValidator() - textbox.win.attron(text_format) + textbox.win.attron(curses.A_NORMAL) user_input = textbox.edit(lambda key: validator.validate(key, textbox)).strip() - textbox.win.attroff(text_format) + textbox.win.attroff(curses.A_NORMAL) if textbox.lastcmd == curses.ascii.BEL: user_input = None

1 0

[webwml/master] Add new Tor Browser versions
by gk＠torproject.org 15 Feb '16

15 Feb '16

commit e85cab01121b8d972e7e0d8c22e6b0428f783966 Author: Georg Koppen <gk(a)torproject.org> Date: Mon Feb 15 13:40:57 2016 +0000 Add new Tor Browser versions --- include/versions.wmi | 10 +++++----- projects/torbrowser/RecommendedTBBVersions | 8 +++++++- 2 files changed, 12 insertions(+), 6 deletions(-) diff --git a/include/versions.wmi b/include/versions.wmi index 3f59644..f135692 100644 --- a/include/versions.wmi +++ b/include/versions.wmi @@ -5,19 +5,19 @@ <define-tag version-torbrowserdevelopbranch whitespace=delete>maint-5.5</define-tag> <define-tag version-torbrowserbundledir whitespace=delete>5.5.2</define-tag> -<define-tag version-torbrowserbundlebetadir whitespace=delete>6.0a1</define-tag> +<define-tag version-torbrowserbundlebetadir whitespace=delete>6.0a2</define-tag> <define-tag version-torbrowserbundle whitespace=delete>5.5.2</define-tag> -<define-tag version-torbrowserbundlebeta whitespace=delete>6.0a1</define-tag> +<define-tag version-torbrowserbundlebeta whitespace=delete>6.0a2</define-tag> <define-tag version-torbrowserbundlelinux32 whitespace=delete>5.5.2</define-tag> <define-tag version-torbrowserbundlelinux64 whitespace=delete>5.5.2</define-tag> -<define-tag version-torbrowserbundlelinux32beta whitespace=delete>6.0a1</define-tag> -<define-tag version-torbrowserbundlelinux64beta whitespace=delete>6.0a1</define-tag> +<define-tag version-torbrowserbundlelinux32beta whitespace=delete>6.0a2</define-tag> +<define-tag version-torbrowserbundlelinux64beta whitespace=delete>6.0a2</define-tag> <define-tag version-torbrowserbundleosx32 whitespace=delete>5.5.2</define-tag> <define-tag version-torbrowserbundleosx64 whitespace=delete>5.5.2</define-tag> -<define-tag version-torbrowserbundleosx64beta whitespace=delete>6.0a1</define-tag> +<define-tag version-torbrowserbundleosx64beta whitespace=delete>6.0a2</define-tag> <define-tag package-win32-stable whitespace=delete>../dist/torbrowser/<version-torbrowserbundledir>/tor-win32-<version-win32-stable>.zip</define-tag> diff --git a/projects/torbrowser/RecommendedTBBVersions b/projects/torbrowser/RecommendedTBBVersions index c7e8114..7fabb7e 100644 --- a/projects/torbrowser/RecommendedTBBVersions +++ b/projects/torbrowser/RecommendedTBBVersions @@ -7,6 +7,12 @@ "6.0a1-Linux", "6.0a1-MacOS", "6.0a1-Windows", +"6.0a2", +"6.0a2-Linux", +"6.0a2-MacOS", +"6.0a2-Windows", "6.0a1-hardened", -"6.0a1-hardened-Linux" +"6.0a1-hardened-Linux", +"6.0a2-hardened", +"6.0a2-hardened-Linux" ]

1 0