← Back to index | Original Bugzilla link

Bug 21789 – Codecov should use default umask for file permissions

Status: RESOLVED
Resolution: FIXED
Severity: enhancement
Priority: P1
Component: druntime
Product: D
Version: D2
Platform: x86_64
OS: Linux
Creation time: 2021-04-01T15:08:52Z
Last change time: 2021-04-03T05:46:20Z
Keywords: pull
Assigned to: No Owner
Creator: omerfirmak

Comments

Comment #0 by omerfirmak — 2021-04-01T15:08:52Z

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 by dlang-bot — 2021-04-02T06:22:58Z

@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 by dlang-bot — 2021-04-03T05:46:20Z

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