Skip to content

Don't require all submodules are checked out#1250

Merged
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:issue-1248
Jan 30, 2015
Merged

Don't require all submodules are checked out#1250
bors merged 1 commit into
rust-lang:masterfrom
alexcrichton:issue-1248

Conversation

@alexcrichton

Copy link
Copy Markdown
Member

When probing a repository for files that should be considered as inputs for a
build script we should just skip submodules that haven't been checked out
instead of throwing an error.

Closes #1248

When probing a repository for files that should be considered as inputs for a
build script we should just skip submodules that haven't been checked out
instead of throwing an error.

Closes rust-lang#1248
@rust-highfive

Copy link
Copy Markdown

r? @huonw

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

@brson

brson commented Jan 30, 2015

Copy link
Copy Markdown
Contributor

@bors: r+ 98bd

@bors

bors commented Jan 30, 2015

Copy link
Copy Markdown
Contributor

⌛ Testing commit 98bd1f7 with merge 2b9a3d6...

bors added a commit that referenced this pull request Jan 30, 2015
When probing a repository for files that should be considered as inputs for a
build script we should just skip submodules that haven't been checked out
instead of throwing an error.

Closes #1248
@bors

bors commented Jan 30, 2015

Copy link
Copy Markdown
Contributor

☀️ Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-32, cargo-win-64

@bors bors merged commit 98bd1f7 into rust-lang:master Jan 30, 2015
@alexcrichton alexcrichton deleted the issue-1248 branch January 30, 2015 07:49
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.

Build script can no longer initialize git submodules

5 participants