wilzbach / tools-test

1 stars 0 forks source link

REG: rdmd --eval='writeln();' keeps failing at almost every release, needs a test #152

Closed wilzbach closed 6 years ago

wilzbach commented 7 years ago

Note: the issue was created automatically migrated from https://issues.dlang.org

Original bug ID: BZ#16952 From: Timothee Cour <timothee.cour2@gmail.com> Reported version: D2 CC: ag0aep6g@gmail.com, @wilzbach, timothee.cour2@gmail.com

wilzbach commented 7 years ago

Comment author: Timothee Cour <timothee.cour2@gmail.com>

it seems to break at every release, probably a symptom there's a test missing:

dmd --version DMD64 D Compiler v2.072.1 rdmd '--eval=writeln();' Error: module cstream is in file 'std/cstream.d' which cannot be read

dmd --version DMD64 D Compiler v2.071.2 rdmd '--eval=writeln();' Error: module syserror is in file 'std/syserror.d' which cannot be read

and also earlier versions: https://issues.dlang.org/show_bug.cgi?id=15175 rdmd --loop and --eval now complain about std.stream deprecation warnings https://issues.dlang.org/show_bug.cgi?id=13288 rdmd --eval fails because it still imports std.metastrings https://issues.dlang.org/show_bug.cgi?id=10352 Regression (2.063): --eval is broken in RDMD

wilzbach commented 7 years ago

Comment author: ag0aep6g@gmail.com

(In reply to Timothee Cour from comment BZ#0)

it seems to break at every release, probably a symptom there's a test missing:

dmd --version DMD64 D Compiler v2.072.1 rdmd '--eval=writeln();' Error: module cstream is in file 'std/cstream.d' which cannot be read

So that people don't go and try to fix this: This specific instance has already been fixed in master.

Regarding the issue that this stuff comes up again and again: We do have a test in rdmd_test.d [1], and it does catch this. I think the problem is that the auto-tester doesn't run rdmd_test.

[1] https://github.com/dlang/tools/blob/3a32331303e2e62f429b8136ba0de0e9a9294722/rdmd_test.d#L135-L138

wilzbach commented 7 years ago

Comment author: Timothee Cour <timothee.cour2@gmail.com>

filed https://issues.dlang.org/show_bug.cgi?id=16953: auto-tester doesn't run rdmd_test (causes a number of regressions in each release)

wilzbach commented 6 years ago

Comment author: @wilzbach

TravisCI now runs the testsuite on every day.