atilaneves / dpp

Directly include C headers in D source code
Boost Software License 1.0
227 stars 31 forks source link

Fix #320 issue #321

Closed denizzzka closed 1 year ago

denizzzka commented 1 year ago

Fix #320

upd: It is necessary to explain why it was done through TemporaryCPATH - this is because if we try to use for testing CLI arguments like:

runPreprocessOnly("issue320.dpp", "--include-path="~buildPath(sandbox.sandboxPath, "some/include/path");

this leads to that directory will be added as last into include dirs list, and first matching directory will always be root of testing sandbox and thus test always passes as good. So here is need this trick for passing include path directly to compiler

denizzzka commented 1 year ago

~master also affected by these errors:

Test it.c.dstep.issues.8 failed.
Test it.issues.303 failed.
denizzzka commented 1 year ago

(squashed to one commit)