fix: remove potential windows newline from the stats

Open louiscaron opened this issue 3 years ago • 2 comments

On windows machines, this could happen, the symptom is a failure message like the following:

FAILED

Test suite:     basedir (line 22)
Test case:      Enabled CCACHE_BASEDIR
Failure reason: Expected direct_cache_hit to be 0, actual 0

Sep 23 '22 13:09 louiscaron

I have split the windows OS related issues in several Pull requests to make the acceptance process faster.

Sep 24 '22 06:09 louiscaron

Thanks for the PR.

A change similar to this is already in PR #1133. I think that it would be better to focus on getting that PR ready instead of redoing the changes in another context. Perhaps you can help out with it?

Sep 24 '22 08:09 jrosdahl

This can be closed now.

Oct 12 '22 19:10 orgads

This PR has been automatically closed due to no response to our request for more information from the PR author. Please feel free to leave a comment to reopen the issue if you have more information.

Dec 01 '22 16:12 ccache-bot[bot]