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 Travis configuration #2312

Merged
merged 5 commits into from Jun 16, 2019

Conversation

@Geod24
Copy link
Contributor

commented May 28, 2019

This does a couple of cleanup to the travis configuration.
See the linked blog posts for the non-D-specific parts.

@Geod24

This comment has been minimized.

Copy link
Contributor Author

commented May 28, 2019

Hum, there's a lot of duplication in the readme as to what's supported.
Shouldn't we state the goal and point to the travis file instead ?

@Geod24 Geod24 force-pushed the Geod24:gdc branch 2 times, most recently from 84d4974 to 46b8559 May 28, 2019
@Geod24 Geod24 changed the title Update Travis configuration & add gdc-9 Update Travis configuration Jun 3, 2019
@Geod24 Geod24 force-pushed the Geod24:gdc branch 2 times, most recently from 1539412 to d884dad Jun 3, 2019
@Geod24

This comment has been minimized.

Copy link
Contributor Author

commented Jun 3, 2019

Well I give up on GDC for the time being...

.travis.yml Outdated Show resolved Hide resolved
@s-ludwig

This comment has been minimized.

Copy link
Member

commented Jun 4, 2019

There is also appveyor.yml with an additional set of duplicate entries, but since updates are relatively rare, it didn't really bother me so far.

@Geod24 Geod24 force-pushed the Geod24:gdc branch from 7e14408 to 0619516 Jun 14, 2019
@s-ludwig

This comment has been minimized.

Copy link
Member

commented Jun 15, 2019

Looks like the DMD 2.085.1/vibe-core failure is genuine, unfortunately, I'd suggest to simply allow it to fail for now:

matrix:
    allow_failures:
        - d: 2.085.1
          env: VIBED_DRIVER=vibe-core

(hopefully got the syntax right there)

@Geod24 Geod24 force-pushed the Geod24:gdc branch from 8e80809 to 75ab0b5 Jun 15, 2019
@s-ludwig

This comment has been minimized.

Copy link
Member

commented Jun 15, 2019

Looks like the DMD 2.085.1/vibe-core failure is genuine, unfortunately, I'd suggest to simply allow it to fail for now:

matrix:
    allow_failures:
        - d: 2.085.1
          env: VIBED_DRIVER=vibe-core

(hopefully got the syntax right there)

@Geod24 Geod24 force-pushed the Geod24:gdc branch from 75ab0b5 to a463871 Jun 16, 2019
Geod24 added 4 commits May 27, 2019
- Drop support for 2.076 (/ LDC 1.6)
- Add testing for LDC 1.15 (FE: 2.085.1)
- Add testing for DMD 2.086 (LDC 1.16 is still in beta)
Trusty reached EOL 2019-04, and Travis is switching to Xenial by default:
https://blog.travis-ci.com/2019-04-15-xenial-default-build-environment
It's quite outdated
Remove supported compiler before v0.8.0
Fix mention of DMD 2.062
@Geod24 Geod24 force-pushed the Geod24:gdc branch from a463871 to 2546c59 Jun 16, 2019
@Geod24

This comment has been minimized.

Copy link
Contributor Author

commented Jun 16, 2019

@s-ludwig : That didn't work. Did a bit of digging and all parts of the build need to be matched, so I added PARTS too but it didn't work either. Why don't we just stick to v2.085.0 ? v2.085.1 is not going to be fixed anyway.

@s-ludwig

This comment has been minimized.

Copy link
Member

commented Jun 16, 2019

Okay, not worth the trouble then, lets stick with 2.085.0.

@s-ludwig s-ludwig merged commit 54273f7 into vibe-d:master Jun 16, 2019
3 checks passed
3 checks passed
codecov/patch Coverage not affected when comparing 0b434ca...2546c59
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@s-ludwig

This comment has been minimized.

Copy link
Member

commented Jun 16, 2019

Finally green 🎉 That looked like a lot more effort than it should be, thanks for finishing this!

@Geod24 Geod24 deleted the Geod24:gdc branch Jun 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.