Skip to content

Run travis tests for 1.22.0 osx, stable on linux, nightly on osx #721

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

Merged
merged 1 commit into from
Feb 11, 2019

Conversation

erickt
Copy link
Contributor

@erickt erickt commented Jan 29, 2019

Travis supports building on osx, but we aren't running the tests in a few configurations, like OSX with rust 1.22.0, native linux on stable, and OSX on nightly. This adds testers for these situations to make sure that bugs like #720 don't slip into the system again.

Travis supports building on osx, but we aren't running the tests
in a few configurations, like OSX with rust 1.22.0, native linux
on stable, and OSX on nightly. This adds testers for these
situations to make sure that bugs like rust-random#720 don't slip into the
system again.
@erickt
Copy link
Contributor Author

erickt commented Jan 29, 2019

As an alternative, libc has a nice setup for multi architecture testing in rust-lang/libc#1233.

Copy link
Member

@dhardy dhardy left a comment

Choose a reason for hiding this comment

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

We tried to minimise the runners we need, but you are right that we should test more comprehensively. This approach is little extra config which I like; I don't see sufficient motivation to re-write based on the libc setup.

@@ -104,6 +110,9 @@ matrix:
env: DESCRIPTION="beta Rust release"

- rust: nightly
os:
- linux
- osx
Copy link
Member

Choose a reason for hiding this comment

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

This could cause a problem: it does a doc-upload at the end. Perhaps you can prefix that:

test "$TRAVIS_OS_NAME" = "linux" && travis-cargo --only nightly doc-upload

@jonathanKingston
Copy link

I know @jcjones was having some trouble with OSX that he hadn't raised as an issue yet: mozilla/authenticator-rs#76
It looks like libc::pthread_atfork isn't available outside linux so was causing us some issues.

@dhardy
Copy link
Member

dhardy commented Feb 11, 2019

@jonathanKingston or @jcjones please create a new issue about that.

I guess I'll make the suggested change myself and merge this manually.

@dhardy dhardy merged commit 47794be into rust-random:master Feb 11, 2019
@dhardy
Copy link
Member

dhardy commented Feb 11, 2019

This doesn't appear to include all the expected builds. Any idea why?

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