Bug 4672 – [patch] rdmd fails when -I is needed

Status
RESOLVED
Resolution
FIXED
Severity
normal
Priority
P2
Component
phobos
Product
D
Version
D2
Platform
Other
OS
Windows
Creation time
2010-08-18T01:54:00Z
Last change time
2011-05-14T16:47:52Z
Assigned to
andrei
Creator
bus_dbugzilla

Attachments

IDFilenameSummaryContent-TypeSize
724rdmd.dModified rdmdtext/x-dsrc15220
726rdmd.d.diffSame as above, but in diff formtext/plain1762

Comments

Comment #0 by bus_dbugzilla — 2010-08-18T01:54:34Z
As discussed on the newsgroup: http://www.mail-archive.com/[email protected]/msg35755.html ----------------------------------------- $ cat src/app/main.d module app.main; import foo; void main(){} $ cat src/lib/foo.d module lib.foo; $ xfbuild -Isrc src/app/main.d Works ok $ rdmd -Isrc src/app/main.d dmd craps out because rdmd told it to compile "src/app/src/lib/foo.d" ----------------------------------------- A modified version of rdmd is attached which fixes the above AND ensures that the following still works correctly: ----------------------------------------- $ cat ../baz/main.d import a.b; void main(){} $ cat ../baz/a/b.d module a.b; $ rdmd ../baz/main.d -----------------------------------------
Comment #1 by bus_dbugzilla — 2010-08-18T01:57:19Z
Created attachment 724 Modified rdmd
Comment #2 by bus_dbugzilla — 2010-08-19T00:12:49Z
Created attachment 726 Same as above, but in diff form
Comment #3 by bus_dbugzilla — 2010-08-19T17:28:54Z
Over here is a unified patch against rdmd r1400 for #4672, #4683 and #4684: http://d.puremagic.com/issues/show_bug.cgi?id=4684
Comment #4 by bus_dbugzilla — 2010-09-03T00:05:07Z
There's a typo in the test case above: ----------------------------------------- $ cat src/app/main.d module app.main; import foo; void main(){} ----------------------------------------- Should be: ----------------------------------------- $ cat src/app/main.d module app.main; import lib.foo; void main(){} ----------------------------------------- (ie, Line 2 of src/app/main.d) With that change, the test case fails with rdmd r1400, but works with xfbuild and with the rdmd patch above.
Comment #5 by bus_dbugzilla — 2010-09-24T14:02:34Z
A combined patch for issues #4672, #4683, #4684, #4688, #4928, #4930 is here: http://d.puremagic.com/issues/show_bug.cgi?id=4930
Comment #6 by andrei — 2011-05-14T16:47:52Z