search

d-language-server / dls

A Language Server implementation for D
http://dls.dub.pm
106 stars 14 forks source link

D language Project tester SEGFAULT #45

Closed ghost closed 5 years ago

ghost commented 5 years ago

I would like to bring your attention on https://buildkite.com/dlang/dmd/builds, DLS seems to fails since one or two days. It's not yet super visible because people does not always rebase their PR.

https://buildkite.com/dlang/dmd/builds/6674 https://buildkite.com/dlang/dmd/builds/6671 https://buildkite.com/dlang/dmd/builds/6666

LaurentTreguier commented 5 years ago

Once again, I can't manage to reproduce this on my machine... It seems to have no problem with the latest DMD master on my end. I haven't been actively working on this as of late, so there have been very few changes recently. The last version was tagged 4 days ago, the previous one is from 18 days ago, and they are only 3 commits between the two. I've updated the dub and containers dependencies; and the two other commits don't really do much regarding memory management, nothing that should possibly lead to a segfault (as far as I can tell).

I'm thinking DLS should be pulled from the project tester; every time it breaks the CI, it's not because of a compiler issue that's revealed, but only because of a weird circumstance, like having a .editorconfig in dlang/ci and such.

ghost commented 5 years ago

You confirm that you tested with DMD ~master ?

LaurentTreguier commented 5 years ago

Yes, this version to be exact:

$ dmd --version
DMD64 D Compiler v2.086.0-342-g6e78dd19f

(the host compiler used to compile it is DMD 2.086.0)

ghost commented 5 years ago

Thanks, so maybe it's nothing.