On 12/18/17 7:20 AM, Szabo Bogdan wrote:
On Saturday, 16 December 2017 at 12:01:49 UTC, Steven Schveighoffer wrote:
On 12/16/17 5:12 AM, bauss wrote:
On Saturday, 16 December 2017 at 08:07:30 UTC, Szabo Bogdan wrote:
On Friday, 15 December 2017 at 21:56:48 UTC, Steven Schveighoffer
wrote:
On 12/15/17 10:08 AM, Kagamin wrote:
Maybe this https://issues.dlang.org/show_bug.cgi?id=18084
Thanks for looking into this. I created a PR to fix.
Szabo, can you please try with this patch and see if it fixes your
issue?
https://github.com/dlang/phobos/pull/5932
-Steve
I have installed DMD 2.77.1 and I can not find the patched file in
the phobos folder... should I try this by building the compiler?
It seams that I can not build phobos without compiling dmd.. or
maybe I don't know how...
Just go and do the changes manually in your local phobos folder,
wherever you have DMD installed, since you can't build phobos or dmd.
Yes, this won't get into the release for a while. So please do this
manually (just copy the file changed by the pull over the existing one).
It's something that doesn't need to be compiled into the phobos
library, as it's a misrepresentation of the actual object when you
compile with unittests, so there is no reason to recompile phobos.
I'm actually a linux and a mac user...
Me too :)
this issue happens on a windows
ci machine at work. I tried to update the file and compile phobos with
that change, and I don't know how to make it work...
Hm... you shouldn't need to recompile phobos, it should be identical
when not in unittest mode.
Just replace the one file and see if the CI starts working.
I don't know where I can find a 64bit toolchain for vs2010... I was
expecting that dmd uses a newer compiler on windows...
I hope someone can help you there, but I would expect most folks here to
either not be Windows developers, or to be using the interactive version
(not CI version). There may be someone with some experience than can help.
I guess that the best approach for me is to wait the next beta release
of dmd and try it then...
That is also an option. However, the PR hasn't been pulled yet...
-Steve
