Skip to content

Clarify rustbuild + msvc + vcvars in README #34690

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 9, 2016

Conversation

alexcrichton
Copy link
Member

The invocation of vcvars is only needed for versions of Visual Studio that
rustbuild or cmake doesn't understand, but if older versions are installed then
there's no need to call vcvars.

Closes #34576

@rust-highfive
Copy link
Contributor

r? @aturon

(rust_highfive has picked a reviewer for you, use r? to override)

@alexcrichton
Copy link
Member Author

r? @brson

cc @retep998

@rust-highfive rust-highfive assigned brson and unassigned aturon Jul 6, 2016
@retep998
Copy link
Member

retep998 commented Jul 6, 2016

Seems fine.

newer with the C++ tools. Then all you need to do is invoke the appropriate
vcvars bat file and kick off rustbuild.
The old build system, based on makefiles, is currently being rewritten into a
Rust-based build system called rustbuild. This can be used to boostrap the
Copy link
Contributor

@durka durka Jul 6, 2016

Choose a reason for hiding this comment

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

👻 (boostrap)

@brson
Copy link
Contributor

brson commented Jul 6, 2016

r=me

The invocation of vcvars is only needed for versions of Visual Studio that
rustbuild or cmake doesn't understand, but if older versions are installed then
there's no need to call vcvars.

Closes rust-lang#34576
@alexcrichton
Copy link
Member Author

@bors: r=brson

@bors
Copy link
Collaborator

bors commented Jul 7, 2016

📌 Commit b67f23c has been approved by brson

bors added a commit that referenced this pull request Jul 8, 2016
Clarify rustbuild + msvc + vcvars in README

The invocation of vcvars is only needed for versions of Visual Studio that
rustbuild or cmake doesn't understand, but if older versions are installed then
there's no need to call vcvars.

Closes #34576
@bors
Copy link
Collaborator

bors commented Jul 8, 2016

⌛ Testing commit b67f23c with merge 5e18b4b...

@bors bors merged commit b67f23c into rust-lang:master Jul 9, 2016
@alexcrichton alexcrichton deleted the clarify-vcvars branch August 26, 2016 18:30
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.

7 participants