wilzbach / tools-test

1 stars 0 forks source link

rdmd -of= broken (cf doesn't understand new dmd -of= syntax) #159

Open wilzbach opened 7 years ago

wilzbach commented 7 years ago

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

Original bug ID: BZ#17064 From: Timothee Cour <timothee.cour2@gmail.com> Reported version: D2 CC: dlang-bugzilla@thecybershadow.net, timothee.cour2@gmail.com

wilzbach commented 7 years ago

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

was mentioned in https://issues.dlang.org/show_bug.cgi?id=16962 but that bug was crowded with other issues, and this particular issue is still not fixed:

from that bug: rdmd doesn't seem to understand the new '=' syntax in '-of=' '-od=' even though dmd does (DMD64 D Compiler v2.072.1)

Also there should be some tests (https://github.com/dlang/tools/commit/bfbcd086bad634066ba142964823f1689e879aaf doesn't mention anything about of=/od=)

wilzbach commented 7 years ago

Comment author: Vladimir Panteleev <dlang-bugzilla@thecybershadow.net>

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

Also there should be some tests (https://github.com/dlang/tools/commit/ bfbcd086bad634066ba142964823f1689e879aaf doesn't mention anything about of=/od=)

Look again.

wilzbach commented 7 years ago

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

Where?

I'm seeing: -odsomefile but not -od=somefile, eg:

res = execute([rdmdApp, compilerSwitch, "--build-only", "--force", "-lib", "-od" ~ libDir, srcName]);
wilzbach commented 7 years ago

Comment author: Vladimir Panteleev <dlang-bugzilla@thecybershadow.net>

Oh, I see what you mean now. The problem is with the equal sign.

wilzbach commented 7 years ago

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

ideally, there would be some code sharing with dmd to avoid dmd's flag becoming out of sync with rdmd's flag.

wilzbach commented 7 years ago

Comment author: Vladimir Panteleev <dlang-bugzilla@thecybershadow.net>

Ideally, rdmd would be moved into dmd and left as a compatibility wrapper. That would make it much faster, too.

wilzbach commented 6 years ago

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

is there any update on this? it's too bad dmd and rdmd are not in sync