← Back to index | Original Bugzilla link

Bug 21712 – [REG 2.096.0] sometimes coverage *.lst files are corrupted

Status
RESOLVED
Resolution
FIXED
Severity
regression
Priority
P1
Component
druntime
Product
D
Version
D2
Platform
All
OS
All
Creation time
2021-03-14T07:15:47Z
Last change time
2021-04-18T14:24:32Z
Keywords
pull
Assigned to
No Owner
Creator
Basile-z

Comments

Comment #0 by b2.temp — 2021-03-14T07:15:47Z
The bug is visible in DMD code coverage reports. For this build [1], download the report [2]. Then look for "# path=./src-dmd-scanmach.lst" and observe what follows. The module is not rendered correctly. Code is absent, just a bunck of pipes and zeroes. You might also observe this in the locally generated lst files. It's verified to be a 2.095.1 regression. As I used 2.095 until today on the project[3] where I could observe the same bug. [1]:https://codecov.io/gh/dlang/dmd/commit/6e6f959b343a71160f66441d7c2dcc01836f6961/build [2]:https://codecov.io/api/gh/dlang/dmd/download/build?path=v4/raw/2021-03-13/6FA607F536B7B910555B478CCB721F18/6e6f959b343a71160f66441d7c2dcc01836f6961/d54d067f-b4f8-44fe-b19a-4883fc3d3594.txt [3]:https://community.codecov.io/t/since-a-few-days-uploads-of-d-lst-files-contain-corrupted-data/2651
Comment #1 by b2.temp — 2021-03-14T13:35:13Z
it's finally a 2.096.0 reg.The bug was found 2 says ago, probably that the dockerhub file was the rc or the beta and not previous point release
Comment #2 by dlang-bot — 2021-03-14T14:56:41Z
@TungstenHeart created dlang/druntime pull request #3395 "fix issue 21712 - [REG 2.096.0] sometimes coverage *.lst files are corrupted" fixing this issue: - fix issue 21712 - [REG 2.096.0] sometimes coverage *.lst files are corrupted dont use the same buffer to read+split the LST and read+split the D https://github.com/dlang/druntime/pull/3395
Comment #3 by dlang-bot — 2021-03-16T10:11:42Z
dlang/druntime pull request #3395 "fix issue 21712 - [REG 2.096.0] sometimes coverage *.lst files are corrupted" was merged into stable: - fe1e0ab311d73b4a26f3a716c12815031ade27ba by Basile Burg: fix issue 21712 - [REG 2.096.0] sometimes coverage *.lst files are corrupted dont use the same buffer to read+split the LST and read+split the D https://github.com/dlang/druntime/pull/3395
Comment #4 by dlang-bot — 2021-04-18T14:24:32Z
dlang/druntime pull request #3434 "merge stable" was merged into master: - 84dbd58ca314081372d518dcbc569d2740486b8e by Basile Burg: fix issue 21712 - [REG 2.096.0] sometimes coverage *.lst files are corrupted dont use the same buffer to read+split the LST and read+split the D https://github.com/dlang/druntime/pull/3434