Skip to content

rustbuild: Fix some parts with out of tree builds #31620

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 2 commits into from
Feb 20, 2016

Conversation

alexcrichton
Copy link
Member

This removes creating some extraneous directories and also fixes some submodule management with out of tree builds.

Closes #31619

As part of the ./configure step don't create any directories if rustbuild is
enable as rustbuild will take care of everything.
@rust-highfive
Copy link
Contributor

r? @brson

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

Be sure to put the git command into the right directory when we run the
submodule management information.

Closes rust-lang#31619
@alexcrichton alexcrichton force-pushed the fix-out-of-tree-builds branch from dbc5521 to 5aef0e8 Compare February 13, 2016 01:16
@brson
Copy link
Contributor

brson commented Feb 20, 2016

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 20, 2016

📌 Commit 5aef0e8 has been approved by brson

@bors
Copy link
Collaborator

bors commented Feb 20, 2016

⌛ Testing commit 5aef0e8 with merge df128bd...

bors added a commit that referenced this pull request Feb 20, 2016
This removes creating some extraneous directories and also fixes some submodule management with out of tree builds.

Closes #31619
@bors bors merged commit 5aef0e8 into rust-lang:master Feb 20, 2016
@alexcrichton alexcrichton deleted the fix-out-of-tree-builds branch February 22, 2016 03:17
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.

[rustbuild] error on out of source build: fatal: Not a git repository
4 participants