dlang-community / D-Scanner

Swiss-army knife for D source code
Boost Software License 1.0
240 stars 80 forks source link

Change rdmd to $DC -run #857

Closed WebFreak001 closed 2 years ago

WebFreak001 commented 2 years ago

Fix #821

LDC has the option -run, but GDC doesn't seem to have it yet.

WebFreak001 commented 2 years ago

I'm not sure why CI says "Invalid source/import path: /home/runner/work/D-Scanner/D-Scanner/bin" after running the prebuild command, need to investigate what exactly happens here first.

Also $DC does not seem to work locally, even though it should be valid according to DUB docs. This might be a deal breaker including this.

WebFreak001 commented 2 years ago

the issue is fixed with https://github.com/dlang/dub/pull/2217

For older DUB versions it's possible to run DUB with env DC=dmd (or other) to make it replace the D compiler.

The setup-dlang Action for GH Actions already sets the DC environment variable so the builds will work.

However until the DUB version is released (and we can bump the toolchain requirements), I will leave this unmerged.

rikkimax commented 2 years ago

New dub version was released with PR.