-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump dub to v1.31.1 #180
Bump dub to v1.31.1 #180
Conversation
There's also https://github.com/atilaneves/reggae/blob/master/dub.json#L16 |
Oh right. But goddammit, more things to fix ( |
The bug seems to be that Reggae uses a |
No idea. |
With DMD, disabling the invariants for the unittest runner is required to overcome a missing symbol (an invariant) for dub's vendored `std.sumtype` copy.
Almost working now; hitting long-path regressions on Windows for the dub-generated
This full 64-chars build hash already claims ~25% of the max length. |
Don't rely on the Phobos `dirEntries()` order; it's e.g. problematic for the `tests.it.rules.static_lib.template` test.
The error above can't really be handled on our side - dub tries to generate this file, as part of generating the CI now works except for DMD on Windows - the |
…ld` with DMD on Windows
As I've just recently hit long-path issues on Windows for this reason, especially when the absolute path of the main package is a bit longer. Also skip the `__dub__` intermediate directory, as all dub packages now produce object files in `<target .obj dir>/<dub package name>/…`.
We have a fix in master, I can just backport this if you'd like ? Although it looks like you worked around the bug. |
Oh cool, looking very much forward to dlang/dub#2589. - Yeah, I've worked around the CI problem now. I've also pushed an unrelated commit (c2cbf0d), which I hope fixes the long-path issues for some SIL plugins on Windows - where I've had to stick to dub for now. I've tested this reggae build with a few SIL subprojects (REPL, lang, a plugin); all working. |
No description provided.