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

Assume that a build.rs file is a build script #3664

Merged
merged 1 commit into from
Feb 7, 2017

Conversation

pwoolcoc
Copy link
Contributor

@pwoolcoc pwoolcoc commented Feb 7, 2017

If cargo sees a build.rs file in the same directory as the current
Cargo.toml, it will assume that the build.rs file is a build script,
unless there is build = false in the Cargo.toml file.

Closes #3391

If cargo sees a `build.rs` file in the same directory as the current
`Cargo.toml`, it will assume that the `build.rs` file is a build script,
_unless there is_ `build = false` _in the _ `Cargo.toml` _file_.

Closes rust-lang#3391
@rust-highfive
Copy link

r? @alexcrichton

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

@alexcrichton
Copy link
Member

@bors: r+

Thanks!

@bors
Copy link
Collaborator

bors commented Feb 7, 2017

📌 Commit 952b0ab has been approved by alexcrichton

@bors
Copy link
Collaborator

bors commented Feb 7, 2017

⌛ Testing commit 952b0ab with merge ce9ddf3...

bors added a commit that referenced this pull request Feb 7, 2017
Assume that a `build.rs` file is a build script

If cargo sees a `build.rs` file in the same directory as the current
`Cargo.toml`, it will assume that the `build.rs` file is a build script,
unless there is `build = false` in the  `Cargo.toml` file.

Closes #3391
@bors
Copy link
Collaborator

bors commented Feb 7, 2017

☀️ Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing ce9ddf3 to master...

@bors bors merged commit 952b0ab into rust-lang:master Feb 7, 2017
@alexcrichton alexcrichton added the relnotes Release-note worthy label Feb 8, 2017
alexcrichton added a commit to alexcrichton/roboime-next that referenced this pull request Mar 9, 2017
In rust-lang/rust#40197 it was discovered that this crate no longer builds on
nightly due to a rust-lang/cargo#3664 where Cargo will now infer that `build.rs`
is a build script unless explicitly configured with `build = false`.

Currently we're not intending to revert the Cargo change, but let me know if
this causes trouble though and we can certainly reconsider!
@ehuss ehuss added this to the 1.17.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
relnotes Release-note worthy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tracking issue for PR #3361
5 participants