Merge pull request #4 from wting/master

Update from upstream
2024-10-27 20:34:07 +00:00 · 2016-06-21 15:53:42 -07:00 · 2016-06-21 15:53:42 -07:00 · 3090963beb
commit 3090963beb
parent f26a38e0f9 ecd272e979
23 changed files with 190 additions and 127 deletions
--- a/.gitignore
+++ b/.gitignore
@ -1,3 +1,4 @@
+.cache
 .coverage
 *.pyc
 __pycache__
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@ -0,0 +1,29 @@
+-   repo: git@github.com:pre-commit/pre-commit-hooks.git
+    sha: 6f2b0a27e5b9047c6c067fb3d575ba323d572793
+    hooks:
+    -   id: autopep8-wrapper
+        args:
+        - --in-place
+        - --aggressive
+        - --aggressive
+    -   id: check-added-large-files
+    -   id: check-ast
+    -   id: check-case-conflict
+    -   id: check-docstring-first
+    -   id: debug-statements
+    -   id: double-quote-string-fixer
+    -   id: end-of-file-fixer
+    -   id: fix-encoding-pragma
+    -   id: flake8
+        args:
+            - --max-complexity=10
+            - --max-line-length=130
+            - --ignore=E126,E128,E731
+            - --exclude=bin/autojump_argparse.py
+    -   id: requirements-txt-fixer
+    -   id: trailing-whitespace
+-   repo: git@github.com:asottile/reorder_python_imports.git
+    sha: 017e2f64306853ec7f000db52b8280da27eb3b96
+    hooks:
+    -   id: reorder-python-imports
+        language_version: python2.7
--- a/CHANGES.md
+++ b/CHANGES.md
@ -1,6 +1,11 @@
 ## Summary of release changes, see commit history for more details:
 ## https://github.com/joelthelion/autojump/commits/master/

+### Release v22.3.0:
+- use colors only if stdout is a terminal
+- updated RedHat docs
+- misc bug fixes for fish and Clink versions
+
 ### Release v22.2.2:

 #### Backwards Incompatible
--- a/12
+++ b/12
@ -13,8 +13,8 @@ docs:
 	pandoc -s -w man docs/manpage_header.md docs/header.md docs/body.md -o docs/autojump.1
 	pandoc -s -w markdown docs/header.md docs/install.md docs/body.md -o README.md

-lint:
-	@flake8 ./ --config=tox.ini
+pre-commit:
+	@tox -e pre-commit -- install -f --install-hooks

 release: docs
 	# Check for tag existence
@ -38,6 +38,10 @@ tar:
 	git archive --format=tar --prefix autojump_v$(VERSION)/ $(TAGNAME) | gzip > autojump_v$(VERSION).tar.gz
 	sha1sum autojump_v$(VERSION).tar.gz

-test: lint
-	@find . -type f -iname "*.pyc" -delete
+test: pre-commit
+	@find . -type f -iname '*.py[co]' -delete
 	tox
+
+test-fast: pre-commit
+	@find . -type f -iname '*.py[co]' -delete
+	tox -e py27
--- a/bin/autojump
+++ b/bin/autojump
@ -60,7 +60,7 @@ from autojump_utils import sanitize
 from autojump_utils import take
 from autojump_utils import unico

-VERSION = '22.2.4'
+VERSION = '22.3.1'
 FUZZY_MATCH_THRESHOLD = 0.6
 TAB_ENTRIES_COUNT = 9
 TAB_SEPARATOR = '__'
