This check us only necessary since the previous check from line 123 isn't effective since the exclude logic is applied afterwards. How about moving the logic into get_file_groups() - which I think would be the most efficient approach since it even avoid crawling / collecting files under an excluded directory. I would expect that would also avoid the need to check all parents.

Nitpick: no parenthesis around Python conditions. The Pythonic way to write this condition would be: if files_check.

Logic moved to get_file_groups() 3b9b453

The exclude logic can already be applied to the dirnames variable to filter them out early (without the need to recurse into them). That will also simplify the check later which doesn't have to consider parents.

moved to dirnames 3b9b453

I don't see where the code filters dirnames based on the excludes?

dirnames only contains the names of the subdirectories of dirpath. As such the condition not in excludes is incorrect again - for the same reasons as the previous comments.

This condition still doesn't make sense.

Please write a unit test which tests a variety of cases to validate the implementation against.

humm, why not? the excludes argument might contains a filename

I don't think an exclude of bar should match a file foo/bar. Instead the exclude should state foo/bar explicitly. Otherwise how do I exclude the file bar but not the file foo/bar?

should I need to include the test?

You don't have to include the test in this PR but as mentioned before writing one will help you to check the logic. Especially the part to consider an example file system structure and excludes and what the filtered result is expected to be.

@dirk-thomas I created a gist with a test: https://gist.github.com/ahcorde/bcc558aedad8560cfa46c4b5de366f32

This is the last commit 6db3ae3

I am not sure how to use the gist. get_file_groups() seems to always return an empty dict?

Just from reading over the logic I would expect that a file include/test/cache_unittest.cpp would be incorrectly skipped even though only test/cache_unittest.cpp is specified in the excludes.

Also I still don't see why is_file_or_directory_to_exclude() should needs any logic like check_path_parents().