try windows support

.travis.yml

@@ -2,7 +2,7 @@ sudo: required
 dist: trusty
 language: python
 python:
-  - 2.7
+# - 2.7
   - 3.4
   - 3.5
   - 3.6
@@ -13,6 +13,7 @@ cache: pip
 install:
   - pip install .[dev,test] coveralls
   - bash scripts/install-linters.sh
+  - pip install -r requirements-dev.txt 
 # command to run tests
 before_script:
   - export PATH=$PATH:/tmp/pmd-bin/bin:/usr/local/bin

gitlint/__init__.py

@@ -50,6 +50,8 @@
 
 import docopt
 import termcolor
+import colorama
+colorama.init()
 import yaml
 
 import gitlint.git as git

gitlint/git.py

@@ -72,7 +72,7 @@ def modified_files(root, tracked_only=False, commit=None):
     status_lines = subprocess.check_output([
         'git', 'status', '--porcelain', '--untracked-files=all',
         '--ignore-submodules=all'
-    ]).decode('utf-8').split(os.linesep)
+    ]).decode('utf-8').splitlines()
 
     modes = ['M ', ' M', 'A ', 'AM', 'MM']
     if not tracked_only:
@@ -93,7 +93,7 @@ def _modified_files_with_commit(root, commit):
     status_lines = subprocess.check_output([
         'git', 'diff-tree', '-r', '--root', '--no-commit-id', '--name-status',
         commit
-    ]).decode('utf-8').split(os.linesep)
+    ]).decode('utf-8').splitlines()
 
     modified_file_status = utils.filter_lines(
         status_lines,
@@ -132,8 +132,7 @@ def modified_lines(filename, extra_data, commit=None):
 
     # Split as bytes, as the output may have some non unicode characters.
     blame_lines = subprocess.check_output(
-        ['git', 'blame', '--porcelain', filename]).split(
-            os.linesep.encode('utf-8'))
+        ['git', 'blame', '--porcelain', filename]).splitlines()
     modified_line_numbers = utils.filter_lines(
         blame_lines, commit + br' (?P<line>\d+) (\d+)', groups=('line', ))
 

gitlint/hg.py

@@ -62,8 +62,7 @@ def modified_files(root, tracked_only=False, commit=None):
         command.append('--change=%s' % commit)
 
     # Convert to unicode and split
-    status_lines = subprocess.check_output(command).decode('utf-8').split(
-        os.linesep)
+    status_lines = subprocess.check_output(command).decode('utf-8').splitlines()
 
     modes = ['M', 'A']
     if not tracked_only:
@@ -105,8 +104,7 @@ def modified_lines(filename, extra_data, commit=None):
     command.append(filename)
 
     # Split as bytes, as the output may have some non unicode characters.
-    diff_lines = subprocess.check_output(command).split(
-        os.linesep.encode('utf-8'))
+    diff_lines = subprocess.check_output(command).splitlines()
     diff_line_numbers = utils.filter_lines(
         diff_lines,
         br'@@ -\d+,\d+ \+(?P<start_line>\d+),(?P<lines>\d+) @@',

gitlint/linters.py

@@ -93,7 +93,7 @@ def lint_command(name, program, arguments, filter_regex, filename, lines):
         output = output.decode('utf-8')
         utils.save_output_in_cache(name, filename, output)
 
-    output_lines = output.split(os.linesep)
+    output_lines = output.splitlines()
 
     if lines is None:
         lines_regex = r'\d+'

gitlint/utils.py

@@ -19,6 +19,7 @@
 
 # This can be just pathlib when 2.7 and 3.4 support is dropped.
 import pathlib2 as pathlib
+from shutil import which
 
 
 def filter_lines(lines, filter_regex, groups=None):
@@ -43,22 +44,6 @@ def filter_lines(lines, filter_regex, groups=None):
                 yield tuple(matched_groups.get(group) for group in groups)
 
 
-# TODO(skreft): add test
-def which(program):
-    """Returns a list of paths where the program is found."""
-    if (os.path.isabs(program) and os.path.isfile(program)
-            and os.access(program, os.X_OK)):
-        return [program]
-
-    candidates = []
-    locations = os.environ.get("PATH").split(os.pathsep)
-    for location in locations:
-        candidate = os.path.join(location, program)
-        if os.path.isfile(candidate) and os.access(candidate, os.X_OK):
-            candidates.append(candidate)
-    return candidates
-
-
 def programs_not_in_path(programs):
     """Returns all the programs that are not found in the PATH."""
     return [program for program in programs if not which(program)]
@@ -75,6 +60,7 @@ def _open_for_write(filename):
 def _get_cache_filename(name, filename):
     """Returns the cache location for filename and linter name."""
     filename = os.path.abspath(filename)[1:]
+    filename = filename.lstrip(":\\")  # for windows
     home_folder = os.path.expanduser('~')
     base_cache_dir = os.path.join(home_folder, '.git-lint', 'cache')
 

requirements-dev.txt

@@ -0,0 +1,7 @@
+termcolor
+colorama
+nose
+docopt
+pyfakefs
+pyyaml
+pathlib2

test/unittest/test_utils.py

@@ -151,4 +151,4 @@ def test_which_absolute_path(self):
         self.fs.create_file(filename)
         os.chmod(filename, 0o755)
 
-        self.assertEqual([filename], utils.which(filename))
+        self.assertEqual(filename, utils.which(filename))