Skip to content
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

Combining parallel files leaves the old data files around. #40

Closed
nedbat opened this issue Dec 29, 2009 · 2 comments
Closed

Combining parallel files leaves the old data files around. #40

nedbat opened this issue Dec 29, 2009 · 2 comments
Labels
enhancement New feature or request

Comments

@nedbat
Copy link
Owner

nedbat commented Dec 29, 2009

From Dirkjan Ochtman (hg dev):

BTW, I modified our coverage.py use to use the parallel files (with
the PID in the filename), but apparently the combination step leaves
behind the old files (leading some people to complain about the disk
use). There wasn't anything in the code that I could use for this, as
far as I could see from a quick glance. Is that right? Might be a
useful option for a future release.


@nedbat
Copy link
Owner Author

nedbat commented Dec 30, 2009

Fixed in <<changeset cd735ea5fc00 (bb)>>. Now all files combined are deleted once done.

@nedbat
Copy link
Owner Author

nedbat commented Feb 25, 2010

Original comment by Geoff Bache (Bitbucket: geoffbache, GitHub: Unknown)


Just saw this. It would be useful if there was a way to get the old behaviour back, perhaps a command line switch. The problem with deleting the source by default is that if you don't get what you expect, it's very difficult to establish what happened.

@nedbat nedbat closed this as completed Feb 25, 2010
@nedbat nedbat added major enhancement New feature or request labels Jun 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant