D issues are now tracked on GitHub. This Bugzilla instance remains as a read-only archive.
Issue 21789 - Codecov should use default umask for file permissions
Summary: Codecov should use default umask for file permissions
Status: RESOLVED FIXED
Alias: None
Product: D
Classification: Unclassified
Component: druntime (show other issues)
Version: D2
Hardware: x86_64 Linux
: P1 enhancement
Assignee: No Owner
URL:
Keywords: pull
Depends on:
Blocks:
 
Reported: 2021-04-01 15:08 UTC by omerfirmak
Modified: 2021-04-03 05:46 UTC (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this issue.
Description omerfirmak 2021-04-01 15:08:52 UTC 
https://github.com/dlang/druntime/blob/6c3adb102a2a804c802649156ca27cc878aad5d7/src/rt/cover.d#L459-L462

codecov .lst files are created with 0600 by default, they should use umask.
Comment 1 Dlang Bot 2021-04-02 06:22:58 UTC 
@omerfirmak updated dlang/druntime pull request #3421 "Fix Issue 21789: Let umask prune permissions of coverage lst files" fixing this issue:

- Fix Issue 21789: Let umask prune permissions of coverage lst files

https://github.com/dlang/druntime/pull/3421
Comment 2 Dlang Bot 2021-04-03 05:46:20 UTC 
dlang/druntime pull request #3421 "Fix Issue 21789: Let umask prune permissions of coverage lst files" was merged into master:

- d2a185d26b7b0fd1eec9dd0d1a0bf79747ff54a0 by Ömer Faruk IRMAK:
  Fix Issue 21789: Let umask prune permissions of coverage lst files

https://github.com/dlang/druntime/pull/3421