--- a/bin/autojump.bash
+++ b/bin/autojump.bash
@ -61,7 +61,11 @@ j() {

    output="$(autojump ${@})"
    if [[ -d "${output}" ]]; then
-        echo -e "\\033[31m${output}\\033[0m"
+        if [ -t 1 ]; then  # if stdout is a terminal, use colors
+                echo -e "\\033[31m${output}\\033[0m"
+        else
+                echo -e "${output}"
+        fi
        cd "${output}"
    else
        echo "autojump: directory '${@}' not found"
--- a/bin/autojump.bat
+++ b/bin/autojump.bat
@ -1,2 +1,2 @@
@echo off
-python %~dp0\autojump %*
+python "%~dp0\autojump" %*
--- a/bin/autojump.lua
+++ b/bin/autojump.lua
@ -1,13 +1,15 @@
+local AUTOJUMP_DIR = debug.getinfo(1, "S").source:match[[^@?(.*[\/])[^\/]-$]] .. "..\\AutoJump"
+local AUTOJUMP_BIN_DIR = AUTOJUMP_DIR .. "\\bin"
 local AUTOJUMP_BIN = (AUTOJUMP_BIN_DIR or clink.get_env("LOCALAPPDATA") .. "\\autojump\\bin") .. "\\autojump"

 function autojump_add_to_database() 
-  os.execute("python " .. AUTOJUMP_BIN .. " --add " .. clink.get_cwd() .. " 2> " .. clink.get_env("TEMP") .. "\\autojump_error.txt")
+  os.execute("python " .. "\"" .. AUTOJUMP_BIN .. "\"" .. " --add " .. "\"" .. clink.get_cwd() .. "\"" .. " 2> " .. clink.get_env("TEMP") .. "\\autojump_error.txt")
 end

 clink.prompt.register_filter(autojump_add_to_database, 99)

 function autojump_completion(word)
-  for line in io.popen("python " .. AUTOJUMP_BIN .. " --complete " .. word):lines() do
+  for line in io.popen("python " .. "\"" .. AUTOJUMP_BIN .. "\"" ..  " --complete " .. word):lines() do
    clink.add_match(line)
  end
  return {} 
--- a/bin/autojump.zsh
+++ b/bin/autojump.zsh
@ -52,7 +52,11 @@ j() {
    setopt localoptions noautonamedirs
    local output="$(autojump ${@})"
    if [[ -d "${output}" ]]; then
-        echo -e "\\033[31m${output}\\033[0m"
+        if [ -t 1 ]; then  # if stdout is a terminal, use colors
+                echo -e "\\033[31m${output}\\033[0m"
+        else
+                echo -e "${output}"
+        fi
        cd "${output}"
    else
        echo "autojump: directory '${@}' not found"
--- a/bin/autojump_argparse.py
+++ b/bin/autojump_argparse.py
@ -1,5 +1,5 @@
-# Author: Steven J. Bethard <steven.bethard@gmail.com>.
 # -*- coding: utf-8 -*-
+# Author: Steven J. Bethard <steven.bethard@gmail.com>.
 # flake8: noqa
 """Command-line parsing library

--- a/bin/autojump_data.py
+++ b/bin/autojump_data.py
@ -2,26 +2,27 @@
 # -*- coding: utf-8 -*-
 from __future__ import print_function

-from codecs import open
-from collections import namedtuple
 import os
 import shutil
 import sys
+from codecs import open
+from collections import namedtuple
 from tempfile import NamedTemporaryFile
 from time import time

+from autojump_utils import create_dir
+from autojump_utils import is_osx
+from autojump_utils import is_python3
+from autojump_utils import move_file
+from autojump_utils import unico
+
+
 if sys.version_info[0] == 3:
    ifilter = filter
    imap = map
 else:
-    from itertools import ifilter
-    from itertools import imap
-
-from autojump_utils import create_dir
-from autojump_utils import unico
-from autojump_utils import is_osx
-from autojump_utils import is_python3
-from autojump_utils import move_file
+    from itertools import ifilter  # noqa
+    from itertools import imap  # noqa


 BACKUP_THRESHOLD = 24 * 60 * 60
@ -95,7 +96,7 @@ def migrate_osx_xdg_data(config):
    Older versions incorrectly used Linux XDG_DATA_HOME paths on OS X. This
    migrates autojump files from ~/.local/share/autojump to ~/Library/autojump
    """
-    assert is_osx(), "This function should only be run on OS X."
+    assert is_osx(), 'This function should only be run on OS X.'

    xdg_data_home = os.path.join(os.path.expanduser('~'), '.local', 'share')
    xdg_aj_home = os.path.join(xdg_data_home, 'autojump')
@ -125,12 +126,12 @@ def save(config, data):

        with open(temp.name, 'w', encoding='utf-8', errors='replace') as f:
            for path, weight in data.items():
-                f.write(unico("%s\t%s\n" % (weight, path)))
+                f.write(unico('%s\t%s\n' % (weight, path)))

            f.flush()
            os.fsync(f)
    except IOError as ex:
-        print("Error saving autojump data (disk full?)" % ex, file=sys.stderr)
+        print('Error saving autojump data (disk full?)' % ex, file=sys.stderr)
        sys.exit(1)

    # move temp_file -> autojump.txt
--- a/bin/autojump_utils.py
+++ b/bin/autojump_utils.py
@ -1,16 +1,15 @@
 #!/usr/bin/env python
 # -*- coding: utf-8 -*-
-
 from __future__ import print_function

 import errno
-from itertools import islice
 import os
 import platform
 import re
 import shutil
 import sys
 import unicodedata
+from itertools import islice

 if sys.version_info[0] == 3:
    imap = map
@ -75,7 +74,7 @@ def get_pwd():
    try:
        return os.getcwdu()
    except OSError:
-        print("Current directory no longer exists.", file=sys.stderr)
+        print('Current directory no longer exists.', file=sys.stderr)
        raise


@ -141,7 +140,7 @@ def move_file(src, dst):


 def print_entry(entry):
-    print_local("%.1f:\t%s" % (entry.weight, entry.path))
+    print_local('%.1f:\t%s' % (entry.weight, entry.path))


 def print_local(string):
--- a/bin/j.bat
+++ b/bin/j.bat
@ -3,7 +3,7 @@ setlocal EnableDelayedExpansion

 echo %*|>nul findstr /rx \-.*
 if ERRORLEVEL 1 (
-  for /f %%i in ('python %~dp0\autojump %*') do set new_path=%%i
+  for /f %%i in ('python "%~dp0\autojump" %*') do set new_path=%%i
  if exist !new_path!\nul (
    echo !new_path!
 	pushd !new_path!
@ -17,5 +17,5 @@ if ERRORLEVEL 1 (
    echo try `autojump --help` for more information
  )
 ) else (
-  python %~dp0\autojump %* 
+  python "%~dp0\autojump" %*
 )
--- a/bin/jc.bat
+++ b/bin/jc.bat
@ -2,7 +2,7 @@

 echo %*|>nul findstr /rx \-.*
 if ERRORLEVEL 1 (
-  %~dp0\j.bat %cd% %*
+  "%~dp0\j.bat" "%cd%" %*
 ) else (
-  python %~dp0\autojump %* 
+  python "%~dp0\autojump" %*
 )
--- a/bin/jco.bat
+++ b/bin/jco.bat
@ -2,7 +2,7 @@

 echo %*|>nul findstr /rx \-.*
 if ERRORLEVEL 1 (
-  %~dp0\jc.bat %cd% %*
+  "%~dp0\jc.bat" "%cd%" %*
 ) else (
-  python %~dp0\autojump %* 
+  python "%~dp0\autojump" %*
 )
--- a/bin/jo.bat
+++ b/bin/jo.bat
@ -3,7 +3,7 @@ setlocal EnableDelayedExpansion

 echo %*|>nul findstr /rx \-.*
 if ERRORLEVEL 1 (
-  for /f %%i in ('python %~dp0\autojump %*') do set new_path=%%i
+  for /f %%i in ('python "%~dp0\autojump" %*') do set new_path=%%i
  if exist !new_path!\nul (
    start !new_path!
  ) else (
@ -11,5 +11,5 @@ if ERRORLEVEL 1 (
    echo try `autojump --help` for more information
  )
 ) else (
-  python %~dp0\autojump %* 
+  python "%~dp0\autojump" %*
 )
--- a/dev-requirements.txt
+++ b/dev-requirements.txt
@ -1,4 +0,0 @@
-flake8
-mock
-pytest
-tox
--- a/install.py
+++ b/install.py
@ -8,13 +8,13 @@ import shutil
 import sys

 sys.path.append('bin')
-from autojump_argparse import ArgumentParser
+from autojump_argparse import ArgumentParser  # noqa

 SUPPORTED_SHELLS = ('bash', 'zsh', 'fish', 'tcsh')


 def cp(src, dest, dryrun=False):
-    print("copying file: %s -> %s" % (src, dest))
+    print('copying file: %s -> %s' % (src, dest))
    if not dryrun:
        shutil.copy(src, dest)

@ -24,18 +24,18 @@ def get_shell():


 def mkdir(path, dryrun=False):
-    print("creating directory:", path)
+    print('creating directory:', path)
    if not dryrun and not os.path.exists(path):
        os.makedirs(path)


 def modify_autojump_sh(etc_dir, share_dir, dryrun=False):
    """Append custom installation path to autojump.sh"""
-    custom_install = "\
+    custom_install = '\
        \n# check custom install \
        \nif [ -s %s/autojump.${shell} ]; then \
        \n    source %s/autojump.${shell} \
-        \nfi\n" % (share_dir, share_dir)
+        \nfi\n' % (share_dir, share_dir)

    with open(os.path.join(etc_dir, 'autojump.sh'), 'a') as f:
        f.write(custom_install)
@ -44,8 +44,8 @@ def modify_autojump_sh(etc_dir, share_dir, dryrun=False):
 def modify_autojump_lua(clink_dir, bin_dir, dryrun=False):
    """Prepend custom AUTOJUMP_BIN_DIR definition to autojump.lua"""
    custom_install = "local AUTOJUMP_BIN_DIR = \"%s\"\n" % bin_dir.replace(
-        "\\",
-        "\\\\")
+        '\\',
+        '\\\\')
    clink_file = os.path.join(clink_dir, 'autojump.lua')
    with open(clink_file, 'r') as f:
        original = f.read()
@ -60,7 +60,7 @@ def parse_arguments():  # noqa
            'autojump')
    else:
        default_user_destdir = os.path.join(
-            os.path.expanduser("~"),
+            os.path.expanduser('~'),
            '.autojump')
    default_user_prefix = ''
    default_user_zshshare = 'functions'
@ -73,10 +73,10 @@ def parse_arguments():  # noqa
        description='Installs autojump globally for root users, otherwise \
            installs in current user\'s home directory.')
    parser.add_argument(
-        '-n', '--dryrun', action="store_true", default=False,
+        '-n', '--dryrun', action='store_true', default=False,
        help='simulate installation')
    parser.add_argument(
-        '-f', '--force', action="store_true", default=False,
+        '-f', '--force', action='store_true', default=False,
        help='skip root user, shell type, Python version checks')
    parser.add_argument(
        '-d', '--destdir', metavar='DIR', default=default_user_destdir,
@ -91,28 +91,28 @@ def parse_arguments():  # noqa
        '-c', '--clinkdir', metavar='DIR', default=default_clink_dir,
        help='set clink directory location to DIR (Windows only)')
    parser.add_argument(
-        '-s', '--system', action="store_true", default=False,
+        '-s', '--system', action='store_true', default=False,
        help='install system wide for all users')

    args = parser.parse_args()

    if not args.force:
        if sys.version_info[0] == 2 and sys.version_info[1] < 6:
-            print("Python v2.6+ or v3.0+ required.", file=sys.stderr)
+            print('Python v2.6+ or v3.0+ required.', file=sys.stderr)
            sys.exit(1)
        if args.system:
            if platform.system() == 'Windows':
-                print("System-wide installation is not supported on Windows.",
+                print('System-wide installation is not supported on Windows.',
                      file=sys.stderr)
                sys.exit(1)
            elif os.geteuid() != 0:
-                print("Please rerun as root for system-wide installation.",
+                print('Please rerun as root for system-wide installation.',
                      file=sys.stderr)
                sys.exit(1)

        if platform.system() != 'Windows' \
                and get_shell() not in SUPPORTED_SHELLS:
-            print("Unsupported shell: %s" % os.getenv('SHELL'),
+            print('Unsupported shell: %s' % os.getenv('SHELL'),
                  file=sys.stderr)
            sys.exit(1)

@ -125,7 +125,7 @@ def parse_arguments():  # noqa

    if args.system:
        if args.custom_install:
-            print("Custom paths incompatible with --system option.",
+            print('Custom paths incompatible with --system option.',
                  file=sys.stderr)
            sys.exit(1)

@ -138,34 +138,34 @@ def parse_arguments():  # noqa

 def show_post_installation_message(etc_dir, share_dir, bin_dir):
    if platform.system() == 'Windows':
-        print("\nPlease manually add %s to your user path" % bin_dir)
+        print('\nPlease manually add %s to your user path' % bin_dir)
    else:
        if get_shell() == 'fish':
            aj_shell = '%s/autojump.fish' % share_dir
-            source_msg = "if test -f %s; . %s; end" % (aj_shell, aj_shell)
+            source_msg = 'if test -f %s; . %s; end' % (aj_shell, aj_shell)
            rcfile = '~/.config/fish/config.fish'
        else:
            aj_shell = '%s/autojump.sh' % etc_dir
-            source_msg = "[[ -s %s ]] && source %s" % (aj_shell, aj_shell)
+            source_msg = '[[ -s %s ]] && source %s' % (aj_shell, aj_shell)

            if platform.system() == 'Darwin' and get_shell() == 'bash':
                rcfile = '~/.profile'
            else:
                rcfile = '~/.%src' % get_shell()

-        print("\nPlease manually add the following line(s) to %s:" % rcfile)
+        print('\nPlease manually add the following line(s) to %s:' % rcfile)
        print('\n\t' + source_msg)
        if get_shell() == 'zsh':
-            print("\n\tautoload -U compinit && compinit -u")
+            print('\n\tautoload -U compinit && compinit -u')

-    print("\nPlease restart terminal(s) before running autojump.\n")
+    print('\nPlease restart terminal(s) before running autojump.\n')


 def main(args):
    if args.dryrun:
-        print("Installing autojump to %s (DRYRUN)..." % args.destdir)
+        print('Installing autojump to %s (DRYRUN)...' % args.destdir)
    else:
-        print("Installing autojump to %s ..." % args.destdir)
+        print('Installing autojump to %s ...' % args.destdir)

    bin_dir = os.path.join(args.destdir, args.prefix, 'bin')
    etc_dir = os.path.join(args.destdir, 'etc', 'profile.d')
@ -212,5 +212,5 @@ def main(args):
    show_post_installation_message(etc_dir, share_dir, bin_dir)


-if __name__ == "__main__":
+if __name__ == '__main__':
    sys.exit(main(parse_arguments()))
--- a/requirements-dev.txt
+++ b/requirements-dev.txt
@ -0,0 +1 @@
+tox
--- a/tests/autojump_utils_test.py
+++ b/tests/autojump_utils_test.py
@ -7,19 +7,19 @@ import mock
 import pytest

 sys.path.append(os.path.join(os.getcwd(), 'bin'))
-import autojump_utils
-from autojump_utils import encode_local
-from autojump_utils import first
-from autojump_utils import get_tab_entry_info
-from autojump_utils import has_uppercase
-from autojump_utils import in_bash
-from autojump_utils import is_python3
-from autojump_utils import last
-from autojump_utils import sanitize
-from autojump_utils import second
-from autojump_utils import surround_quotes
-from autojump_utils import take
-from autojump_utils import unico
+import autojump_utils  # noqa
+from autojump_utils import encode_local  # noqa
+from autojump_utils import first  # noqa
+from autojump_utils import get_tab_entry_info  # noqa
+from autojump_utils import has_uppercase  # noqa
+from autojump_utils import in_bash  # noqa
+from autojump_utils import is_python3  # noqa
+from autojump_utils import last  # noqa
+from autojump_utils import sanitize  # noqa
+from autojump_utils import second  # noqa
+from autojump_utils import surround_quotes  # noqa
+from autojump_utils import take  # noqa
+from autojump_utils import unico  # noqa


 if is_python3():
@ -37,27 +37,27 @@ def u(string):


 # strings
-@pytest.mark.skipif(is_python3(), reason="Unicode sucks.")
+@pytest.mark.skipif(is_python3(), reason='Unicode sucks.')
@mock.patch.object(sys, 'getfilesystemencoding', return_value='ascii')
 def test_encode_local_ascii(_):
    assert encode_local(u('foo')) == b'foo'


-@pytest.mark.skipif(is_python3(), reason="Unicode sucks.")
-@pytest.mark.xfail(reason="disabled due to pytest bug: https://bitbucket.org/hpk42/pytest/issue/534/pytest-fails-to-catch-unicodedecodeerrors")  # noqa
+@pytest.mark.skipif(is_python3(), reason='Unicode sucks.')
+@pytest.mark.xfail(reason='disabled due to pytest bug: https://bitbucket.org/hpk42/pytest/issue/534/pytest-fails-to-catch-unicodedecodeerrors')  # noqa
@mock.patch.object(sys, 'getfilesystemencoding', return_value='ascii')
 def test_encode_local_ascii_fails(_):
    with pytest.raises(UnicodeDecodeError):
        encode_local(u('日本語'))


-@pytest.mark.skipif(is_python3(), reason="Unicode sucks.")
+@pytest.mark.skipif(is_python3(), reason='Unicode sucks.')
@mock.patch.object(sys, 'getfilesystemencoding', return_value=None)
 def test_encode_local_empty(_):
    assert encode_local(b'foo') == u('foo')


-@pytest.mark.skipif(is_python3(), reason="Unicode sucks.")
+@pytest.mark.skipif(is_python3(), reason='Unicode sucks.')
@mock.patch.object(sys, 'getfilesystemencoding', return_value='utf-8')
 def test_encode_local_unicode(_):
    assert encode_local(b'foo') == u('foo')
@ -86,7 +86,7 @@ def test_sanitize():
    assert sanitize([r'/foo/bar/', r'/']) == [u('/foo/bar'), u('/')]


-@pytest.mark.skipif(is_python3(), reason="Unicode sucks.")
+@pytest.mark.skipif(is_python3(), reason='Unicode sucks.')
 def test_unico():
    assert unico(str('blah')) == u('blah')
    assert unico(str('日本語')) == u('日本語')
--- a/tools/autojump_ipython.py
+++ b/tools/autojump_ipython.py
@ -1,3 +1,4 @@
+# -*- coding: utf-8 -*-
 """
 IPython autojump magic

@ -13,8 +14,8 @@ And copy this file into the "startup" folder of your new profile (e.g.

@TODO: extend %cd to call "autojump -a"
 """
-from subprocess import Popen
 from subprocess import PIPE
+from subprocess import Popen

 from IPython.core.magic import register_line_magic

@ -25,9 +26,9 @@ ip = get_ipython()  # noqa
 def j(path):
    cmd = ['autojump'] + path.split()
    newpath = Popen(
-            cmd,
-            stdout=PIPE,
-            shell=False).communicate()[0].strip()
+        cmd,
+        stdout=PIPE,
+        shell=False).communicate()[0].strip()

    if newpath:
        ip.magic('cd %s' % newpath.decode('utf-8'))
--- a/tox.ini
+++ b/tox.ini
@ -1,32 +1,40 @@
 [tox]
 envlist =
-	py26,
-	py27,
-	py32,
-	py33,
-	py34
+    py26,
+    py27,
+    py32,
+    py33,
+    py34
 # ignore missing setup.py
 skipsdist = True

-[testenv]
-deps = -rdev-requirements.txt
-commands = py.test -rsxX -q
+[testenv:py]
+setenv =
+    PYTHONDONTWRITEBYTECODE = 1
+deps =
+    mock
+    coverage
+    ipdb
+    ipython
+    pytest
+commands =
+    coverage run --source=bin/ -m pytest -vv -rxs --tb native -s --durations 10 --strict {posargs:tests}
+    coverage report -m

 [testenv:flake8]
-deps = flake8
-commands = flake8 .
+deps =
+    flake8
+    pyflakes
+    pep8
+    mccabe
+commands =
+    flake8 .

-[flake8]
-filename =
-	*.py,
-	autojump
-ignore =
-	E126,
-	E128
-max-line-length = 79
-max-complexity = 10
-show-pep8 = True
+[testenv:pre-commit]
+deps =
+    pre-commit
+command =
+    pre-commit {posargs}

 [pytest]
-addopts = -rsxX -q
 norecursedirs = .git .tox docs
--- a/uninstall.py
+++ b/uninstall.py
@ -1,6 +1,5 @@
 #!/usr/bin/env python
 # -*- coding: utf-8 -*-
-
 from __future__ import print_function

 import os
@ -9,7 +8,7 @@ import shutil
 import sys

 sys.path.append('bin')
-from autojump_argparse import ArgumentParser
+from autojump_argparse import ArgumentParser  # noqa


 def is_empty_dir(path):
@ -28,10 +27,10 @@ def parse_arguments():
    parser = ArgumentParser(
        description='Uninstalls autojump.')
    parser.add_argument(
-        '-n', '--dryrun', action="store_true", default=False,
+        '-n', '--dryrun', action='store_true', default=False,
        help='simulate installation')
    parser.add_argument(
-        '-u', '--userdata', action="store_true", default=False,
+        '-u', '--userdata', action='store_true', default=False,
        help='delete user data')
    parser.add_argument(
        '-d', '--destdir', metavar='DIR',
@ -61,7 +60,7 @@ def remove_custom_installation(args, dryrun=False):
    if not os.path.exists(share_dir):
        return

-    print("\nFound custom installation...")
+    print('\nFound custom installation...')
    rm(os.path.join(bin_dir, 'autojump'), dryrun)
    rm(os.path.join(bin_dir, 'autojump_data.py'), dryrun)
    rm(os.path.join(bin_dir, 'autojump_utils.py'), dryrun)
@ -94,18 +93,27 @@ def remove_system_installation(dryrun=False):
    default_zshshare = '/usr/share/zsh/site-functions'

    bin_dir = os.path.join(default_destdir, default_prefix, 'bin')
-    doc_dir = os.path.join(default_destdir, default_prefix, 'share', 'man', 'man1')
+    doc_dir = os.path.join(
+        default_destdir,
+        default_prefix,
+        'share',
+        'man',
+        'man1')
    etc_dir = os.path.join(default_destdir, 'etc', 'profile.d')
-    share_dir = os.path.join(default_destdir, default_prefix, 'share', 'autojump')
+    share_dir = os.path.join(
+        default_destdir,
+        default_prefix,
+        'share',
+        'autojump')
    zshshare_dir = os.path.join(default_destdir, default_zshshare)

    if not os.path.exists(share_dir):
        return

-    print("\nFound system installation...")
+    print('\nFound system installation...')

    if os.geteuid() != 0:
-        print("Please rerun as root for system-wide uninstall, skipping...",
+        print('Please rerun as root for system-wide uninstall, skipping...',
              file=sys.stderr)
        return

@ -142,7 +150,7 @@ def remove_user_data(dryrun=False):
                'autojump'))

    if os.path.exists(data_home):
-        print("\nFound user data...")
+        print('\nFound user data...')
        rmdir(data_home, dryrun)


@ -152,10 +160,10 @@ def remove_user_installation(dryrun=False):
                                       'autojump')
        clink_dir = os.path.join(os.getenv('LOCALAPPDATA', ''), 'clink')
    else:
-        default_destdir = os.path.join(os.path.expanduser("~"), '.autojump')
+        default_destdir = os.path.join(os.path.expanduser('~'), '.autojump')

    if os.path.exists(default_destdir):
-        print("\nFound user installation...")
+        print('\nFound user installation...')
        rmdir(default_destdir, dryrun)
        if platform.system() == 'Windows' and os.path.exists(clink_dir):
            rm(os.path.join(clink_dir, 'autojump.lua'), dryrun)
@ -163,23 +171,23 @@ def remove_user_installation(dryrun=False):

 def rm(path, dryrun):
    if os.path.exists(path):
-        print("deleting file:", path)
+        print('deleting file:', path)
        if not dryrun:
            os.remove(path)


 def rmdir(path, dryrun):
    if os.path.exists(path):
-        print("deleting directory:", path)
+        print('deleting directory:', path)
        if not dryrun:
            shutil.rmtree(path)


 def main(args):
    if args.dryrun:
-        print("Uninstalling autojump (DRYRUN)...")
+        print('Uninstalling autojump (DRYRUN)...')
    else:
-        print("Uninstalling autojump...")
+        print('Uninstalling autojump...')

    remove_user_installation(args.dryrun)
    remove_system_installation(args.dryrun)
@ -188,5 +196,5 @@ def main(args):
        remove_user_data(args.dryrun)


-if __name__ == "__main__":
+if __name__ == '__main__':
    sys.exit(main(parse_arguments()))