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

[LDC] update travis.yml #18

Closed
wants to merge 2 commits into from
Closed

[LDC] update travis.yml #18

wants to merge 2 commits into from

Conversation

9il
Copy link
Member

@9il 9il commented Jan 12, 2015

No description provided.

@9il
Copy link
Member Author

9il commented Jan 12, 2015

Failed because #19.

@kyllingstad
Copy link
Member

I'm not very familiar with Travis yet. Is it possible to specify just DMD and LDC as compilers, rather than specific versions? With this change, I have to remember to update travis.yml every time a new compiler version is released, if I want to test the code with the latest version.

@9il
Copy link
Member Author

9il commented Jan 12, 2015

ping @MartinNowak
I have tried to write - dmd but it seems there is an bug in Travis-CI:

The command "curl  > ~/dmd.zip" failed and exited with 2 during .

If you forget to update .travis.yml then travis will use old version of compiler.
In 99% cases if Travis success with old version then it will success with newer one.

@MartinNowak
Copy link
Contributor

For now only compilers with specific versions are supported (e.g. dmd-2.066.1). This will remain the case until there is a generic way to obtain the latest version of each compiler. The default compiler is dmd-2.066.1. It's often a good idea to maunally update the compiler version, so that you can handle deprecations and such.

@@ -1,4 +1,10 @@
language: d
os:
- linux
- osx
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do you need to test OSX. That wouldn't work with apt-get anyhow.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When (if) Travis will support OS X apt-get raise an error like reminder.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, but that error will be completely unrelated to the pull request that triggers it, and that is unfortunate. If Travis does not support D on OS X, the os: section should just be removed.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It already does support D on OSX, but there isn't anything platform specific in this library, so you shouldn't needlessly waste Travis' resources.

@kyllingstad
Copy link
Member

Ok, I'll merge this as soon as the LDC issue is sorted out and the build succeeds. Thanks!

@9il
Copy link
Member Author

9il commented Mar 1, 2015

This PR is a part of #22

@9il 9il closed this Mar 1, 2015
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.

3 participants