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

1.43 beta backports #8007

Merged
merged 3 commits into from
Mar 17, 2020
Merged

1.43 beta backports #8007

merged 3 commits into from
Mar 17, 2020

Conversation

ehuss
Copy link
Contributor

@ehuss ehuss commented Mar 16, 2020

…excrichton

Partially revert change to filter debug_assertions.

This partially reverts the changes from rust-lang#7943. It caused a regression with the rocket_contrib crate. I knew that was the only crate that had a `cfg(debug_assertions)` dependency, and I saw that it had been fixed, but I did not realize the fix hadn't been published (and will be in a semver incompatible release).

This retains the old behavior for `cfg(debug_assertions)` of issuing a warning. I kept the filter for `CARGO_CFG_DEBUG_ASSERTIONS` for build scripts because that was the original intent for the change, and I don't see anyone using that.

Closes rust-lang#7966.
…crichton

Support old html anchors in manifest chapter.

rust-lang#7733 unintentionally broke some old HTML anchors in the manifest chapter.  This would cause any links out in the wild to not scroll to the correct position.

Unfortunately it is too late for the 1.42 release.  However, I'd like to backport this for 1.43.
@rust-highfive
Copy link

r? @steveklabnik

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

@rust-highfive
Copy link

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.43.0. Please double check that you specified the right target!

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 16, 2020
@ehuss
Copy link
Contributor Author

ehuss commented Mar 16, 2020

r? @alexcrichton

@ehuss
Copy link
Contributor Author

ehuss commented Mar 16, 2020

I also considered instead of backporting #7996 to revert #7939. I don't feel strongly one way or the other. The latest libgit2 probably hasn't had much testing done, but presumably there were some important fixes in the newer versions.

This will require a coordinated update with clippy due to the git2 changes ("links" restriction), which I'll try to handle.

@alexcrichton
Copy link
Member

@bors: r+

I think updating a bit further for git2 is fine

@bors
Copy link
Contributor

bors commented Mar 17, 2020

📌 Commit fb5abcd has been approved by alexcrichton

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 17, 2020
@bors
Copy link
Contributor

bors commented Mar 17, 2020

⌛ Testing commit fb5abcd with merge 7b1e9f5...

@bors
Copy link
Contributor

bors commented Mar 17, 2020

☀️ Test successful - checks-azure
Approved by: alexcrichton
Pushing 7b1e9f5 to rust-1.43.0...

@bors bors merged commit 7b1e9f5 into rust-lang:rust-1.43.0 Mar 17, 2020
@ehuss ehuss added this to the 1.43.0 milestone Feb 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants