Bug 14063 – Add coverage enforcement for Phobos' posix.mak

Status
NEW
Severity
enhancement
Priority
P4
Component
dmd
Product
D
Version
D2
Platform
x86
OS
Mac OS X
Creation time
2015-01-27T16:24:03Z
Last change time
2024-12-13T18:39:33Z
Assigned to
No Owner
Creator
Andrei Alexandrescu
Moved to GitHub: dmd#18938 →

Comments

Comment #0 by andrei — 2015-01-27T16:24:03Z
win32.mak has -cov=nn to enforce minimum coverage levels are preserved. We should have such in posix.mak as well. Ideally we'd find a simple mechanism by which both makefiles use the same numbers.
Comment #1 by public — 2015-01-28T12:54:29Z
That is a bit more effort-consuming than it may seem. Current win32.mak has hard-coded list of per-module required coverages which is not maintainable. Better approach would be to define Phobos-wide minimal coverage value and apply it to dflags - but that requires actually adding tests to many modules. Which is good on its own but will take time.
Comment #2 by andrei — 2015-01-28T16:48:37Z
A good start is to add it at its current minimum value, and improve it slowly over time. Could you please effect that?
Comment #3 by public — 2015-01-28T19:30:53Z
Quick attempt has resulted in quick ICE : https://issues.dlang.org/show_bug.cgi?id=14071 Kind of blocked until it is fixed.
Comment #4 by code — 2015-04-10T10:46:02Z
*** Issue 10430 has been marked as a duplicate of this issue. ***
Comment #5 by robert.schadek — 2024-12-13T18:39:33Z
THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/dmd/issues/18938 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB