Skip to content

move the built cratesfyi into the vagrant container #218

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
Jul 6, 2018

Conversation

QuietMisdreavus
Copy link
Member

Works around rust-lang/rust#49710 when developing docs.rs with a Windows host.

This sets the CARGO_TARGET_DIR environment variable within the vagrant box to a folder inside the container rather than in the share. This means that when building the cratesfyi executable or the docs.rs service, the files will be placed inside the container (in /home/cratesfyi/target) rather than in /vagrant. This does mean that the binaries will be unavailable outside the container without manually copying them out. However, without this change, any build will fail with an error like the one in the linked issue. Combined with https://github.com/onur/docs.rs/pull/216, I can fully build docs.rs from a Windows host.

@onur
Copy link
Member

onur commented Jul 6, 2018

Thanks @QuietMisdreavus, but that vagrant image really needs some work. I am planning make a great return to this project really soon.

@onur onur merged commit 07896a9 into rust-lang:master Jul 6, 2018
@QuietMisdreavus QuietMisdreavus deleted the vagrant-target branch July 6, 2018 18:58
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.

2 participants