Skip to content
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

Update list of supported compilers #192

Merged
merged 3 commits into from
Apr 16, 2020
Merged

Conversation

Geod24
Copy link
Contributor

@Geod24 Geod24 commented Feb 17, 2020

Since LDC v1.9.0 and v1.14.0 are consistently failing, remove them from the CI.
It's not my preferred solution to remove a failing test, but no one seems interested in fixing this (master has been red for over 2 months), and it seems to be an LDC specific issue (so not a language thing).

See commit-per-commit for an explanation of the whats and whys.

@s-ludwig
Copy link
Member

Hm, am I looking at the right build here?
https://travis-ci.org/vibe-d/vibe-core/builds/642558400
https://ci.appveyor.com/project/s-ludwig/vibe-core/branch/master

Both master builds are green for LDC 1.9.0/1.14.0, AFAICS. There have been two failures for DMD, though. I have restarted them to see whether they are flaky.

@s-ludwig
Copy link
Member

Ah, now I see, are you are looking at #188? I've restarted the failing builds, but it seems like the changes actually do somehow trigger them. I haven't seen those on master and the restarted builds fail in the same way.

@Geod24
Copy link
Contributor Author

Geod24 commented Feb 18, 2020

Yes I was under the impression that it triggered on master as well, but you're right it doesn't, and the change do trigger them. There is no consistent failure, only a somewhat flakey dirwatcher test.

Now looking again at #188 I can't really figure out why this set of change would trigger this bug... But that's a separate topic. Should I just reinstante 1.4.0 and 1.14.0 LDC ?

It is deprecated and hence not required anymore.
They have been consistently failing on the CI for months,
and there doesn't seem to be any interest in fixing it,
however they do block new PRs.
.travis.yml Outdated Show resolved Hide resolved
Update the list to include the latest compilers (DMD 2.091.0 and LDC 1.20.1),
as well as the previous 3 releases, plus the oldest supported release.
Also switch from beta to the latest CI version, which is more informative,
but still allow it to fail so as not to block development.
@Geod24
Copy link
Contributor Author

Geod24 commented Apr 16, 2020

Updated with LDC 1.20.1 and DMD 2.091.0

@Geod24
Copy link
Contributor Author

Geod24 commented Apr 16, 2020

Travis is green, but I've seen spurious failures

@s-ludwig
Copy link
Member

s-ludwig commented Apr 16, 2020

I have restarted them, they happen on the macOS VMs due to their reliance on timing and the huge spikes that occur there. One day I'll fix them with proper synchronization primitives.

@s-ludwig s-ludwig merged commit 071e325 into vibe-d:master Apr 16, 2020
@Geod24 Geod24 deleted the update-compiler branch April 16, 2020 16:00
@Geod24
Copy link
Contributor Author

Geod24 commented Apr 16, 2020

Since I am developing on Mac OSX, I'd love to see that day :D
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants