From 661215788752bb7b06e8ba16ac53051a67a451b3 Mon Sep 17 00:00:00 2001 From: kali Date: Thu, 11 Oct 2012 04:36:01 +0900 Subject: another attempt at fixing recursion error with expanduser --- src/leap/base/config.py | 24 ++++++++++++++++++++---- 1 file changed, 20 insertions(+), 4 deletions(-) (limited to 'src/leap/base/config.py') diff --git a/src/leap/base/config.py b/src/leap/base/config.py index 57f9f1b7..776c0a49 100644 --- a/src/leap/base/config.py +++ b/src/leap/base/config.py @@ -220,10 +220,26 @@ def get_config_dir(): # check for $XDG_CONFIG_HOME var? # get a more sensible path for win/mac # kclair: opinion? ^^ - return os.path.expanduser( - os.path.join('~', - '.config', - 'leap')) + + # XXX DEBUG for #744 + #logger.debug('expanduser? --- %s', os.path.expanduser('~')) + logger.debug('$HOME? --- %s', os.environ.get('HOME', None)) + logger.debug('user? --- %s', os.getlogin()) + + try: + return os.path.expanduser( + os.path.join('~', + '.config', + 'leap')) + except RuntimeError: + # We're getting a recursion error + # that I suspect is caused by some bug on + # expanduser... + return os.path.join( + 'home', + '%s' % os.getlogin(), + '.config', + 'leap') def get_config_file(filename, folder=None): -- cgit v1.2.3 From 272d72506d667ae4dc8719066422f782983e6746 Mon Sep 17 00:00:00 2001 From: kali Date: Thu, 11 Oct 2012 04:53:22 +0900 Subject: removed debug info since eval HOME is causing the bug --- src/leap/base/config.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/leap/base/config.py') diff --git a/src/leap/base/config.py b/src/leap/base/config.py index 776c0a49..642470e6 100644 --- a/src/leap/base/config.py +++ b/src/leap/base/config.py @@ -223,8 +223,8 @@ def get_config_dir(): # XXX DEBUG for #744 #logger.debug('expanduser? --- %s', os.path.expanduser('~')) - logger.debug('$HOME? --- %s', os.environ.get('HOME', None)) - logger.debug('user? --- %s', os.getlogin()) + #logger.debug('$HOME? --- %s', os.environ.get('HOME', None)) + #logger.debug('user? --- %s', os.getlogin()) try: return os.path.expanduser( -- cgit v1.2.3 From a964b9e55a53b1d87d296e0f3b5295b0366c5967 Mon Sep 17 00:00:00 2001 From: kali Date: Thu, 11 Oct 2012 07:22:49 +0900 Subject: comment out the set_password call until we fix #744 --- src/leap/base/config.py | 23 ++++------------------- 1 file changed, 4 insertions(+), 19 deletions(-) (limited to 'src/leap/base/config.py') diff --git a/src/leap/base/config.py b/src/leap/base/config.py index 642470e6..cf01d1aa 100644 --- a/src/leap/base/config.py +++ b/src/leap/base/config.py @@ -221,25 +221,10 @@ def get_config_dir(): # get a more sensible path for win/mac # kclair: opinion? ^^ - # XXX DEBUG for #744 - #logger.debug('expanduser? --- %s', os.path.expanduser('~')) - #logger.debug('$HOME? --- %s', os.environ.get('HOME', None)) - #logger.debug('user? --- %s', os.getlogin()) - - try: - return os.path.expanduser( - os.path.join('~', - '.config', - 'leap')) - except RuntimeError: - # We're getting a recursion error - # that I suspect is caused by some bug on - # expanduser... - return os.path.join( - 'home', - '%s' % os.getlogin(), - '.config', - 'leap') + return os.path.expanduser( + os.path.join('~', + '.config', + 'leap')) def get_config_file(filename, folder=None): -- cgit v1.2.3