Compatibility fixes

cppdep/cppdep.py

@@ -86,7 +86,7 @@ def path_common(paths):
     """Returns common prefix path for the argument absolute normalized paths."""
     if not paths:
         return ''
-    path = os.path.commonprefix(paths)
+    path = os.path.commonprefix(list(paths))
     assert os.path.isabs(path)
     if path[-1] == os.path.sep:
         return os.path.dirname(path)
@@ -372,6 +372,8 @@ def __init__(self, name, group, src_paths, include_paths, alias_paths,
         self.ignore_paths = set()
         self.alias_paths = set()
         self.include_patterns = include_patterns
+        if src_paths is None:
+          raise Exception("No src_paths for {}".format(name))
         self.__init_paths(src_paths, include_paths, alias_paths, ignore_paths)
         self.root = path_common(self.src_paths)
         self.components = []
@@ -395,10 +397,10 @@ def _update(path_container, arg_paths, check_dir=True):
                         '%s is not a directory in %s (group %s).' %
                         (path, self.group.path, self.group.name))
                 if abs_path in path_container:
-                    raise InvalidArgumentError(
-                        '%s is duplicated in %s.%s' %
+                    warn('%s is duplicated in %s.%s' %
                         (abs_path, self.group.name, self.name))
-                path_container.add(abs_path)
+                else:
+                    path_container.add(abs_path)
 
         _update(self.src_paths, src_paths, check_dir=False)
         _update(self.ignore_paths, ignore_paths, check_dir=False)
@@ -809,7 +811,7 @@ def _dep_filter(nodes):
         for group_name, package_group in self.internal_groups.items():
             for pkg_name, package in package_group.packages.items():
                 if not package.components:
-                    assert not package.src_paths
+                    assert not package.src_paths, pkg_name
                     continue
                 printer('\n' + '#' * 80)
                 printer('analyzing dependencies among components in '

requirements-test.txt

@@ -1,2 +1,2 @@
 mock
-pytest
+pytest=3.6

requirements.txt

@@ -1,4 +1,4 @@
-networkx
+networkx=2.0
 pydot
 pydotplus
 PyYAML

test/test_cppdep.py

@@ -21,7 +21,7 @@
 import platform
 import re
 
-import mock
+from unittest import mock
 import pytest
 
 from cppdep import cppdep