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

OSX and Windows CI #74

Merged
merged 4 commits into from Sep 28, 2017

Conversation

Projects
None yet
3 participants
@opilar
Copy link
Contributor

opilar commented Apr 13, 2017

Partially resolves #65.

ma.rybakov added some commits Apr 13, 2017

ma.rybakov
@Kimundi

This comment has been minimized.

Copy link
Contributor

Kimundi commented Apr 14, 2017

Thanks! Could you add a new line at the end to both config files though?

ma.rybakov
@Kimundi

This comment has been minimized.

Copy link
Contributor

Kimundi commented Apr 18, 2017

Seems like osx does not like the way travis-cargo is called there.

@opilar

This comment has been minimized.

Copy link
Contributor Author

opilar commented Apr 18, 2017

Yes. Check out my issue: huonw/travis-cargo#70

Valid PATH manipulation on macOS
`pip install package-name --user` installs binaries to USER_SITE.
USER_SITE is different on different systems:
https://docs.python.org/3/library/site.html#site.USER_SITE

Using `python -m site --user-base` makes travis-cargo valid command:
https://docs.python.org/3/library/site.html#cmdoption-site--user-site
@KodrAus

This comment has been minimized.

Copy link
Contributor

KodrAus commented Sep 19, 2017

Is there anything else blocking this @Kimundi? I guess we'll need to enable AppVeyor for this repo.

@Kimundi

This comment has been minimized.

Copy link
Contributor

Kimundi commented Sep 28, 2017

Seems nothing else is blocking it. I'll merge and see if I can get appvoyer running.

@Kimundi Kimundi merged commit 34796c6 into rust-lang-nursery:master Sep 28, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.