Combine should allow for recursive searches of data files

Issue #444 resolved
Evan Albright created an issue

Hey there, thanks for the work on this great tool, much obliged.

When using the coverage run with parallel tracking on the resulting ".coverage" files from branch tracking are scattered in hard to predict locations within a directory based on where the sub-tasks were started from. I appreciate the recent addition of the input file and directory names to the combine task but unfortunately that doesn't help my situation.

It would be tremendously helpful to allow the combine command to search recursively for any contained coverage files. I've got a hacky workaround in place now but I'd love to be able to ditch it.

I wasn't sure the best way to approach adding the new feature. In the command line defintion it seems like the thing to do would be to add an Opts.recursive as the positional arguments all seem spoken for as data file addresses or directories to look in. Happy to draft something if you think that's the best way ahead.

Comments (6)

  1. Ned Batchelder repo owner

    @e-m-albright This sounds like it might be a reasonable addition. But I wonder if a simpler solution wouldn't be to set COVERAGE_FILE (or [run]data_file in the rc file) to a path with an absolute directory. Then no matter where the sub-tasks were started, they'd write their data files to a single directory.

  2. Evan Albright reporter

    Ah, that's a good idea. Just to make sure, that isn't currently implemented is it? I can't find anything that seems related. Anyways I'll see if I can't draft something for the change to run.

  3. Ned Batchelder repo owner

    @e-m-albright COVERAGE_FILE is available in the current code, you should be able to just use it.

  4. Log in to comment