Skip to content

Start testing the DMD DUB package on Travis#7415

Merged
dlang-bot merged 1 commit intodlang:masterfrom
wilzbach:dub_testsuite
Dec 13, 2017
Merged

Start testing the DMD DUB package on Travis#7415
dlang-bot merged 1 commit intodlang:masterfrom
wilzbach:dub_testsuite

Conversation

@wilzbach
Copy link
Contributor

@wilzbach wilzbach commented Dec 9, 2017

Supersedes #6999

Submitting this to see the list of errors we run into. The last time there has been a problem with gdc.

See also: #6771

@dlang-bot
Copy link
Contributor

Thanks for your pull request, @wilzbach!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

@jacob-carlborg
Copy link
Contributor

So why doesn't it find the Dub command? It works before the tests are started [1]. Are we messing with the PATH or something?

[1] https://travis-ci.org/dlang/dmd/jobs/313877077#L903

@wilzbach
Copy link
Contributor Author

wilzbach commented Dec 9, 2017

So why doesn't it find the Dub command? It works before the tests are started [1]. Are we messing with the PATH or something?

Ah because we need to explicitly load the compiler into the environment. I think this is done to ensure that the host compiler is only used in the build step and not by accident in other steps.

@wilzbach wilzbach removed the Review:WIP Work In Progress - not ready for review or pulling label Dec 11, 2017
@wilzbach wilzbach changed the title [WIP] Start testing the DMD DUB package on Travis Start testing the DMD DUB package on Travis Dec 11, 2017
@wilzbach
Copy link
Contributor Author

We are all green :)

image

@jacob-carlborg
Copy link
Contributor

LGTM 👌.

@wilzbach wilzbach added the Review:Blocking Other Work review and pulling should be a priority label Dec 12, 2017
@wilzbach
Copy link
Contributor Author

@RazvanN7 @jacob-carlborg FYI: I managed to build a DUB package which exposes the entire frontend: #7425

@wilzbach wilzbach self-assigned this Dec 13, 2017
@dlang-bot dlang-bot merged commit 7a109b7 into dlang:master Dec 13, 2017
@JackStouffer
Copy link
Contributor

@wilzbach @mdparker BTW, has there been an article on the blog about the dub functionality? If not, we should really advertise this.

@wilzbach wilzbach deleted the dub_testsuite branch December 13, 2017 21:14
@wilzbach
Copy link
Contributor Author

@wilzbach @mdparker BTW, has there been an article on the blog about the dub functionality?

Yes there was:

https://dlang.org/blog/2017/08/01/a-dub-case-study-compiling-dmd-as-a-library

If not, we should really advertise this.

The problem is that's still far from usable. While with #7425, it will finally be possible to run through all semantic stages with the DMD as a library package, there's still much work that needs to be done in:

  • clean, modern D APIs
  • overly complicated "hello world" -> add wrapper
  • semantic versioning for DMD (at the moment one can only use master as other tags are "invalid semver" tags)
  • Improve the easiness of writing a custom visitor (@RazvanN7 is working on this -> https://wiki.dlang.org/Projects/Visitormania)
  • Introduce more structure in the DMD code base & refactorings (e.g. ddmd -> dmd transition, start to use D strings at more places) -> https://wiki.dlang.org/Projects/Libdcomp
  • Provide an example project for a simple use case to get people started (e.g. a DScanner-like style check, C++ header generation, CTags/Etags output, AST XML dump, Find declaration, ...)

@jacob-carlborg
Copy link
Contributor

Provide an example project for a simple use case to get people started (e.g. a DScanner-like style check, C++ header generation, CTags/Etags output, AST XML dump, Find declaration, ...)

I have started on an AST dumper, but as a debugging tool within DMD and not outputting XML.

https://github.com/jacob-carlborg/dmd/tree/ast-printer

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merge:auto-merge Review:Blocking Other Work review and pulling should be a priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants