Bug 10432 – RDMD: --dry-run option tries to read non-existent file

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
All
OS
All
Creation time
2013-06-20T15:39:00Z
Last change time
2013-06-20T15:41:13Z
Keywords
pull
Assigned to
andrej.mitrovich
Creator
andrej.mitrovich

Comments

Comment #0 by andrej.mitrovich — 2013-06-20T15:39:50Z
$ rdmd --dry-run read C:\Users\ADMINI~1\AppData\Local\Temp\.rdmd\rdmd-rdmd_dry_run.d-CAF07156CF44F49E21D3ED1 AC377E211\rdmd.deps std.exception.ErrnoException@std\stdio.d(325): Cannot open file `C:\Users\ADMINI~1\AppData\ Local\Temp\.rdmd\rdmd-rdmd_dry_run.d-CAF07156CF44F49E21D3ED1AC377E211\rdmd.deps' in mode `r b' (No such file or directory) The reason why is this code: ----- immutable depsExitCode = run(depsGetter, depsFilename); if (depsExitCode) { stderr.writeln("Failed: ", escapeShellCommand(depsGetter)); collectException(std.file.remove(depsFilename)); exit(depsExitCode); } return readDepsFile(); ----- If --dry-run is set, the run() function will not call anything, and then readDepsFile will attempt to read a file that it expects was created by the run() call.
Comment #1 by github-bugzilla — 2013-06-20T15:41:02Z
Commits pushed to master at https://github.com/D-Programming-Language/tools https://github.com/D-Programming-Language/tools/commit/dd878ab9c373bec6e4f7ad366d49c106b07d4114 Fixes Issue 10432 - --dry-run option attempts to read non-existent file. https://github.com/D-Programming-Language/tools/commit/bbb63cbd99a3dca50705daaee655215d569d7acc Merge pull request #69 from AndrejMitrovic/Fix10432 Issue 10432 - --dry-run option attempts to read non-existent file.