-
-
Notifications
You must be signed in to change notification settings - Fork 443
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
coverage combine doesn't work with rcfile #385
Comments
Can you provide more details about how it isn't working? Can you give me the exact contents of your rcfile? |
Original comment by Maoz Guttman (Bitbucket: maozguttman, GitHub: maozguttman) Hi, By saying not working, I mean that none of the .coverage.* files were combined into .coverage file. They were also not deleted after running "coverage combine". .coveragerc file content is as following, where is a full path pointer to a directory in NFS
p.s. I am running it on Linux (Suse10). Thanks, |
@maozguttman Thanks. Do you know if NFS is a key factor here? Have you tried a similar operation not on NFS? Do you mind pasting the exact literal text of the .coveragerc file? I'm looking for any clue at all... :) |
Original comment by Maoz Guttman (Bitbucket: maozguttman, GitHub: maozguttman) It is not related to NFS. I was trying it on /tmp local disk. Here is rc file:
|
Original comment by Maoz Guttman (Bitbucket: maozguttman, GitHub: maozguttman) Do you want me to create a toy example for you? or can you reproduce it without it? Thanks, |
@maozguttman if you could make an example, that would be amazing, thanks! |
Original comment by Maoz Guttman (Bitbucket: maozguttman, GitHub: maozguttman) How to reproduce:
Bug: run above test case with both 4.05a and 4.06a versions. "coverage combine" works in 4.05a, but not in 4.06a |
Original comment by Maoz Guttman (Bitbucket: maozguttman, GitHub: maozguttman) Hi, I attached a test case. How to reproduce:
Bug: run above test case with both 4.05a and 4.06a versions. "coverage combine" works in 4.05a, but not in 4.06a Thanks, |
@maozguttman Thanks! That made it very clear. Fixed in 0007fc5762b6 (bb) |
Original comment by Maoz Guttman (Bitbucket: maozguttman, GitHub: maozguttman) Tnx! Maoz |
Originally reported by Anonymous
Hi,
I have plenty of .coverage.* files and trying to combine them by running:
% coverage combine --rcfile .coveragerc
Where .coveragerc looks like:
[paths]
my_dirs =
<path 1>
<path 2>
It used to work with 4.0a5 but not working with 4.0a6
Thanks,
Maoz
The text was updated successfully, but these errors were encountered: